Skip to content

Fix external link icon content offset#289

Merged
aduth merged 2 commits intomainfrom
aduth-external-link-offset
Feb 1, 2022
Merged

Fix external link icon content offset#289
aduth merged 2 commits intomainfrom
aduth-external-link-offset

Conversation

@aduth
Copy link
Contributor

@aduth aduth commented Feb 1, 2022

Why: So that existing intended styles are applied as expected.

In #240, we customized the external link icon margins to increase spacing relative to surrounding content. This regressed as part of the recent upgrade to USWDS v2.13.1, which modified how external link icons were styled in a manner incompatible with our previous revisions.

The changes included here restore the intended margin effect.

Screenshot:

These screenshots show links in context of the IdP IAL2 flow. In particular, note the "Learn more" link in the last paragraph of the content.

Before After
Screen Shot 2022-02-01 at 8 33 55 AM Screen Shot 2022-02-01 at 8 33 13 AM

aduth added 2 commits February 1, 2022 08:40
Sync to USWDS folder structure
**Why**: So that existing intended styles are applied as expected.
Copy link
Contributor

@nickttng nickttng left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@zachmargolis zachmargolis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@anniehirshman-gsa anniehirshman-gsa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM III - much improved!

@aduth aduth merged commit d8fb5b6 into main Feb 1, 2022
@aduth aduth deleted the aduth-external-link-offset branch February 1, 2022 19:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants