-
Notifications
You must be signed in to change notification settings - Fork 231
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
Don't wrap "Clear Filter" text #2439
Don't wrap "Clear Filter" text #2439
Conversation
something is wrong with the line height, those two lines of text are too close together |
This is using the normal |
We adjust the bootstrap header sizes in our variables, maybe we screwed up the line heights |
We have many instances of "Clear Filter" now (browse pages). We should probably change them all? |
mucking with header line heights is risky at this point this change can go in as is since it is low risk |
546644a
to
faf361a
Compare
Updated the others as well. |
/test |
@jwforres another look since I added the class to other instances? |
/lgtm |
Automatic merge from submit-queue. |
Before:
After:
@jwforres @rhamilto