Put the Eraser within reach of a pen, and give Preferences one line per setting - #83
Merged
Merged
Conversation
Off by default: the Eraser stays off the toolbar unless finger or mouse drawing puts it there, because the pen's reverse end already erases and the button costs the toolbar a row. A pen whose back end is not an eraser has no way to reach the tool at all, and this is it. Pan is deliberately not part of the option. It is on the toolbar for devices with no touch and no Space to reach it with, which is not a property of the pen in hand. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
A second row holding one button doubles the compact toolbar's height to say very little, so in Chevron and Icons-beside-the-size-chips the Eraser joins the row of tools instead. The dual palette is stacked groups already, so there a row beneath reads as one more group and the Eraser stays where it was. The button is one instance moved between the two hosts rather than two that have to be kept in step, and the layout setting moves it. Bumps VersionPrefix to 1.2.2. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Preferences had grown into a wall of prose: the longest description ran to six wrapped lines, and a category had to be read to be skipped. Every setting now carries a one-line summary that is always on the row, and the defaults, reasoning and consequences move behind a chevron. A setting whose summary is the whole story keeps an empty description and gets no chevron. Not a tooltip: this application is used with a pen and a finger, neither of which hovers, and on a Cintiq the reasoning would simply be gone. The chevron is a real button because pressing is the one gesture every input here can perform. A search that matches only the hidden prose opens the row it matched, so a hit never arrives looking like a mistake. The chevrons take a fixed column right of the editors. Between title and editor they landed somewhere different on every row, and they took the width the summary needed to stay on one line. The window goes to 820 so the widest combo cannot starve that column either. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Four changes to Preferences, from review of the first pass. The chevron moves under the title, in front of the summary. At the right-hand edge it landed somewhere different on every row, and on Snippet format order it sat in the same column as that editor's own reordering chevrons, where it read as one of them. Search now highlights its matches in the title, the summary and the prose. A match lying only in the prose has nothing on the row to mark, so the chevron is marked instead, and the automatic expansion that used to handle that case is gone: it made the list jump on every keystroke. A row matched only by a keyword marks nothing, because the word is not in the prose either. The search box carries a clear button while it holds anything. Always show the Eraser becomes two pictures of the toolbar rather than a switch. They are identical but for the Eraser, drawn in the accent color, and they follow the Layout setting - which is what decides whether the Eraser joins the bar or takes a row beneath it - so the editor is redrawn when that choice changes rather than going stale. Co-Authored-By: Claude Opus 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
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.
Some pens have no eraser end. On those, the Eraser tool could not be reached at all: the toolbar shows it only when finger or mouse drawing puts it there, because the pen's reverse end normally does the job. Help → Preferences → Toolbar → Always show the Eraser is the way to it, off by default so nothing changes for a pen that already erases. Pan is deliberately not part of the option — it is on the toolbar for devices with no touch and no Space to reach it with, which is not a property of the pen in hand.
In the compact toolbar layouts the Eraser joins the row of tools; a second row holding one button doubles the toolbar's height to say very little. The dual palette is stacked groups already, so there it stays on its own row beneath. The button is one instance moved between the two hosts rather than two kept in step, and changing the layout moves it.
Adding a sixteenth setting made the state of the Preferences dialog hard to ignore: the longest description ran to six wrapped lines, and a category had to be read in order to be skipped. Every setting now carries a one-line summary that is always on the row, with the defaults, reasoning and consequences behind a chevron under the title. A setting whose summary is the whole story keeps an empty description and gets no chevron.
The reasoning is not in a tooltip. This application is used with a pen and a finger, neither of which hovers, and on a Cintiq that text would simply be gone. The chevron is a real button because pressing is the one gesture every input here can perform.
Search now marks what it matched, in the title, the summary and the prose. A match lying only in the prose has nothing on the row to mark, so the chevron is marked instead. The search box gained a clear button, shown only while it holds something.
Always show the Eraser is itself two pictures of the toolbar rather than a switch: identical but for the Eraser, drawn in the accent color, and following the Layout setting — which is what decides whether the Eraser joins the bar or takes a row beneath it.
Decision 25 in
docs/decisions.mdrecords the parts a later change would otherwise undo without knowing.VersionPrefixgoes to 1.2.2, and the settings schema to 15.Verified beyond the build:
dotnet build -c Releaseis clean withTreatWarningsAsErrors, the Core smoke tests pass, and the toolbar and dialog were checked by launching the Release build and capturing both toolbar layouts, the collapsed and searched dialog, and both states of the Eraser choice. The Core smoke tests cover the new setting's default, round trip, and upgrade from a file written before it existed; the UI itself is outside their reach by design, soREADME.mdcarries the manual step.🤖 Generated with Claude Code