Skip to content

Commit

Permalink
Adjust heading anchor styles - more subtle
Browse files Browse the repository at this point in the history
See #132 for details
  • Loading branch information
NicolaSaunders committed Oct 10, 2024
1 parent b779c13 commit f7ca7d6
Show file tree
Hide file tree
Showing 8 changed files with 88 additions and 109 deletions.
18 changes: 16 additions & 2 deletions assets-src/styles/sass/30-base/_links.scss
Original file line number Diff line number Diff line change
Expand Up @@ -18,8 +18,7 @@ a.with-icon--after {
border: 0;
color: $link-color;
cursor: pointer;
padding-left: rem(2);
padding-right: rem(2);
padding-inline: rem(2);
text-decoration: underline; /* 1 */
text-decoration-skip-ink: auto; // Not supported by Safari
text-underline-offset: 0.25em; // Supported by Safari
Expand Down Expand Up @@ -53,7 +52,22 @@ a.with-icon--after {
.heading-anchor {
@extend a, :not([class]);

color: $storm-gray;
font-weight: normal;
margin-inline-start: 0.25em;
opacity: 0.82;
text-underline-offset: auto;

&:visited {
color: $storm-gray;
}

&:hover {
opacity: 1;
}

&:focus {
color: $black;
opacity: 1;
}
}
1 change: 1 addition & 0 deletions assets-src/styles/sass/30-base/_print.scss
Original file line number Diff line number Diff line change
Expand Up @@ -723,6 +723,7 @@ footer {
.banner,
#lang-nav,
.logo-link .visuallyhidden,
.heading-anchor,
[data-trigger],
[data-toggle],
#global-nav ul,
Expand Down
24 changes: 8 additions & 16 deletions public/dist/assets/styles/advanced.css
Original file line number Diff line number Diff line change
Expand Up @@ -158,8 +158,7 @@ article {
[data-trigger=account-menu] .sr-only, .visuallyhidden:not(:focus):not(:active) {
border: 0;
clip: rect(0 0 0 0);
-webkit-clip-path: inset(100%);
clip-path: inset(100%);
clip-path: inset(100%);
height: 1px;
overflow: hidden;
padding: 0;
Expand Down Expand Up @@ -226,8 +225,7 @@ br {
}
@media screen and (min-width: 70em) {
[data-trigger=account-menu] {
-webkit-margin-start: auto;
margin-inline-start: auto;
margin-inline-start: auto;
padding: 0.5rem 0.1875rem 0.3125rem 0.1875rem;
}
[data-trigger=account-menu]:hover::before {
Expand All @@ -254,8 +252,7 @@ br {
[data-trigger=account-menu] .sr-only {
border: initial;
clip: auto;
-webkit-clip-path: none;
clip-path: none;
clip-path: none;
height: auto;
margin: initial;
overflow: initial;
Expand All @@ -267,8 +264,7 @@ br {
}
@media screen and (min-width: 70em) {
[data-trigger=account-menu] .avatar {
-webkit-margin-start: 0.5rem;
margin-inline-start: 0.5rem;
margin-inline-start: 0.5rem;
}
}

Expand Down Expand Up @@ -426,13 +422,11 @@ br {
}

.icon--submenu {
-webkit-margin-start: auto;
margin-inline-start: auto;
margin-inline-start: auto;
}
@media screen and (min-width: 70em) {
.icon--submenu {
-webkit-margin-start: 0.25rem;
margin-inline-start: 0.25rem;
margin-inline-start: 0.25rem;
transform: rotate(90deg);
}
}
Expand Down Expand Up @@ -577,8 +571,7 @@ br {
}
.global-nav__inner [data-button=mobile-back] .icon--submenu {
height: 1.875rem;
-webkit-margin-start: 0;
margin-inline-start: 0;
margin-inline-start: 0;
transform: rotate(180deg);
width: 1.875rem;
}
Expand Down Expand Up @@ -898,8 +891,7 @@ br {
}

#js-cancel-reply {
-webkit-margin-start: 0.625rem;
margin-inline-start: 0.625rem;
margin-inline-start: 0.625rem;
}

/*------------------------------------*\
Expand Down
2 changes: 1 addition & 1 deletion public/dist/assets/styles/advanced.min.css

Large diffs are not rendered by default.

Loading

0 comments on commit f7ca7d6

Please sign in to comment.