Skip to content

Commit

Permalink
Merge pull request #3161 from alphagov/fix-sass-deprecations
Browse files Browse the repository at this point in the history
Fix Sass mixed declarations deprecations
  • Loading branch information
yndajas authored Sep 11, 2024
2 parents 0963d45 + b054731 commit 1533f78
Show file tree
Hide file tree
Showing 4 changed files with 11 additions and 12 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,6 @@
}

.user-research-recruitment-banner__buttons {
@include govuk-responsive-padding(6, "bottom");
align-items: center;
@include govuk-responsive-padding(6, "bottom");
}
10 changes: 5 additions & 5 deletions app/assets/stylesheets/components/_accessible-autocomplete.scss
Original file line number Diff line number Diff line change
@@ -1,15 +1,15 @@
@import "accessible-autocomplete/dist/accessible-autocomplete.min";

.autocomplete__wrapper * {
// The styles for the dropdown arrow set the z-index to -1, which place it
// behind the autocomplete. Setting the `z-index` on the wrapper seems to fix
// this. See: https://github.com/alphagov/accessible-autocomplete/issues/351#issuecomment-582935867
z-index: 0;
// This ensures the font family of all children is '"GDS Transport", arial,
// sans-serif'. Without this (and the universal selector above), the font of
// the input and options will be nonstandard. For me, Arial and Times
// respectively. See: https://github.com/alphagov/accessible-autocomplete/issues/285
@include govuk-typography-common;
// The styles for the dropdown arrow set the z-index to -1, which place it
// behind the autocomplete. Setting the `z-index` on the wrapper seems to fix
// this. See: https://github.com/alphagov/accessible-autocomplete/issues/351#issuecomment-582935867
z-index: 0;
}

.autocomplete__input--focused + .autocomplete__dropdown-arrow-down {
Expand All @@ -35,4 +35,4 @@
&--enabled {
display: inline-block;
}
}
}
9 changes: 4 additions & 5 deletions app/assets/stylesheets/components/_contact-details.scss
Original file line number Diff line number Diff line change
@@ -1,13 +1,12 @@
.app-c-contact-details {
padding-left: govuk-spacing(3);
clear: both;
border-left: 1px solid $govuk-border-colour;

@include govuk-font($size: 19);
@include govuk-text-colour;
padding-left: govuk-spacing(3);
// Margin top intended to collapse
// This adds an additional 10px to the paragraph above
@include govuk-responsive-margin(6, "top");
@include govuk-responsive-margin(6, "bottom");

clear: both;

border-left: 1px solid $govuk-border-colour;
}
2 changes: 1 addition & 1 deletion app/assets/stylesheets/components/_table.scss
Original file line number Diff line number Diff line change
Expand Up @@ -36,11 +36,11 @@ $table-row-even-background-colour: govuk-colour("light-grey");
}

.app-table__sort-link {
@include govuk-link-style-no-visited-state;
position: relative;
padding-right: $sort-link-arrow-size;
color: $govuk-link-colour;
text-decoration: none;
@include govuk-link-style-no-visited-state;
}

.app-table__sort-link:focus {
Expand Down

0 comments on commit 1533f78

Please sign in to comment.