Skip to content
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

MM-57506: use old CSS syntax #568

Merged
merged 1 commit into from
Apr 5, 2024
Merged

Conversation

lieut-data
Copy link
Member

Summary

Firefox ESR doesn't support nested CSS selectors: switch back to the old-style version instead, and fix a handful of other syntax errors. Push it through a CSS beautifier as well.

Ticket Link

Fixes: https://mattermost.atlassian.net/browse/MM-57506

Firefox ESR doesn't support nested CSS selectors: switch back to the old-style version instead, and fix a handful of other syntax errors.

Fixes: https://mattermost.atlassian.net/browse/MM-57506
@lieut-data lieut-data added 2: Dev Review Requires review by a core committer 3: QA Review Requires review by a QA tester labels Mar 30, 2024
@lieut-data lieut-data added this to the v1.11 milestone Mar 30, 2024
@sbishel sbishel modified the milestones: v1.11, v1.12 Apr 3, 2024
@lieut-data lieut-data requested review from larkox and removed request for hmhealey April 4, 2024 20:21
@lieut-data lieut-data enabled auto-merge (squash) April 4, 2024 20:21
@lieut-data lieut-data added QA/Deferred Agreement with QA that these changes will be tested post-merge and removed 3: QA Review Requires review by a QA tester labels Apr 4, 2024
@lieut-data lieut-data merged commit 5d08199 into main Apr 5, 2024
7 checks passed
@lieut-data lieut-data deleted the mm-57506-use-old-css-format branch April 5, 2024 08:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2: Dev Review Requires review by a core committer QA/Deferred Agreement with QA that these changes will be tested post-merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants