-
Notifications
You must be signed in to change notification settings - Fork 89
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
fix(css): Adjust heading sizes (Fixes: #5515) #5594
Conversation
/backport to stable29 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks much nicer, thanks @mejo- π
Let's wait for @jancborchardt to double check
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good for me!
Let's wait for designers :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM as well :)
@mejo- Mind to update the print screenshots used to compare in the CI? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice, much better :)
Did now, thanks for the pointer π |
π Summary
Fixes: #5515
Also aligns with nextcloud/server#43640
πΌοΈ Screenshots
π Checklist
npm run lint
/npm run stylelint
/composer run cs:check
)