Closed (fixed)
Project:
Admin Toolbar
Version:
8.x-2.3
Component:
User interface
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
22 Jun 2020 at 07:02 UTC
Updated:
2 Mar 2021 at 01:19 UTC
Jump to comment: Most recent, Most recent file

Comments
Comment #2
unarain commentedComment #3
romainj commentedCould you provide any screenshot please?
Comment #4
unarain commentedComment #5
adriancidI tested the module with the themes:
Claro
Seven
Bartik
And I can't see this problem.
Comment #6
unarain commentedComment #7
briangon commentedThis is worth revisiting, I have the same display error. With bootstrap theme on pages that aren't the admin pages, the .form-control class overrides the input box height. And the margin-top set in rem is inconsistent with the admin pages because of custom theme font sizes. Height should probably be included in the #admin-toolbar-search-input style and the sizes should be kept px for consistency.
Comment #8
jasonluttrellI agree with the above comment, #7. The search field appears squished to me when I switched themes, which tells me there is a theme conflict. That said, I do not know whether the suggested CSS changes would completely fix the problem either. For example, the #admin-toolbar-search-input padding attribute also uses rem.
For what it's worth, I am using adminimal on the back end and a proprietary theme based on classy on the frontend.
Comment #9
aaronchristian commentedComment #10
aaronchristian commentedI would agree, I've seen this on multiple installs when using a custom FE theme.
Rather than rework the padding with rem units (which will be dependant on current themes base font size), I think a simple fix is just to add a max-height on that particular input ID.
BEFORE:
AFTER:
Comment #11
aaronchristian commentedComment #12
chrissnyderThe patch provided in comment #10 worked for me.
Before patch:
After patch
I am using a bootstrap 4 based frontend theme.
Comment #14
romainj commentedCommited to latest dev version.
Comment #15
romainj commentedComment #17
briangon commentedI logged back in today to bump this but I'm delighted to see it was addressed again.
Switched to 2.x-dev or 3.x-dev and the fix is present, thanks.
As #8 pointed out, other values still use rem. And in my case the margin-top is slightly off still. Would be best if this were a fixed px value as the rem leaves room for inconsistencies like this to happen.