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

Label and Content Text Sizes of input field and select(or) from select box differ #6823

Open
2 tasks done
TomFreudenberg opened this issue Mar 1, 2025 · 4 comments
Open
2 tasks done
Labels
bug Something isn't working

Comments

@TomFreudenberg
Copy link

TomFreudenberg commented Mar 1, 2025

Describe the bug

Hi, thanks for that great work first of all.

Please find attached a screen shot from Android Chrome viewing

https://v4.shadcn.com/forms (but also v3 has the same mistake)

The text size for "Select a color" differs from all other input fields.

Thats happen on all select box elements.

Image

I copied the last chars from the email field as image beneeth the select box

Image

If I run the same code on Desktop - all looks nice and from same sizes.

Wonder if this is a bug or can't be changed?

Affected component/components

Select

How to reproduce

Just open https://v4.shadcn.com/forms

Codesandbox/StackBlitz link

https://v4.shadcn.com/forms

Logs

System Info

Running Huawei P30 / Android EMUI 12 / 2340 x 1080 px Screen / Chrome Version 132

Before submitting

  • I've made research efforts and searched the documentation
  • I've searched for existing issues
@TomFreudenberg TomFreudenberg added the bug Something isn't working label Mar 1, 2025
@Jacksonmills
Copy link
Contributor

@shadcn after looking into this, it seems Textarea and Input handle text differently from other form components. Right now, they use text-base with md:text-sm, while Select and others default to text-sm.

Is this the intended behavior, or should we update Select to follow this responsive size pattern? Alternatively, should we align everything to just text-sm and let users handle responsiveness in their implementation?

Let me know what makes the most sense!

@shadcn
Copy link
Collaborator

shadcn commented Mar 3, 2025

The text-base prevents the auto zoom on mobile.

@Jacksonmills
Copy link
Contributor

Jacksonmills commented Mar 3, 2025

Gotcha! That makes sense. So two potential options:

  1. Keep things as they are.

  2. Align select/date/etc inputs with text fields by setting them to text-base on smaller breakpoints to maintain consistency.

@Jacksonmills
Copy link
Contributor

LMK what you think is best 🫡

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants