Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change direction of dropdown arrow when opened #2923

Merged
merged 2 commits into from Feb 25, 2019
Merged

Conversation

greensteph
Copy link

@greensteph greensteph commented Feb 25, 2019

Preview

Closes #2616

Added up arrow images assets.
Swapped out down arrow for up arrow when the menu is expanded.

@greensteph greensteph requested a review from a team February 25, 2019 15:24
@thisisdano
Copy link
Member

Hm — I'm not seeing this in action here: https://federalist-proxy.app.cloud.gov/preview/uswds/uswds/dw-grid-important/components/preview/header--default.html

Am I missing something?

@saracope
Copy link
Contributor

@thisisdano It looks like you may be on the wrong branch? You linked to dw-grid-important. But the change is on https://federalist-proxy.app.cloud.gov/preview/uswds/uswds/sg-header-arrow/components/preview/header--default.html

@thisisdano
Copy link
Member

Oooof. Poor tab management skills.

@thisisdano thisisdano merged commit 715b9b9 into release-2.0 Feb 25, 2019
@thisisdano thisisdano deleted the sg-header-arrow branch February 25, 2019 19:18
@thisisdano
Copy link
Member

Nice work — sorry for the confusion.

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.

None yet

3 participants