Skip to content

feat(ui): consolidate document actions and release notes - #242

Open
ThisIs-Developer wants to merge 22 commits into
mainfrom
agent/consolidate-icon-toolbar
Open

feat(ui): consolidate document actions and release notes#242
ThisIs-Developer wants to merge 22 commits into
mainfrom
agent/consolidate-icon-toolbar

Conversation

@ThisIs-Developer

@ThisIs-Developer ThisIs-Developer commented Aug 11, 2026

Copy link
Copy Markdown
Owner

What changed

  • Consolidated document actions into bordered icon controls with dropdown arrows where menus are available.
  • Ordered document actions as View, Sync, New, Copy, Export, Share, Live Share, and Review, followed by application preferences.
  • Reorganized the Markdown formatting toolbar into standard History, Text, Paragraph, Content, Technical, Additional, and Workspace groups.
  • Refined theme and private-mode switches while preserving the requested active colors and icon behavior.
  • Treats Release Notes as an application-owned reader surface, disables document-only controls, and removes its document overflow menu.
  • Adds stable active states and directional, reduced-motion-aware transitions to dropdown menus.
  • Redesigns Comments & Suggestions as a neutral enterprise-style panel using existing CSS theme tokens.
  • Uses one shared segmented-control design for review and Explorer filters.
  • Makes Open, Resolved, and All mathematically equal-width with a fixed 3px outer inset and 3px inter-tab gap in every active state.
  • Applies a compact 11/12/13px interface type scale and a 50px desktop review header.
  • Moves Edit, Resolve/Reopen, and Delete into each thread header as grouped Lucide icon buttons with accessible names.
  • Modernizes review threads with a separate neutral header, structured content region, subtle elevation, clearer anchor hierarchy, and an icon-led metadata footer.
  • Uses clock, open-state, and resolved-state icons for review timestamps without removing their readable labels.
  • Rebuilds the feedback composer as a compact header, body, and footer card with clearer hierarchy.
  • Removes the redundant “Feedback for” eyebrow and removes icons from Comment and Suggestion.
  • Makes Comment and Suggestion use the same neutral, equal-width segmented-control styling as Open, Resolved, and All.
  • Reserves blue for the enabled submit action while retaining a visible field label, character count, focus states, and disabled state.
  • Keeps review actions at 40–44px on touch layouts with 6–8px separation and reduced-motion support.
  • Mirrors all shared UI changes into the desktop resources.

Why

The toolbar and review workflow now communicate state more consistently. The previous Comments & Suggestions design spread accent color and visual weight across too many elements, while the filter active effect, thread hierarchy, and feedback form felt uneven. Neutral structure, mathematically consistent segmented spacing, and compact card sections make the panel easier to scan and understand.

The composer now follows the same control language as the rest of the review panel. It removes redundant copy, keeps the feedback type choice visually secondary, and leaves the submit button as the single primary action.

Dropdown menus also retain their positioned surface during a short exit phase, avoiding the snap caused when Bootstrap removed positioning immediately on close.

Testing

  • npm run check:static
  • npm test completed with 115 passing tests; one unrelated header dropdown-width measurement missed its threshold by about 2px during the parallel run and passed immediately on isolated rerun.
  • 12 focused Chromium review workflow scenarios passed.
  • Dedicated design tests verify equal filter and composer-choice widths, 3px spacing in every active state, removal of redundant composer copy/icons, neutral selected styling, single-primary-action color use, structured composer/thread markup, icon action labels, timestamp icons, dark/light themes, 375px portrait, small landscape, 44px mobile targets, mobile fit, and reduced motion.
  • Existing Explorer filter, review persistence, deletion, resolution, orphan handling, live rendering, toolbar, Release Notes, dropdown, share, and snapshot tests remain covered.
  • Manual screenshot QA completed for the composer in desktop light and dark modes.

@vercel

vercel Bot commented Aug 11, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
markdown-viwer Ready Ready Preview Aug 16, 2026 10:01am

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Aug 11, 2026

Copy link
Copy Markdown

Deploying markdownviewer with  Cloudflare Pages  Cloudflare Pages

Latest commit: ec8612c
Status: ✅  Deploy successful!
Preview URL: https://a8bcd982.markdownviewer.pages.dev
Branch Preview URL: https://agent-consolidate-icon-toolb.markdownviewer.pages.dev

View logs

@ThisIs-Developer ThisIs-Developer changed the title feat(toolbar): consolidate document actions feat(editor): add Markdown productivity tools Aug 12, 2026
@ThisIs-Developer
ThisIs-Developer marked this pull request as ready for review August 12, 2026 17:11
@ThisIs-Developer
ThisIs-Developer force-pushed the agent/consolidate-icon-toolbar branch from 40a744b to 28ba18e Compare August 12, 2026 17:27
@ThisIs-Developer ThisIs-Developer changed the title feat(editor): add Markdown productivity tools feat(toolbar): consolidate document actions Aug 12, 2026
@ThisIs-Developer ThisIs-Developer changed the title feat(toolbar): consolidate document actions feat(ui): consolidate document actions and release notes Aug 12, 2026
@ThisIs-Developer

Copy link
Copy Markdown
Owner Author

@copilot resolve the merge conflicts in this pull request

Co-authored-by: ThisIs-Developer <109382325+ThisIs-Developer@users.noreply.github.com>

Copilot AI commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

@copilot resolve the merge conflicts in this pull request

Resolved by merging origin/main into this branch and fixing the conflict in tests/e2e/tab-split-sidebar-update.spec.js in commit 8befe8e.

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