Skip to content

chore: UI cleanup and reference link fix - #5

Merged
magnusgp merged 1 commit into
mainfrom
dev
Nov 6, 2025
Merged

magnusgp merged 1 commit into
mainfrom
dev

Conversation

@magnusgp

@magnusgp magnusgp commented Nov 6, 2025

Copy link
Copy Markdown
Owner

This pull request introduces improvements to both backend and frontend components, focusing on user experience, visual consistency, and share URL correctness. The backend now ensures share URLs are generated without redundant path components, and a corresponding test has been added. On the frontend, UI elements for session history and insights dialogs have been visually refined, and the insights toolbar now disables unavailable options with clearer messaging.

Backend: Share URL generation and testing

  • Refactored InsightService._build_share_url to use urlsplit, ensuring that share URLs do not include extraneous path components from the configured base URL.
  • Added a new test test_share_url_strips_path_component to verify that share URLs are correctly generated without the /insights path.
  • Imported urlsplit from urllib.parse to support the URL parsing logic.

Frontend: UI/UX improvements

  • Refined the SessionHistory component layout for better readability and alignment, updated color classes for improved accessibility, and adjusted button placement for consistency. [1] [2] [3] [4]
  • Enhanced the InsightsReportDialog component with updated styling, clearer sectioning, and improved button states for a more polished stakeholder report experience. [1] [2]

Frontend: Insights toolbar usability

  • Updated the WINDOW_OPTIONS in InsightsToolbar to disable the "7 days" option, add descriptive messaging, and prevent user interaction with unavailable options. [1] [2]

@vercel

vercel Bot commented Nov 6, 2025

Copy link
Copy Markdown

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

Project Deployment Preview Comments Updated (UTC)
clipnotes Ready Ready Preview Comment Nov 6, 2025 5:07pm

@magnusgp
magnusgp merged commit f09bb77 into main Nov 6, 2025
4 checks passed

This branch was successfully deployed

1 active deployment
Preview — 9b46bd10 Deployed Nov 6, 2025 by vercel[bot]
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