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

Move properties before nested selector to resolve deprecation warnings #807

Merged
merged 1 commit into from
Nov 20, 2024

Conversation

codders
Copy link
Contributor

@codders codders commented Nov 20, 2024

SASS has been emitting a deprecation warning because of a coming change in SASS behaviour:

https://sass-lang.com/documentation/breaking-changes/mixed-decls/

In NewLexemeForm.vue we were using the & parent selector before applying additional properties to the .wbl-snl-form element.

In this specific case, moving the parent selector after the property declarations has no effect on the semantics of the SASS or the generated CSS, and resolves the deprecation warning.

Bug: T372250

SASS has been emitting a deprecation warning because of a coming
change in SASS behaviour:

https://sass-lang.com/documentation/breaking-changes/mixed-decls/

In `NewLexemeForm.vue` we were using the `&` parent selector
before applying additional properties to the `.wbl-snl-form` element.

In this specific case, moving the parent selector after the
property declarations has no effect on the semantics of the SASS or
the generated CSS, and resolves the deprecation warning.

Bug: T372250
@codders codders requested a review from yerdua November 20, 2024 11:58
@yerdua yerdua merged commit 2d2baee into main Nov 20, 2024
3 checks passed
@yerdua yerdua deleted the feat/resolve-sass-deprecation-warnings-20241120 branch November 20, 2024 13:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants