docs: restructure sidebar navigation for clarity#747
docs: restructure sidebar navigation for clarity#747devin-ai-integration[bot] wants to merge 6 commits intomainfrom
Conversation
Co-Authored-By: Jem Conlon <jem@walletconnect.com>
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
|
Claude finished @devin-ai-integration[bot]'s task in 1m 30s —— View job PR Review
Found 4 issue(s)Issue 1: Anchor objects placed inside
|
Co-Authored-By: Jem Conlon <jem@walletconnect.com>
Co-Authored-By: Jem Conlon <jem@walletconnect.com>
Co-Authored-By: Jem Conlon <jem@walletconnect.com>
Co-Authored-By: Jem Conlon <jem@walletconnect.com>
…Plans section Co-Authored-By: Jem Conlon <jem@walletconnect.com>
Description
Restructures the AppKit Overview sidebar to improve discoverability and organization, and rewrites the overview page content. Requested by Jem Conlon.
Sidebar changes (
docs.json):Page content changes (
appkit/overview.mdx):appkit/features/index.mdxinto this page#featuresanchor link (removed reference to deleted heading)Known limitations / items for review
pagesarray — doing so broke the entire AppKit Overview dropdown. Anchors have been left in the originalglobal.anchorsconfig, which renders them at the top of every sidebar.appkit/authentication/siwx/defaultacross two sidebar groups also contributed to the dropdown routing failure. SIWX remains accessible under Accounts & Authentication > Authentication.appkit/features/index.mdxis no longer in the sidebar but the file still exists. Consider whether a redirect from the old URL is needed.<video>element (appkit-overview-demo.mp4) was removed as part of replacing the intro text. Verify this is intended.Items requiring human review
Updates since last revision
Link to Devin run
Tests
Direct link to the deployed preview files