Sidebar + sheet polish: conditional Add button, Export icon, aligned floating page controls#69
Merged
Merged
Conversation
With the empty-state hero now carrying the add action, the left rail's button was redundant before any photo existed. Hide it until there's at least one photo and move it below the strip, pinned to the bottom.
The shape toggle and page label sat in a row above the sheet, dropping the sheet's top edge below where both sidebars begin. Overlay them on the sheet's top margin instead (screen-only, never in the PDF) so the sheet top lines up with the photo strip and the options panel.
Pinning it with mt-auto alone parked it at the bottom of the full-height pane, off-screen until you scrolled. Stick it to the viewport bottom so it stays reachable.
The overlay read like a raised button. Switch to a translucent white glass with a hairline border, drop the shadow, and shrink the toggle so it sits quietly on the sheet. Styled for white paper so it stays legible in dark mode too.
It overlapped the sheet's margin guide. Show just the current shape as a small glassy icon at rest, tucked into the corner, and reveal the full three-way selector on hover or focus (focus so touch works too).
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.
A round of UI polish on top of the warmer identity.
Reworked the sheet's width measurement onto a callback ref so it still attaches when the empty-state hero gives way to the first real page.
Typecheck, lint, and build all pass.