Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

gh-128062: Fix the font size and shortcut display of the turtledemo menu #128063

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

Xiaokang2022
Copy link
Contributor

@Xiaokang2022 Xiaokang2022 commented Dec 18, 2024

Leave the font of the menu bar the default to keep it consistent with the rest of the world. Display the shortcut keys in the right way, using the 'accelerator' option.

…s too large and the shortcut key is displayed inaccurately
Copy link
Member

@ZeroIntensity ZeroIntensity left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Could we be more clear (in both the PR title and the blurb entry) about what this is changing? ("improve" is too vague of a term)

@Xiaokang2022 Xiaokang2022 changed the title gh-128062: Improve the menu bar of turtledemo gh-128062: Revert the font of turtledemo's menu bar to its default value and display the shortcut keys in the correct position Dec 18, 2024
@terryjreedy terryjreedy changed the title gh-128062: Revert the font of turtledemo's menu bar to its default value and display the shortcut keys in the correct position gh-128062: Fix the font size and shortcut display of the turtledemo menu Dec 18, 2024
@terryjreedy terryjreedy self-assigned this Dec 18, 2024
@terryjreedy
Copy link
Member

I fixed title and commit message. The macOS failure seems to be a known unrelated issue, #128069. Merging seems to be blocked until this is fixed.

Copy link
Member

@terryjreedy terryjreedy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On my MacBook Air, with a small screen, fontsize 12 is too small as the default, used for the top bar, appears to be 14. Command+- and Command+= are the keys that work in the text widget. Agree with leaving size alone and using the standard accelerator option.

@terryjreedy terryjreedy added needs backport to 3.12 bug and security fixes needs backport to 3.13 bugs and security fixes labels Dec 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting merge needs backport to 3.12 bug and security fixes needs backport to 3.13 bugs and security fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants