-
Notifications
You must be signed in to change notification settings - Fork 21
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
Closes #3567 Styling adjustments for the AZ Finder exposed form (AZDIGITAL-158) #3577
Conversation
@bberndt-uaz I'm loving the visual consistency here! Thank you! Here are some thoughts of mine.
|
Is this something we'd want to include in patch release(s) (since it's updating an experimental module)? |
@danahertzberg Thanks for your thoughts! I have updated the PR accordingly. I was still wary of increasing the indentation very much, but if you would like more I'd be happy to increase it further. @joeparsons I think a patch release makes sense, but I don't know if we would like to limit this change to 2.11 to go along with the Finder overrides PR. This PR increases the height of the exposed form, so maybe it would be best to include it only with the override functionality so that users can set terms be collapsed by default if they wish? |
Co-authored-by: Dana Hertzberg <[email protected]>
…GITAL-158) (#3577) Co-authored-by: Dana Hertzberg <[email protected]>
…GITAL-158) (#3577) Co-authored-by: Dana Hertzberg <[email protected]>
Description
The goal of this PR is to establish a more uniform presentation for the taxonomy term filters in the AZ Finder exposed form.
This PR proposes the following styling changes to the AZ Finder exposed form:
display: flex
as part of this change.line-height
of filter items has been reduced to improve the display of multi-line text.Related issues
#3567
Closes AZDIGITAL-158
How to test
Probo review site: https://7b5e2985-cedf-4a13-ab91-3a35916c835d--pr-3577.probo.build/
Types of changes
Arizona Quickstart (install profile, custom modules, custom theme)
Drupal core
Drupal contrib projects
Checklist