Repository navigation
Mobile layout fixes and one SeedQR button - #84
Merged
Merged
Conversation
- One "SeedQR" button; the QR window switches between SeedQR and Compact SeedQR and remembers the choice (Seed Workspace, BIP-85 child seeds and Seed XOR shares) - Seed input tabs put the label under the icon at every width - Word boxes keep room for the word number (a more specific input rule reset the padding, so typed words ran over the numbers) - Seed cards fit the longest BIP39 word: 2 columns on phones, up to 4 - Text boxes are sized once they are shown: textareaResize skips hidden boxes, adjustPanelHeight is exposed to shell.js (the re-measure never ran), and boxes are re-measured when the seed changes - Dropdowns never grow past their container (iPhone Safari sizes them to the longest option) - Sub-tabs share one width; backup sheet options line up Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
pingtozen
pushed a commit
to pingtozen/seedtool
that referenced
this pull request
Oct 14, 2026
PR BitcoinQnA#79 stops the generic dark input rule applying to word boxes, which fixes the overlap at its source, so the padding override added in BitcoinQnA#84 is no longer needed. Scoped update of two development packages flagged by Dependabot. Neither is in index.html: - shell-quote 1.10.0 to 1.12.0 (GHSA-pqg4-j6r4-53mv, critical), via browserify - brace-expansion 1.1.18 to 1.1.21 (GHSA-q2hr-2g5m-vwhr, medium) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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
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.
Summary
Fixes for the layout problems Q found testing 2.4.0 on a phone, plus one SeedQR button.
.view--tool input[type="text"]) reset the padding.textareaResize()squashed hidden boxes to 5px.adjustPanelHeightwas a top-levelconst, not onwindow.Testing
npm run release:checkpasses. New tests cover the SeedQR format data andwindow.adjustPanelHeight.🤖 Generated with Claude Code