Skip to content

settings: use a bezeled multi-line text field for notes on macOS - #7906

Merged
osy merged 1 commit into
mainfrom
settings/notes-field
Sep 23, 2026
Merged

osy merged 1 commit into
mainfrom
settings/notes-field

Conversation

@osy

@osy osy commented Sep 23, 2026

Copy link
Copy Markdown
Contributor

The macOS notes box was a TextEditor with a hand-drawn half-point border that did not match the bezel of the Name field above it, had no focus ring, and bled square corners under the drawn outline in dark mode. A vertical-axis TextField looks right but submits on Return, and a drawn overlay cannot track the bezel style of each macOS release (12/13 differ from 26+).

This wraps an NSTextField configured for wrapping so AppKit draws the same bezel, highlight and focus ring as the Name field on every supported macOS version. Return inserts a newline through the field editor, the field grows with its contents from a 200pt minimum, and the page's disableAutocorrection is applied to the field editor as the TextEditor did. iOS keeps the TextEditor.

Testing: Tested by a human on macOS 27.0 and macOS 12.6 VM, Mac. The author acknowledges that this change has been tested and/or reviewed by a human in accordance with UTM's AI contribution guidelines.

The macOS notes box was a TextEditor with a hand-drawn half-point
border, which did not match the bezel of the name field above it and
had no focus ring. A TextField with a vertical axis has the right look
but submits on Return instead of inserting a newline, and a drawn
overlay cannot follow the bezel style of each macOS release.

Wrap an NSTextField configured for wrapping instead, so AppKit draws
the same bezel, highlight and focus ring as the name field on every
supported macOS version. Return is routed through the field editor
to insert a newline, the field grows with its contents from a 200pt
minimum, and the page's disableAutocorrection setting is applied to
the field editor as the TextEditor did. iOS keeps the TextEditor.

Assisted-by: Claude:claude-fable-5-1
@osy osy added this to the v5.0 milestone Sep 23, 2026
@osy
osy merged commit 5a01ce6 into main Sep 23, 2026
54 checks passed
@osy
osy deleted the settings/notes-field branch September 23, 2026 14:33
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.

1 participant