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

Modal close button: multiplication "✕" instead of "x" #121

Closed
wants to merge 1 commit into from
Closed

Modal close button: multiplication "✕" instead of "x" #121

wants to merge 1 commit into from

Conversation

ressawyj
Copy link

This corrects a small visual glitch where the "x" is not centred inside the circular button. "✕" is slightly larger and easier to see, as well as being centred it so it looks a little more polished.

This corrects a small visual glitch where the "x" is not centred inside the circular button. "✕" is slightly larger and easier to see, as well as being centred it so it looks a little more polished.
Copy link
Contributor

@bcomnes bcomnes left a comment

Choose a reason for hiding this comment

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

I agree, this is probably better. Going to run this by design. Thank you!

@bcomnes
Copy link
Contributor

bcomnes commented Apr 24, 2018

@ressawyj Thank you for the contribution! We made a few alternate tweaks, but very much appreciate the PR and bringing this to our attention.

@bcomnes bcomnes closed this Apr 24, 2018
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.

3 participants