Repository navigation
Move Annual Reports to subpage under status. - #350
Conversation
Structure the Annual Report landing page to include the current report, all other pages are listed in the leftside pane. Updates references to the annual report in other parts of the code. Adds a new shortcode to insert a page (no need to duplicate content).
|
Is there a preview on the staging site? It looks good but @hjellinek may want to take a look too. |
|
@pamoroso The build did not complete successfully so a staging site wasn't deployed. The failure wasn't due to these changes but an unrelated bibliography change - a new type bibliography entry type was added - Transcript. This issue will be resolved in a separate PR. |
How should I review it without a staging site to look at? |
Longterm fix for new item type will be in a seperate PR.
|
Staging preview for this pull request is available at https://interlisp.github.io/Interlisp.staging/pr-350/ |
masinter
left a comment
There was a problem hiding this comment.
The strucure is confusing:
There's no header per se for the 2025 Annual Rerport at https://interlisp.org/Interlisp.staging/pr-350/project/status/reports/
That page just flows from the header Annual Reports (signifying all of them) into the 2025 report.
|
I second Larry's feedback. I would expect the /project/status/reports/ page to have something like a bullet list with links to the reports. |
|
Staging preview for this pull request is available at https://interlisp.github.io/Interlisp.staging/pr-350/ |
1 similar comment
|
Staging preview for this pull request is available at https://interlisp.github.io/Interlisp.staging/pr-350/ |
|
The organization of commit 408e424 looks good. |
Structure the Annual Report landing page to include the current report, all other pages are listed in the leftside pane.
Updates references to the annual report in other parts of the code.
Adds a new shortcode to insert a page (no need to duplicate content).