Skip to content

Commit

Permalink
fix: no depress UX for links
Browse files Browse the repository at this point in the history
also, remove now unnecessary bug fix
  • Loading branch information
wesleyboar committed Apr 25, 2022
1 parent 847d0e2 commit 53a3a6c
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 7 deletions.
1 change: 0 additions & 1 deletion source/_imports/elements/html-elements.css
Original file line number Diff line number Diff line change
Expand Up @@ -143,7 +143,6 @@ Styleguide Elements.InlineTextSemantics
a,
a:hover { color: var(--global-color-link-on-light--normal); /* overwrite Bootstrap */ }
a:focus { text-decoration: underline dotted; }
a:active { position: relative; top: 0.1em; /* to simulate "depress" of link */ }

/* FAQ: Bootstrap already provides basic text-decoration */
/*
Expand Down
6 changes: 0 additions & 6 deletions source/_imports/trumps/s-article-list.css
Original file line number Diff line number Diff line change
Expand Up @@ -187,12 +187,6 @@ Styleguide Trumps.Scopes.ArticleList
.s-article-list--layout-gapless.s-article-list--links p:not(:last-child) a:hover::before {
@extend .x-article-link-hover--gapless;
}
/* Disable link click feedback because `x-article-link-stretch` is used */
/* FP-1249: If <a> wraps list item, then no need to `x-article-link-stretch` */
/* SEE: https://github.com/TACC/Core-CMS/pull/429 */
.s-article-list--links p:not(:last-child) a:active {
position: static; /* override html-elements.css `a:active` */
}


/* Modifiers: Layout */
Expand Down

0 comments on commit 53a3a6c

Please sign in to comment.