Skip to content

feat(kno-12444): document guides toolbar#1384

Open
scotidodson wants to merge 10 commits intomainfrom
sd-KNO-12444-document-guides-toolbar
Open

feat(kno-12444): document guides toolbar#1384
scotidodson wants to merge 10 commits intomainfrom
sd-KNO-12444-document-guides-toolbar

Conversation

@scotidodson
Copy link
Copy Markdown
Contributor

@scotidodson scotidodson commented Apr 8, 2026

KNO-12444: [Docs] Document guides toolbar

Summary

  • Restructured debugging-guides.mdx to provide better TOC navigation options and add guides toolbar in place of live preview documentation
  • Added full guides toolbar documentation: activation, toolbar views, status badges, eligibility/visibility indicators, focus button, and toolbar controls
  • Moved live preview to a FAQ accordion for users on older SDK versions
  • Removed the old "Testing guides" wrapper section and flattened the page structure

Changes

  • content/in-app-ui/guides/debugging-guides.mdx — restructured page; added guides toolbar section with SDK requirement callout; moved live preview to FAQ accordion; updated dashboard preview callout to reference toolbar
  • Removed a section at the bottom about engagement request debugging which needs to be updated. I'll ticket that as a follow up.

TODO

  • Update based on newest designs
  • Add images where helpful

@linear
Copy link
Copy Markdown

linear bot commented Apr 8, 2026

@vercel
Copy link
Copy Markdown

vercel bot commented Apr 8, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview, Comment Apr 10, 2026 6:28pm

Request Review

@scotidodson scotidodson changed the title docs(kno-12444): document guides toolbar feat(kno-12444): document guides toolbar Apr 10, 2026
scotidodson and others added 8 commits April 10, 2026 11:22
Co-authored-by: Thomas <thomas.whyyou@gmail.com>
Co-authored-by: Thomas <thomas.whyyou@gmail.com>
Co-authored-by: Thomas <thomas.whyyou@gmail.com>
Co-authored-by: Thomas <thomas.whyyou@gmail.com>
Co-authored-by: Thomas <thomas.whyyou@gmail.com>
Co-authored-by: Thomas <thomas.whyyou@gmail.com>
Co-authored-by: Thomas <thomas.whyyou@gmail.com>
Co-authored-by: Thomas <thomas.whyyou@gmail.com>
Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Risk MEDIUM: Documents the guides toolbar feature by restructuring the debugging-guides page and adding new images.

Reasons

  • The MDX file debugging-guides.mdx has 104 additions and 70 deletions, qualifying as a substantial content rewrite (30+ additions AND 30+ deletions)
  • Two new images are added to /public/images/ (Guides_ToolbarEx.png, Guides_ToolbarSettings.png)
  • No component, layout, library, or configuration files are modified, so this does not reach HIGH risk
  • Changes are confined to a single documentation section (content/in-app-ui/guides/), which avoids the cross-section MEDIUM trigger
  • No sidebar or navigation changes are included

Notes

  • Verify the page restructuring maintains correct heading hierarchy and TOC behavior after flattening the old "Testing guides" wrapper section
  • Confirm the two new images render correctly at various viewport sizes
  • The PR description mentions TODOs ("Update based on newest designs", "Add images where helpful") — reviewer should confirm the current state is ready to ship or if this is still a draft
  • The old "engagement request debugging" section was removed; the author mentions a follow-up ticket, so verify nothing critical is lost in the interim
  • Check that the live preview content moved to the FAQ accordion is still accurate and accessible for users on older SDK versions
Open in Web View Automation 

Sent by Cursor Automation: Docs PR classifier

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants