Skip to content

Conversation

web-padawan
Copy link
Member

Description

Improved some incorrect formatting in CSS files.

Type of change

  • Internal change

@web-padawan web-padawan requested a review from vursen August 8, 2025 12:33
@@ -191,10 +191,9 @@

/* Disable hover for touch devices */
@media (pointer: coarse) {
/* prettier-ignore */
Copy link
Member Author

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.

Copy link

sonarqubecloud bot commented Aug 8, 2025

+ 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);
Copy link
Member Author

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 */
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this still relevant?

Copy link
Member Author

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.

@web-padawan web-padawan merged commit 07c143b into main Aug 11, 2025
11 of 12 checks passed
@web-padawan web-padawan deleted the chore/lumo-css-prettier branch August 11, 2025 10:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants