Skip to content

Update accordion to use Login Design System#3974

Merged
aduth merged 1 commit intomasterfrom
aduth-lg-accordion
Jul 30, 2020
Merged

Update accordion to use Login Design System#3974
aduth merged 1 commit intomasterfrom
aduth-lg-accordion

Conversation

@aduth
Copy link
Contributor

@aduth aduth commented Jul 24, 2020

Related: #3957

Removes custom accordion implementation, instead taking advantage of existing Login Design System component availability.

Why: To take advantage of accessibility improvements offered by leveraging the upstream USWDS.

Why: To remove duplicate code, and to avoid the overhead involved in maintaining duplicate code.

Why: To use Login Design System standardized HTML markup.

Why: In the effort to migrate Slim views to ERB.

Screenshot:

accordion

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.

so much red! LGTM

**Why**: To take advantage of accessibility improvements offered by leveraging the upstream USWDS.

**Why**: To remove duplicate code, and to avoid the overhead involved in maintaining duplicate code.

**Why**: To use Login Design System standardized HTML markup.

**Why**: In the effort to migrate Slim views to ERB.
@aduth aduth force-pushed the aduth-lg-accordion branch from b957121 to afc0d5c Compare July 30, 2020 13:02
@aduth
Copy link
Contributor Author

aduth commented Jul 30, 2020

Noting that this will indirectly resolve LG-3194

@aduth aduth merged commit dd33d91 into master Jul 30, 2020
@aduth aduth deleted the aduth-lg-accordion branch July 30, 2020 14:49
@aduth aduth mentioned this pull request Oct 31, 2023
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