Skip to content

fix(ui-egui): font size list shows OneNote's whole sizes, without pt - #95

Closed
mameen wants to merge 1 commit into
mainfrom
fix/sup-009-font-size-list/agent-1
Closed

mameen wants to merge 1 commit into
mainfrom
fix/sup-009-font-size-list/agent-1

Conversation

@mameen

@mameen mameen commented Nov 11, 2026

Copy link
Copy Markdown
Collaborator

Summary

The font size list in the Basic Text group now matches OneNote's whole sizes: 8, 9, 10, 11, 12, 14, 16, 18, 20, 22, 24, 26, 28, 36, 48 and 72. The " pt" text is dropped from each entry.

Not in this PR: 9.5 and 10.5, which OneNote also offers. The page format stores whole sizes only, so supporting them needs a page-format change, which is a separate decision.

Tests

  • New unit test the_font_size_list_is_onenotes_whole_sizes in crates/ui-egui/src/ribbon.rs, written first and seen to fail.
  • The UI tests that clicked a size by its old "N pt" label now click the label "N". The size box text is unchanged.

Checks

  • cargo xtask ci: all checks passed, on the branch rebased onto origin/main.

🤖 Generated with Claude Code

The size list now offers OneNote's whole sizes (8 to 72, 16 sizes) and
drops the " pt" text. 9.5 and 10.5 wait for a page-format change, since
the page stores whole sizes.

Co-Authored-By: Claude Haiku 5.5 <noreply@anthropic.com>
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.

2 participants