Skip to content

feat: View plan details on hover - #906

Open
alyssaholland99 wants to merge 2 commits into
garethgeorge:mainfrom
alyssaholland99:feat-Plan_Details_On_Hover
Open

feat: View plan details on hover#906
alyssaholland99 wants to merge 2 commits into
garethgeorge:mainfrom
alyssaholland99:feat-Plan_Details_On_Hover

Conversation

@alyssaholland99

@alyssaholland99 alyssaholland99 commented Sep 17, 2025

Copy link
Copy Markdown

Added a tooltip to show the plan details when the user hovers over them so that the user doesn't have to go into the settings to view what they're doing.

@alyssaholland99

Copy link
Copy Markdown
Author
Screenshot from 2025-09-17 15-48-53 Screenshot from 2025-09-17 15-48-47 Screenshot from 2025-09-17 15-48-42 Screenshot from 2025-09-17 15-48-37

@alyssaholland99
alyssaholland99 force-pushed the feat-Plan_Details_On_Hover branch 3 times, most recently from e137cd8 to 7ab0f74 Compare September 21, 2025 19:02
@alyssaholland99
alyssaholland99 force-pushed the feat-Plan_Details_On_Hover branch from 7ab0f74 to 8365a64 Compare September 21, 2025 19:04
@garethgeorge

Copy link
Copy Markdown
Owner

Hi -- thanks for the interest in contributing, afraid for this one I have some UX / maintenance concerns.

introducing the tool tip means that JS logic to stringify the representation of the schedule must be maintained for every supported schedule type, I lean towards keeping one view of this implemented as the form UI.

The second issue I see is the paths included in the tooltip -- these aren't necessarily bounded to a small number, many users may have dozens or hundreds of paths included which would make for a massive tool tip.

@jburnham

jburnham commented Oct 2, 2025

Copy link
Copy Markdown

The second issue I see is the paths included in the tooltip -- these aren't necessarily bounded to a small number, many users may have dozens or hundreds of paths included which would make for a massive tool tip.

What about a maxPathLines and then an overflow + N more line at the end. Sort of like this example
image

It's definitely a UX feature I've seen before, but of course when you need a live example you can't find one.

@alyssaholland99

Copy link
Copy Markdown
Author
Screenshot from 2025-10-07 10-30-37 Screenshot from 2025-10-07 10-29-08

@alyssaholland99

Copy link
Copy Markdown
Author

@garethgeorge Is this any better for limiting the paths to 6, with the +x more?

@garethgeorge
garethgeorge force-pushed the main branch 3 times, most recently from 1a75a0a to aeade74 Compare July 10, 2026 09:45
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.

3 participants