Skip to content

feat: add code font size zoom control#141

Open
Sha17g wants to merge 1 commit into
gbdev:masterfrom
Sha17g:feature/font-size-control
Open

feat: add code font size zoom control#141
Sha17g wants to merge 1 commit into
gbdev:masterfrom
Sha17g:feature/font-size-control

Conversation

@Sha17g

@Sha17g Sha17g commented May 31, 2026

Copy link
Copy Markdown
Contributor

The font size is changeable by using keyboard shortcuts or the new buttons.
Apart from the newly added buttons and shortcut keys, there are no other changes.
The feature is implemented through ACE editor API.

@Sha17g Sha17g force-pushed the feature/font-size-control branch from 08592cd to 3e917f2 Compare May 31, 2026 15:10
Comment thread index.html
View
<ul>
<li>
<label>

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Why a <label>?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Probably just copy-pasted from the others. When a <label> wraps text and a single <input>, like in #143, then it allows clicking the text to affect the input. But here there are two buttons, so the "Font size:" label shouldn't actually be a <label> for either of them.

@Rangi42 Rangi42 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks; please just remove the <label> and fix the merge conflict, and then I think this will be ready.

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.

3 participants