Skip to content

feat: Visual View proposals dashboard - #130

Open
com55 wants to merge 15 commits into
mainfrom
feat/proposals-visual-view
Open

com55 wants to merge 15 commits into
mainfrom
feat/proposals-visual-view

Conversation

@com55

@com55 com55 commented Nov 11, 2026

Copy link
Copy Markdown
Owner

Summary

Replace the /ai-access sidebar with a sticky dashboard header, and turn Proposals into a summary-only list plus a Visual View detail page, matching the approved mockup.

  • Sticky header: logo + “AI Access”, text nav (Overview, Access tokens, MCP OAuth, Proposals) with the existing sidebar icons, and Settings / Audit log as icon buttons. On mobile, a left hamburger opens a drawer with all items (Settings and Audit log after a divider).
  • Summary-only proposal list with Pending / Conflict / Resolved / All filters, change-count chips, and an inline deploy-pending warning after the title. Rows open the detail page; Approve and Reject are not in the list.
  • Detail page: Visual View canvas (change highlights, related-node pills, Compact / Editor positions, moved-node ghosts), outline “← Proposals” on the left, Reject + Node-RED-style Approve split on the right. The approve type defaults from Settings defaultDeploymentType and both segments are disabled while a conflict is unresolved.
  • The link returned after submitting a proposal (reviewUrl) now opens #/proposals/<id>. Legacy #proposal=<id> still opens the same detail page.

Screenshots

Taken from the plugin running on a throwaway Node-RED at port 1890:

Test plan

  • yarn type-check, lint, format, tests, and build
  • Playwright checks of the list, detail, Approve menu, and mobile drawer on throwaway Node-RED 5 and 4 (port 1890 only)

@com55
com55 force-pushed the feat/proposals-visual-view branch from 332e6c1 to 0b796e3 Compare November 11, 2026 10:13
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.

1 participant