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

.c-select-sm and .c-select-lg missing #20316

Closed
niftylettuce opened this issue Jul 17, 2016 · 2 comments
Closed

.c-select-sm and .c-select-lg missing #20316

niftylettuce opened this issue Jul 17, 2016 · 2 comments

Comments

@niftylettuce
Copy link

If you have .c-select form-control form-control-lg it does not use proper padding and so it is not a lg, it's tiny.

@niftylettuce
Copy link
Author

Temporary fix:

select.c-select.form-control-lg {
  padding: $input-padding-y-lg $input-padding-x-lg;
}

@cvrebert cvrebert changed the title [V4] .c-select-md .c-select-lg missing .c-select-sm and .c-select-lg missing Jul 17, 2016
@mdo mdo modified the milestone: v4.0.0-alpha.4 Jul 28, 2016
@judetucker
Copy link

It looked to me like there were two options for this one: adding a .custom-select-lg or making a special nested .form-control-sm that used to proper line height for the custom-selects.

Before
https://codepen.io/judetucker/pen/rLkKAL

After
https://codepen.io/judetucker/pen/VjEkRd

I went with adding a .custom-select-lg (via PR #20464) because it seemed cleaner than a nested .custom-select.form-control-sm

@mdo mdo modified the milestones: v4.0.0-alpha.5, v4.1 ideas Sep 12, 2016
@mdo mdo mentioned this issue Nov 24, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants