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

fixes overly rounded select #75

Closed
wants to merge 4 commits into from
Closed

fixes overly rounded select #75

wants to merge 4 commits into from

Conversation

lucoel
Copy link

@lucoel lucoel commented Dec 14, 2017

This will resolve #69 and is tested in chrome, safari (also mobile), ff and edge (virtual)

@kbrsh
Copy link
Owner

kbrsh commented Dec 31, 2017

Hey! Sorry for the late response—thanks for being patient. Looks good, and I've tried this solution before, but I'm worried about the weight it adds to Wing.

Do you think an extra kb is worth fixing a rounded select? Would love to know your thoughts on it and how this can be further optimized.

Thanks again.

@lucoel
Copy link
Author

lucoel commented Jan 2, 2018

Thanks for your honest reply. Actually I've missed to check the weight.
I definitely see your point and will try to optimize this solution!

@lucoel
Copy link
Author

lucoel commented Jan 4, 2018

With 991155f I managed to reduce the increase to 0.51 kb and do some stylistic adjustments.
Since Wing being designed to be lightweight it demands your decision whether it's worth a feather more. 😉

In any case, I learned something, thanks!

@lucoel lucoel closed this May 12, 2022
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.

Overly Rounded select
2 participants