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.

Sign up for GitHub

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

Jump to bottom

Limit maximum height of logo #763

Merged

nonprofittechy merged 1 commit into main from css-tweak

Sep 12, 2023

Member

nonprofittechy commented Sep 12, 2023

fix #748

This just mirrors the defaults in various production repos so more projects will work out of the box:

https://github.com/SuffolkLITLab/docassemble-ALWeaver/blob/fef677dd2daa9dd65d04b5621841f8e464910e86/docassemble/ALWeaver/data/static/styles.css#L25

https://github.com/SuffolkLITLab/docassemble-ALThemeTemplate/blob/2b0a0140a816c12f366a93feea1cdc7cdb95eb4e/docassemble/ALThemeTemplate/data/static/colors.css#L7


          fix #748

28c3478

nonprofittechy requested a review from BryceStevenWilley

September 12, 2023 13:54

BryceStevenWilley approved these changes

View reviewed changes

Contributor

BryceStevenWilley left a comment

LGTM, confirmed that it won't change ILAO's, and I agree with changing default for everyone else.

nonprofittechy merged commit 08b22f7 into main

5 checks passed

BryceStevenWilley deleted the css-tweak branch

September 25, 2023 13:50

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet