-
-
Notifications
You must be signed in to change notification settings - Fork 79.1k
select.form-control and input[type=text].form-control are different heights #17194
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
Comments
Selects are a pain. Safari renders them with an undesirable background gradient, and Firefox attaches that awful Win98-esque dropdown button. But if you set Here's an extension for
Got a little frustrated by the text abruptly cutting off at the padding line, so I also added the |
@arthurbath What you're proposing is something more appropriate for the completely custom form controls (http://v4-alpha.getbootstrap.com/components/forms/#select-menu ), not our vanilla |
CC: @mdo |
- Bootstrap 4 is currently in beta so things can, and probably will still change. Bootstrap 4 packages for DataTables won't be made available until at least a beta release of Bootstrap 4. - A few things have changed in Bootstrap 4 and rather than trying to make the exisiting Bootstrap files work for both BS3 and BS4 it makes sense to offer both. We'd need to detect which version of BS is loaded and that can be quite unreliable (and delayed since the body would need to be available). - Pagingation requires a couple of classes, but no change in structure - BS4 don't include icons any more, so need to use UTF8 arrows to show sorting (don't particularly want other dependencies, although it is easy to add with CSS if you want more icon control) - Outstanding issue: The select element for paging control is not of a consistent height with the search input. This appears to be a Bootstrap issue: twbs/bootstrap#17194
- Bootstrap 4 is currently in beta so things can, and probably will still change. Bootstrap 4 packages for DataTables won't be made available until at least a beta release of Bootstrap 4. - A few things have changed in Bootstrap 4 and rather than trying to make the exisiting Bootstrap files work for both BS3 and BS4 it makes sense to offer both. We'd need to detect which version of BS is loaded and that can be quite unreliable (and delayed since the body would need to be available). - Pagingation requires a couple of classes, but no change in structure - BS4 don't include icons any more, so need to use UTF8 arrows to show sorting (don't particularly want other dependencies, although it is easy to add with CSS if you want more icon control) - Outstanding issue: The select element for paging control is not of a consistent height with the search input. This appears to be a Bootstrap issue: twbs/bootstrap#17194 Sync to source repo @105ba6c24a7959ca63d1c9ccfbb32010babf52d8
- Bootstrap 4 is currently in beta so things can, and probably will still change. Bootstrap 4 packages for DataTables won't be made available until at least a beta release of Bootstrap 4. - A few things have changed in Bootstrap 4 and rather than trying to make the exisiting Bootstrap files work for both BS3 and BS4 it makes sense to offer both. We'd need to detect which version of BS is loaded and that can be quite unreliable (and delayed since the body would need to be available). - Pagingation requires a couple of classes, but no change in structure - BS4 don't include icons any more, so need to use UTF8 arrows to show sorting (don't particularly want other dependencies, although it is easy to add with CSS if you want more icon control) - Outstanding issue: The select element for paging control is not of a consistent height with the search input. This appears to be a Bootstrap issue: twbs/bootstrap#17194 Sync to source repo @105ba6c24a7959ca63d1c9ccfbb32010babf52d8
- Bootstrap 4 is currently in beta so things can, and probably will still change. Bootstrap 4 packages for DataTables won't be made available until at least a beta release of Bootstrap 4. - A few things have changed in Bootstrap 4 and rather than trying to make the exisiting Bootstrap files work for both BS3 and BS4 it makes sense to offer both. We'd need to detect which version of BS is loaded and that can be quite unreliable (and delayed since the body would need to be available). - Pagingation requires a couple of classes, but no change in structure - BS4 don't include icons any more, so need to use UTF8 arrows to show sorting (don't particularly want other dependencies, although it is easy to add with CSS if you want more icon control) - Outstanding issue: The select element for paging control is not of a consistent height with the search input. This appears to be a Bootstrap issue: twbs/bootstrap#17194 Sync to source repo @105ba6c24a7959ca63d1c9ccfbb32010babf52d8
- Bootstrap 4 is currently in beta so things can, and probably will still change. Bootstrap 4 packages for DataTables won't be made available until at least a beta release of Bootstrap 4. - A few things have changed in Bootstrap 4 and rather than trying to make the exisiting Bootstrap files work for both BS3 and BS4 it makes sense to offer both. We'd need to detect which version of BS is loaded and that can be quite unreliable (and delayed since the body would need to be available). - Pagingation requires a couple of classes, but no change in structure - BS4 don't include icons any more, so need to use UTF8 arrows to show sorting (don't particularly want other dependencies, although it is easy to add with CSS if you want more icon control) - Outstanding issue: The select element for paging control is not of a consistent height with the search input. This appears to be a Bootstrap issue: twbs/bootstrap#17194 Sync to source repo @105ba6c24a7959ca63d1c9ccfbb32010babf52d8
- Bootstrap 4 is currently in beta so things can, and probably will still change. Bootstrap 4 packages for DataTables won't be made available until at least a beta release of Bootstrap 4. - A few things have changed in Bootstrap 4 and rather than trying to make the exisiting Bootstrap files work for both BS3 and BS4 it makes sense to offer both. We'd need to detect which version of BS is loaded and that can be quite unreliable (and delayed since the body would need to be available). - Pagingation requires a couple of classes, but no change in structure - BS4 don't include icons any more, so need to use UTF8 arrows to show sorting (don't particularly want other dependencies, although it is easy to add with CSS if you want more icon control) - Outstanding issue: The select element for paging control is not of a consistent height with the search input. This appears to be a Bootstrap issue: twbs/bootstrap#17194 Sync to source repo @105ba6c24a7959ca63d1c9ccfbb32010babf52d8
- Bootstrap 4 is currently in beta so things can, and probably will still change. Bootstrap 4 packages for DataTables won't be made available until at least a beta release of Bootstrap 4. - A few things have changed in Bootstrap 4 and rather than trying to make the exisiting Bootstrap files work for both BS3 and BS4 it makes sense to offer both. We'd need to detect which version of BS is loaded and that can be quite unreliable (and delayed since the body would need to be available). - Pagingation requires a couple of classes, but no change in structure - BS4 don't include icons any more, so need to use UTF8 arrows to show sorting (don't particularly want other dependencies, although it is easy to add with CSS if you want more icon control) - Outstanding issue: The select element for paging control is not of a consistent height with the search input. This appears to be a Bootstrap issue: twbs/bootstrap#17194
Fixed in caa36b8. |
@mdo Still not quite right (OS X Chrome): |
Is this a similar issue to #18786 ? |
@eliottrobson Only slightly. |
@cvrebert when I was looking at that issue I did notice that my changes did alter the height of the select element. But the way I did it isn't ideal, hard-coded heights etc... Is there any way we can apply the same logic but calculate the values? |
I might miss something, but does the addition introduced by |
@torbjoernk Quite possibly. See #19771. |
I found that the issue for me was the top & bottom border widths being added to the calculated height of text form controls, but not being part of the $input-height: (($font-size-base * $input-line-height) + ($input-padding-y * 2)) + ($input-btn-border-width * 2); Note: in order to do this, I had to set I also used |
No description provided.
The text was updated successfully, but these errors were encountered: