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

Font of the Pop-up editor sheet should default to MySQL content font #316

Closed
gboudreau opened this issue Aug 23, 2020 · 1 comment · Fixed by #330
Closed

Font of the Pop-up editor sheet should default to MySQL content font #316

gboudreau opened this issue Aug 23, 2020 · 1 comment · Fixed by #330
Assignees
Labels
Feature Request New feature or request

Comments

@gboudreau
Copy link
Contributor

If one chooses a new font for MySQL content font:
image

That font should also be used by default in the pop-up sheet used to edit values.

Currently, those two fonts are different:
image
image

That's because the pop-up sheet uses either the default system font, or the default monospaced font (if you enabled Use monospaced fonts in the preferences).
Instead, if should use the user-selected content font (if defined), and then fallback to its current behaviour.

Of note: one can also change the font used in the pop-up sheet by right-clicking in it, choosing "Font > Show Fonts", and changing the font there:
image
Sequel-Ace will persist that change, so obviously, we should keep that behaviour: using the content font as default should only happen when noFieldEditorSheetFont has been selected.

@gms8994
Copy link

gms8994 commented Aug 31, 2020

This is similar (same, maybe) as sequelpro/sequelpro#3495. There may be useful information there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Request New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants