fix: punchcard tooltip cropping; remove menubar Workflow strip - #963
Merged
Conversation
…flow strip - Punchcard tooltips flipped below the cursor on the top rows (the overflow container clips anything above its own edge) and clamp horizontally near the strip edges. Applied to both the app and dash copies of the component. - The menubar Workflow strip is removed (view, strip model, tests): the popover is the compact surface and the workflow metrics live in the desktop app, dash and TUI. The payload keeps emitting the block (add-only contract; the other surfaces read it).
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Two user-requested polish items from live use: the punchcard tooltip cropped at the container's top edge (flip below cursor on top rows, clamp near edges - fixed in both the app and dash components), and the menubar drops its Workflow strip (the compact popover isn't the place; the metrics stay on app/dash/TUI, and the payload keeps emitting the block per the add-only contract). swift build + 137 tests green, app suite 468 green, dash build clean.