Skip to content

Commit

Permalink
[8.x] Move nested declarations above rule (#193931) (#194598)
Browse files Browse the repository at this point in the history
# Backport

This will backport the following commits from `main` to `8.x`:
- [Move nested declarations above rule
(#193931)](#193931)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT
[{"author":{"name":"Kurt","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-10-01T14:43:52Z","message":"Move
nested declarations above rule (#193931)\n\n## Summary\r\n\r\n> Sass's
behavior for declarations that appear after nested\r\nrules will be
changing to match the behavior specified by CSS in
an\r\nupcoming\r\nversion. To keep the existing behavior, move the
declaration above the\r\nnested\r\nrule. To opt into the new behavior,
wrap the declaration in & {}.\r\n\r\nI have moved the declarations above
the nested rules.\r\n\r\nI had attempted to opt in to the new behavior,
but the resulting CSS has\r\nduplicate selectors, which CI does not
allow\r\n\r\nFor more information about the new behavior,
see\r\n[here](https://sass-lang.com/documentation/breaking-changes/mixed-decls/)\r\n\r\nCloses
https://github.com/elastic/kibana/issues/190898\r\n[screenshots](https://github.com/elastic/kibana/pull/193931#issuecomment-2375127180)\r\nCloses
https://github.com/elastic/kibana/issues/190899\r\n[screenshots](https://github.com/elastic/kibana/pull/193931#issuecomment-2375128228)\r\nCloses
#190900 Access Page
uses\r\n`authentication_state_page`, but I was unable to
find\r\n'overwrite_session` and `logged_out` pages\r\nCloses
https://github.com/elastic/kibana/issues/190901\r\n[screenshots](https://github.com/elastic/kibana/pull/193931#issuecomment-2375126897)\r\nCloses
#190902 ^used on same
UI\r\nCloses
https://github.com/elastic/kibana/issues/190903\r\n[screenshots](https://github.com/elastic/kibana/pull/193931#issuecomment-2375127629)\r\nCloses
https://github.com/elastic/kibana/issues/190904\r\n[screenshots](https://github.com/elastic/kibana/pull/193931#issuecomment-2375127865)","sha":"c8c74399a0b282b9b687379bd324db24fea59c44","branchLabelMapping":{"^v9.0.0$":"main","^v8.16.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Team:Security","release_note:skip","v9.0.0","v8.16.0","backport:version"],"title":"Move
nested declarations above
rule","number":193931,"url":"https://github.com/elastic/kibana/pull/193931","mergeCommit":{"message":"Move
nested declarations above rule (#193931)\n\n## Summary\r\n\r\n> Sass's
behavior for declarations that appear after nested\r\nrules will be
changing to match the behavior specified by CSS in
an\r\nupcoming\r\nversion. To keep the existing behavior, move the
declaration above the\r\nnested\r\nrule. To opt into the new behavior,
wrap the declaration in & {}.\r\n\r\nI have moved the declarations above
the nested rules.\r\n\r\nI had attempted to opt in to the new behavior,
but the resulting CSS has\r\nduplicate selectors, which CI does not
allow\r\n\r\nFor more information about the new behavior,
see\r\n[here](https://sass-lang.com/documentation/breaking-changes/mixed-decls/)\r\n\r\nCloses
https://github.com/elastic/kibana/issues/190898\r\n[screenshots](https://github.com/elastic/kibana/pull/193931#issuecomment-2375127180)\r\nCloses
https://github.com/elastic/kibana/issues/190899\r\n[screenshots](https://github.com/elastic/kibana/pull/193931#issuecomment-2375128228)\r\nCloses
#190900 Access Page
uses\r\n`authentication_state_page`, but I was unable to
find\r\n'overwrite_session` and `logged_out` pages\r\nCloses
https://github.com/elastic/kibana/issues/190901\r\n[screenshots](https://github.com/elastic/kibana/pull/193931#issuecomment-2375126897)\r\nCloses
#190902 ^used on same
UI\r\nCloses
https://github.com/elastic/kibana/issues/190903\r\n[screenshots](https://github.com/elastic/kibana/pull/193931#issuecomment-2375127629)\r\nCloses
https://github.com/elastic/kibana/issues/190904\r\n[screenshots](https://github.com/elastic/kibana/pull/193931#issuecomment-2375127865)","sha":"c8c74399a0b282b9b687379bd324db24fea59c44"}},"sourceBranch":"main","suggestedTargetBranches":["8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/193931","number":193931,"mergeCommit":{"message":"Move
nested declarations above rule (#193931)\n\n## Summary\r\n\r\n> Sass's
behavior for declarations that appear after nested\r\nrules will be
changing to match the behavior specified by CSS in
an\r\nupcoming\r\nversion. To keep the existing behavior, move the
declaration above the\r\nnested\r\nrule. To opt into the new behavior,
wrap the declaration in & {}.\r\n\r\nI have moved the declarations above
the nested rules.\r\n\r\nI had attempted to opt in to the new behavior,
but the resulting CSS has\r\nduplicate selectors, which CI does not
allow\r\n\r\nFor more information about the new behavior,
see\r\n[here](https://sass-lang.com/documentation/breaking-changes/mixed-decls/)\r\n\r\nCloses
https://github.com/elastic/kibana/issues/190898\r\n[screenshots](https://github.com/elastic/kibana/pull/193931#issuecomment-2375127180)\r\nCloses
https://github.com/elastic/kibana/issues/190899\r\n[screenshots](https://github.com/elastic/kibana/pull/193931#issuecomment-2375128228)\r\nCloses
#190900 Access Page
uses\r\n`authentication_state_page`, but I was unable to
find\r\n'overwrite_session` and `logged_out` pages\r\nCloses
https://github.com/elastic/kibana/issues/190901\r\n[screenshots](https://github.com/elastic/kibana/pull/193931#issuecomment-2375126897)\r\nCloses
#190902 ^used on same
UI\r\nCloses
https://github.com/elastic/kibana/issues/190903\r\n[screenshots](https://github.com/elastic/kibana/pull/193931#issuecomment-2375127629)\r\nCloses
https://github.com/elastic/kibana/issues/190904\r\n[screenshots](https://github.com/elastic/kibana/pull/193931#issuecomment-2375127865)","sha":"c8c74399a0b282b9b687379bd324db24fea59c44"}},{"branch":"8.x","label":"v8.16.0","branchLabelMappingKey":"^v8.16.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Kurt <[email protected]>
  • Loading branch information
kibanamachine and kc13greiner authored Oct 1, 2024
1 parent bee5cc2 commit 9267266
Show file tree
Hide file tree
Showing 9 changed files with 21 additions and 13 deletions.
4 changes: 2 additions & 2 deletions src/plugins/interactive_setup/public/app.scss
Original file line number Diff line number Diff line change
Expand Up @@ -9,10 +9,10 @@
}

.interactiveSetup__logo {
margin-bottom: $euiSizeXL;

@include kibanaCircleLogo;
@include euiBottomShadowMedium;

margin-bottom: $euiSizeXL;
}

.interactiveSetup__content {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,9 +7,10 @@
}

.secAccessAgreementPage__text {
@include euiYScrollWithShadows;
max-height: 400px;
padding: $euiSize $euiSizeL 0;

@include euiYScrollWithShadows;
}

.secAccessAgreementPage__footer {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,10 +9,10 @@
}

.secAuthenticationStatePage__logo {
margin-bottom: $euiSizeXL;

@include kibanaCircleLogo;
@include euiBottomShadowMedium;

margin-bottom: $euiSizeXL;
}

.secAuthenticationStatePage__content {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -23,11 +23,11 @@
}

&:focus {
@include euiFocusRing;

border-color: transparent;
border-radius: $euiBorderRadius;

@include euiFocusRing;

.secLoginCard__title {
text-decoration: underline;
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,10 +9,10 @@
}

.loginWelcome__logo {
margin-bottom: $euiSizeXL;

@include kibanaCircleLogo;
@include euiBottomShadowMedium;

margin-bottom: $euiSizeXL;
}

.loginWelcome__content {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,9 @@
}

.spcMenu__spacesList {
@include euiYScrollWithShadows;
max-height: $euiSizeXL * 10;

@include euiYScrollWithShadows;
}

.spcMenu__searchFieldWrapper {
Expand Down

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,9 @@

.spcSelectorBackground {
@include kibanaFullScreenGraphics;
}

.spcSelectorBackground__nonMixinAttributes {
z-index: -1;
pointer-events: none;
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -124,7 +124,10 @@ export class SpaceSelector extends Component<Props, State> {
>
{/* Portal the fixed background graphic so it doesn't affect page positioning or overlap on top of global banners */}
<EuiPortal>
<div className="spcSelectorBackground" role="presentation" />
<div
className="spcSelectorBackground spcSelectorBackground__nonMixinAttributes"
role="presentation"
/>
</EuiPortal>

<KibanaPageTemplate.Section color="transparent" paddingSize="xl">
Expand Down

0 comments on commit 9267266

Please sign in to comment.