-
Notifications
You must be signed in to change notification settings - Fork 87
chore: fix some incorrect formatting in Lumo CSS files #9954
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
Conversation
@@ -191,10 +191,9 @@ | |||
|
|||
/* Disable hover for touch devices */ | |||
@media (pointer: coarse) { | |||
/* prettier-ignore */ |
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.
Seems to be not needed, Prettier doesn't complain about next line length.
|
+ 57px | ||
); | ||
/* Width: date cell widths + month calendar side padding + year scroller width */ | ||
width: calc(var(--lumo-size-m) * 7 + var(--lumo-space-xs) * 2 + 57px); |
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.
IMO this makes more sense in a single line, both the code and the comment look readable enough.
@@ -95,19 +95,19 @@ | |||
|
|||
/* prettier-ignore */ |
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.
Is this still relevant?
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.
I think the CSS selector wraps into second line as too long, which is why it's there.
Description
Improved some incorrect formatting in CSS files.
Type of change