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

Moved customer support to its own accordion #2214

Merged
merged 3 commits into from
Nov 19, 2018

Conversation

mmmavis
Copy link
Collaborator

@mmmavis mmmavis commented Nov 16, 2018

Closes #2137

@cadecairos cadecairos temporarily deployed to foundation-mofostaging-pr-2214 November 16, 2018 22:28 Inactive
@mmmavis mmmavis requested review from Pomax and kristinashu November 16, 2018 22:38
@kristinashu
Copy link

Yay! So much cleaner looking.

The accordion just looks a little low. Can you vertically center it between the two lines?

image

@mmmavis mmmavis temporarily deployed to foundation-mofostaging-pr-2214 November 16, 2018 23:35 Inactive
@mmmavis
Copy link
Collaborator Author

mmmavis commented Nov 16, 2018

PR updated!

Copy link

@kristinashu kristinashu left a comment

Choose a reason for hiding this comment

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

Nice!

@Pomax Pomax merged commit 0e318a3 into master Nov 19, 2018
@Pomax Pomax deleted the issue-2137-customer-support-accordion branch November 19, 2018 19:42
mmmavis added a commit that referenced this pull request Nov 20, 2018
* Related to #2137 - moved customer support to its own accordion
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