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

Adjust combobox item height to the default font size #5615

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tronical
Copy link
Member

No description provided.

@tronical tronical requested a review from FloVanGH July 12, 2024 13:23
@tronical
Copy link
Member Author

There had to be a catch... Cannot convert between rem and logical length in a global component, because the default font size is not known

Ideally the rem would be added in the globa, but it's not possible to
combine px with rem in a global without access to the window.
@tronical
Copy link
Member Author

This version now is a fair bit uglier. If this is too ugly, then I don't mind leaving this PR behind and we aim for a cleaner solution another time.

@tronical tronical requested a review from FloVanGH July 12, 2024 13:34
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.

None yet

2 participants