fix: mobile accessibility dropdown height (#1480)#1502
Conversation
|
Hi @slash-init, thank you for this PR. I tested it locally, and here is what I found. The height fix is correct and confirmed working. Adding Issue still remaining The font size mismatch mentioned in issue #1480 is still not addressed. After inspecting:
The dropdown text is hardcoded and set to 0.75rem (12px), which overrides inheritance and makes it smaller than the Search bar. The fix would be to either remove the hardcoded |
|
Hey @Nwakaego-Ego, Thanks for the review! |
|
@slash-init You're right, and that PR has been merged now. I will merge your change now as well. |
Summary
On mobile, the Accessibility dropdown was being placed into an implicit grid row, which defaulted to
autoheight. This caused the dropdown to appear compressed compared to the Language dropdown and Search bar.This change adds
grid-auto-rows: var(--settings-container-height-touch)so implicit grid rows use the same height as the explicitly defined rows, keeping all controls consistently sized on mobile.Before :

After:

Fixes #1480