Link to basic navigation information in Using OpenSpace -> Navigation - #59
Link to basic navigation information in Using OpenSpace -> Navigation#59WeirdRubberDuck wants to merge 2 commits into
Conversation
Reuse existing page in Getting Started section
There was a problem hiding this comment.
🟡 Changes recommended
The new redirect page uses a body-level <meta http-equiv="refresh"> that can be ignored by browsers; switching to a head-emitted meta directive avoids a broken redirect.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
This PR makes “basic navigation” easier to discover from the Using OpenSpace → Navigation section by linking to the existing Getting Started navigation content and adding a sidebar entry that redirects to it.
Changes:
- Updated
using-openspace/navigation/index.mdto point readers to the Getting Started navigation page and reorganized navigation subtopics. - Renamed the
camera-pathspage title to “Automatic Flight Paths”. - Added a new “Navigation Basics (Mouse and Keyboard)” page intended to redirect to the Getting Started navigation page.
File summaries
| File | Description |
|---|---|
| using-openspace/navigation/index.md | Adds a prominent link to Getting Started navigation and includes a hidden toctree entry for the new sidebar “basic navigation” page. |
| using-openspace/navigation/camera-paths.md | Updates the page title for clarity. |
| using-openspace/navigation/basic-navigation.md | Introduces a redirect-style “dummy page” for sidebar discoverability. |
Review details
- Files reviewed: 3/3 changed files
- Comments generated: 2
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| @@ -0,0 +1,5 @@ | |||
| # Navigation Basics (Mouse and Keyboard) | |||
|
|
|||
| <meta http-equiv="refresh" content="0; url=../../getting-started/navigation/index.html" /> | |||
There was a problem hiding this comment.
This suggestion results in an error:
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
alexanderbock
left a comment
alexanderbock
left a comment
There was a problem hiding this comment.
Should we add a redirect via the readthedocs interface instead?
https://docs.readthedocs.com/platform/latest/guides/redirects.html
Reuse the existing page in the Getting Started section for basic navigation.
I wanted to make the basic navigation also appear in the tree menu to the left, to make it easier to find when skimming through the available pages. So, I made a small "dummy page" that redirects to the one in the Getting Started section.
Let me know what you think of this redirection solution
closes OpenSpace/OpenSpace#4224