Repository navigation
feat(portal): Page layouts and nav reorg - #7548
Open
cloudjumpercat wants to merge 5 commits into
Open
cloudjumpercat wants to merge 5 commits into
cloudjumpercat wants to merge 5 commits into
Conversation
Signed-off-by: Diana <75819066+cloudjumpercat@users.noreply.github.com>
✅ Deploy Preview for kongdeveloper ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Signed-off-by: Diana <75819066+cloudjumpercat@users.noreply.github.com>
cloudjumpercat
marked this pull request as ready for review
October 16, 2026 17:02
Contributor
|
✅ KSAI Review: Finished Run report (federated) · 1 paid run · $0.2093 total
Reviewing cost $0.2093 |
cloudjumpercat
added
the
do not merge
![kong-ksai[bot]](https://avatars.githubusercontent.com/in/4495376?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
| Check | Result |
|---|---|
| Scope | Dev Portal documentation only: a sidebar navigation bullet and a new Page layouts section. |
| Mandate | default-code-review |
| Findings | 1 Medium, 1 Low |
| Findings audit | Independent audit completed: both findings verified in the tree; the endpoint wording corrected, severities unchanged. |
The change documents Dev Portal page layouts and sidebar navigation. One documented customization path is absent from the repository's Portal Management spec, and the layout list repeats one link three times.
Comment on lines
+136
to
+137
| * `reference`: Contains [sidebar navigation](/dev-portal/customizations/dev-portal-customizations/#dev-portal-navigation) and content. Use this for shorter conceptual pages. | ||
| * `wide`: Contains [sidebar navigation](/dev-portal/customizations/dev-portal-customizations/#dev-portal-navigation) and content, but the contents space has a wider column. Use this for content that needs more horizontal room, like a wide table or a generated API reference. |
Contributor
There was a problem hiding this comment.
🔵 Low nit
The guide bullet above already links the sidebar navigation section. The reference and wide bullets repeat that link and its wording. Keep the link once.
Suggested change
| * `reference`: Contains [sidebar navigation](/dev-portal/customizations/dev-portal-customizations/#dev-portal-navigation) and content. Use this for shorter conceptual pages. | |
| * `wide`: Contains [sidebar navigation](/dev-portal/customizations/dev-portal-customizations/#dev-portal-navigation) and content, but the contents space has a wider column. Use this for content that needs more horizontal room, like a wide table or a generated API reference. | |
| * `reference`: Contains sidebar navigation and content. Use this for shorter conceptual pages. | |
| * `wide`: Contains sidebar navigation and content, but the contents space has a wider column. Use this for content that needs more horizontal room, like a wide table or a generated API reference. |
React 👍 if this finding helped, 👎 if it did not.
adamdehaven
reviewed
Oct 16, 2026
Member
There was a problem hiding this comment.
I believe this should be updated to include the new one as well?
| * **Footer bottom menu**: Footer bottom menu is a flat list of links that will be added to the bottom of every page. Footer bottom menus are placed below footer menu sections. | ||
| * **Sidebar navigation**: The sidebar navigation menu is a vertical list of links that displays on the left side of the page. | ||
| You can configure this in the **Sidebar Navigation** section of the Portal Editor or using the [`/portals/{portalId}/customization/navigations` endpoint](/api/konnect/portal-management/). | ||
| Whether a sidebar menu is displayed depends on the [page layout](/dev-portal/pages-and-content/#page-layouts). |
Member
There was a problem hiding this comment.
This sentence is very important and IMO should stand out more; without it, the UI doesn't render. Can we call this out better?
|
|
||
| The following layouts are available: | ||
|
|
||
| * `custom` (default): No built-in chrome. You build the whole page yourself, including sidebars and header. Use this for a landing page or anything the other layouts don't fit. Because `custom` doesn't render a title or description from front matter, give the page its own heading in the body. |
Member
There was a problem hiding this comment.
suggestion: since we want customers to utilize the other layouts by default, can we move custom to the end of the list?
|
|
||
| Any section a layout renders (a sidebar or the header) can be replaced with a snippet using the `layout-options` front matter property. | ||
|
|
||
| For information about how to customize a layout's sidebars and header, or configure page navigation, see the [Markdown Components reference](https://portaldocs.konghq.com/). |
Member
There was a problem hiding this comment.
Links you may use (currently unpublished):
- Pages (including front matter):
https://portaldocs.konghq.com/pages/overview - Layout options:
- Layouts:
https://portaldocs.konghq.com/pages/layouts - Layout options:
https://portaldocs.konghq.com/pages/layout-options
Co-authored-by: Adam DeHaven <2229946+adamdehaven@users.noreply.github.com>
Signed-off-by: Diana <75819066+cloudjumpercat@users.noreply.github.com>
tomek-labuk
requested changes
Oct 21, 2026
tomek-labuk
left a comment
tomek-labuk
left a comment
Contributor
There was a problem hiding this comment.
To me, this feels like content for a reference page rather than a landing page. Adding screenshots in a column next to the text breaks the layout and makes larger chunks text hardly readable
Signed-off-by: Diana <75819066+cloudjumpercat@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Fixes #7066 #7391
Preview Links
Broken links are expected since the MDC docs aren't live yet.
Checklist
descriptionentry in frontmatter.