Closed marteki closed 6 years ago
Whew! The build passed in Travis now. Needed to shift one of the dependencies and point it at a different location. Thanks to @Scotchester for his help on troubleshooting that.
@Scotchester Hmm. Maybe, if it's merged with the State section. I feel odd about having such an extensive Style section without explicitly calling out the Interactions, since it's an interactive web element.
Alright, I'll add that piece as a TODO to the list on our internal design patterns tracking page, and merging this.
Hmmm, we are using the accordion on the OAH pages ( closing-disclosure / loan-estimate ) and I'm not in favor of removing them. The accordion allows you manage long content on mobile, without forcing the user to scroll or close other expandables.
@sebworks There's been a good bit of discussion about this already over the last several months, where designers and developers came to an understanding about the removal of the accordions. For more details and to catch up, see the summary at https://[GHE]/CFPB/hubcap/wiki/Design-pattern-additions-and-changes#expandable-accordions-removed.
The current plan was to adjust the expandable groups for the one known place where accordions are being used, but the Loan Estimate page was not previously called out as a place! Let's discuss it more internally.
(Somewhat related, and an issue you were previously tagged on for comment that might have gotten buried in your inbox: https://github.com/cfpb/capital-framework/issues/536)
Merging this, especially since the Travis work is needed for any other PRs to be merged successfully and #594 is waiting on it.
As shared in https://github.com/cfpb/capital-framework/issues/536 and https://github.com/cfpb/capital-framework/issues/665, we’ve removed the accordion option from our design patterns. This updates our documentation to match the implementation guidelines.
Removals
Review
Screenshots
Documentation before changes
Documentation after changes
Checklist