Skip to content

changelog: Improvements, pull in ad-hoc styling from idp, LG-5883#309

Merged
SammySteiner merged 3 commits intomainfrom
LG-5883-mfa-icon-styles
Mar 31, 2022
Merged

changelog: Improvements, pull in ad-hoc styling from idp, LG-5883#309
SammySteiner merged 3 commits intomainfrom
LG-5883-mfa-icon-styles

Conversation

@SammySteiner
Copy link
Contributor

No description provided.

CHANGELOG.md Outdated
- Reconcile redundant focus styles with U.S. Web Design System. ([#302](https://github.com/18F/identity-style-guide/pull/302))
- Remove documentation site styles from design system artifact. ([#305](https://github.com/18F/identity-style-guide/pull/305))
- Reduce size of SVG images. ([#307](https://github.com/18F/identity-style-guide/pull/307))
- Incorporate mfa icon styles, which were previously added to the identity-idp. ([#309](https://github.com/18F/identity-style-guide/pull/309))
Copy link
Contributor

Choose a reason for hiding this comment

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

This might fit better under an "Improvements" or "Bug Fixes" section, since it's not really a performance optimization. Could we describe what the change is. IIRC it's improving/increasing the spacing around the icon?

Copy link
Contributor

@aduth aduth left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@aduth
Copy link
Contributor

aduth commented Mar 30, 2022

Visual regression build failure seems to be expected, based on changes to the tile fields.

Via artifacts:

image

@SammySteiner SammySteiner merged commit f2c54e4 into main Mar 31, 2022
@SammySteiner SammySteiner deleted the LG-5883-mfa-icon-styles branch March 31, 2022 13:17
@aduth aduth mentioned this pull request Mar 31, 2022
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.

2 participants