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

Delete Button Bug in iOS Safari (inside Tags, Messages and Notifications) #963

Closed
saschalichtenstein opened this issue Jul 30, 2017 · 5 comments
Labels

Comments

@saschalichtenstein
Copy link

img_0156

Overview of the problem

Delete button with messages on ios safari. Happens only inside tag, messages and notifications.

Description

Steps to Reproduce

Expected behavior

Actual behavior

@ialhashim
Copy link

I found a hack by setting padding to zero

@alcovegan
Copy link

Looks like same as #514 and proposed solution in #549, but not merged.

@johan13
Copy link

johan13 commented Aug 16, 2017

I think it's also the same as #490. Both #514 and #490 have been closed without fixing the problem. I hope this issue doesn't face the same fate.

@martdavidson
Copy link

martdavidson commented Oct 22, 2017

It needs font-size: 0.

It's because the min-width of a button in ios is dependant on the font size, so it's wider than it is tall.

https://stackoverflow.com/questions/3777494/is-it-possible-to-override-the-minimum-width-of-a-button-in-safari-on-ipad

jgthms pushed a commit that referenced this issue Oct 22, 2017
iOS <button> element's minimum width is dependant on its font-size, so the .delete width wasn't able to be as small as it's height, resulting in one arm of the X being longer than the other. font-size: 0 does the trick.
@stale
Copy link

stale bot commented Jan 21, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants