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

CSS Deprecation Issue for Team: kibana-presentation, Found in file: debug.scss #190895

Closed
Tracked by #190345
petrklapka opened this issue Aug 20, 2024 · 1 comment · Fixed by #194721
Closed
Tracked by #190345

CSS Deprecation Issue for Team: kibana-presentation, Found in file: debug.scss #190895

petrklapka opened this issue Aug 20, 2024 · 1 comment · Fixed by #194721
Assignees
Labels
impact:low Addressing this issue will have a low level of impact on the quality/strength of our product. loe:small Small Level of Effort Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas

Comments

@petrklapka
Copy link
Member

DEPRECATION: Sass's behavior for declarations that appear after nested
rules will be changing to match the behavior specified by CSS in an upcoming
version. To keep the existing behavior, move the declaration above the nested
rule. To opt into the new behavior, wrap the declaration in & {}.

The file below has one or more instances where CSS behavior will change unless
the declaration is moved above the nested rule. Line numbers for the declaration
along with the file and line numbers in which the nested rule appears are
shown below.

More info: https://sass-lang.com/d/mixed-decls

kibana/src/plugins/expression_error/public/components/debug/debug.scss

   ┌──> kibana/src/plugins/expression_error/public/components/debug/debug.scss
9  │       width: 100%;
   │       ^^^^^^^^^^^ declaration
   ╵
   ┌──> kibana/node_modules/@elastic/eui/src/global_styling/mixins/_helpers.scss
31 │ ┌   &::-webkit-scrollbar-corner,
32 │ │   &::-webkit-scrollbar-track {
33 │ │     background-color: $trackBackgroundColor;
34 │ │   }
   │ └─── nested rule
   ╵

   ┌──> kibana/src/plugins/expression_error/public/components/debug/debug.scss
10 │       height: 100%;
   │       ^^^^^^^^^^^^ declaration
   ╵
   ┌──> kibana/node_modules/@elastic/eui/src/global_styling/mixins/_helpers.scss
31 │ ┌   &::-webkit-scrollbar-corner,
32 │ │   &::-webkit-scrollbar-track {
33 │ │     background-color: $trackBackgroundColor;
34 │ │   }
   │ └─── nested rule
   ╵

   ┌──> kibana/src/plugins/expression_error/public/components/debug/debug.scss
11 │       overflow: auto;
   │       ^^^^^^^^^^^^^^ declaration
   ╵
   ┌──> kibana/node_modules/@elastic/eui/src/global_styling/mixins/_helpers.scss
31 │ ┌   &::-webkit-scrollbar-corner,
32 │ │   &::-webkit-scrollbar-track {
33 │ │     background-color: $trackBackgroundColor;
34 │ │   }
   │ └─── nested rule
   ╵

   ┌──> kibana/src/plugins/expression_error/public/components/debug/debug.scss
12 │       padding: $euiSize;
   │       ^^^^^^^^^^^^^^^^^ declaration
   ╵
   ┌──> kibana/node_modules/@elastic/eui/src/global_styling/mixins/_helpers.scss
31 │ ┌   &::-webkit-scrollbar-corner,
32 │ │   &::-webkit-scrollbar-track {
33 │ │     background-color: $trackBackgroundColor;
34 │ │   }
   │ └─── nested rule
   ╵
@botelastic botelastic bot added the needs-team Issues missing a team label label Aug 20, 2024
@petrklapka petrklapka added the Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas label Aug 20, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-presentation (Team:Presentation)

@botelastic botelastic bot removed the needs-team Issues missing a team label label Aug 20, 2024
@ThomThomson ThomThomson added loe:small Small Level of Effort impact:low Addressing this issue will have a low level of impact on the quality/strength of our product. labels Aug 21, 2024
@nreese nreese self-assigned this Oct 2, 2024
@nreese nreese closed this as completed in 6cf30e4 Oct 4, 2024
kibanamachine pushed a commit to kibanamachine/kibana that referenced this issue Oct 4, 2024
…lastic#194721)

closes elastic#190895,
elastic#190896, and
elastic#190897

## Summary

> Sass's behavior for declarations that appear after nested
rules will be changing to match the behavior specified by CSS in an
upcoming
version. To keep the existing behavior, move the declaration above the
nested
rule. To opt into the new behavior, wrap the declaration in & {}.

PR moves declarations above the nested rules.

Co-authored-by: Elastic Machine <[email protected]>
(cherry picked from commit 6cf30e4)
jbudz pushed a commit to jbudz/kibana that referenced this issue Oct 4, 2024
…lastic#194721)

closes elastic#190895,
elastic#190896, and
elastic#190897

## Summary

> Sass's behavior for declarations that appear after nested
rules will be changing to match the behavior specified by CSS in an
upcoming
version. To keep the existing behavior, move the declaration above the
nested
rule. To opt into the new behavior, wrap the declaration in & {}.

PR moves declarations above the nested rules.

Co-authored-by: Elastic Machine <[email protected]>
tiansivive pushed a commit to tiansivive/kibana that referenced this issue Oct 7, 2024
…lastic#194721)

closes elastic#190895,
elastic#190896, and
elastic#190897

## Summary

> Sass's behavior for declarations that appear after nested
rules will be changing to match the behavior specified by CSS in an
upcoming
version. To keep the existing behavior, move the declaration above the
nested
rule. To opt into the new behavior, wrap the declaration in & {}.

PR moves declarations above the nested rules.

Co-authored-by: Elastic Machine <[email protected]>
tiansivive pushed a commit to tiansivive/kibana that referenced this issue Oct 7, 2024
…lastic#194721)

closes elastic#190895,
elastic#190896, and
elastic#190897

## Summary

> Sass's behavior for declarations that appear after nested
rules will be changing to match the behavior specified by CSS in an
upcoming
version. To keep the existing behavior, move the declaration above the
nested
rule. To opt into the new behavior, wrap the declaration in & {}.

PR moves declarations above the nested rules.

Co-authored-by: Elastic Machine <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
impact:low Addressing this issue will have a low level of impact on the quality/strength of our product. loe:small Small Level of Effort Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants