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

Fix button alignment on older safari #1227

Merged
merged 2 commits into from
Apr 12, 2018

Conversation

maciejmatu
Copy link
Contributor

@maciejmatu maciejmatu commented Apr 3, 2018

- Summary
Fix issue with misaligned button icons on older browsers. - closes #914

This issue was connected to a flex-bug, that disallowed adding display: flex to certain HTML elements such as <button>
https://bugs.webkit.org/show_bug.cgi?id=169700

- Test plan
Testing this fix requires an older version of safari, or access to browserstack 😞 (I tested it, but it would be good, for somebody else to also take a look).

- Description for the changelog
Fix issue with misaligned button icons on older browsers.

- A picture of a cute animal (not mandatory but encouraged)
download
Blob fish is the best

@verythorough
Copy link
Contributor

verythorough commented Apr 3, 2018

Deploy preview for netlify-cms-www ready!

Built with commit 903fe13

https://deploy-preview-1227--netlify-cms-www.netlify.com

@verythorough
Copy link
Contributor

verythorough commented Apr 3, 2018

Deploy preview for cms-demo ready!

Built with commit 903fe13

https://deploy-preview-1227--cms-demo.netlify.com

@erquhart
Copy link
Contributor

Thanks @maciejmatu! Working on getting a Browserstack account for Netlify CMS testing, should have it soon, at which point we can test this PR out.

Copy link
Contributor

@erquhart erquhart left a comment

Choose a reason for hiding this comment

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

Pending browserstack acount/testing.

Copy link
Contributor

@erquhart erquhart left a comment

Choose a reason for hiding this comment

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

Tested, fixes confirmed in Safari 10, no regressions in latest Chrome.

Thanks @maciejmatu!

@erquhart erquhart force-pushed the fix-buttons-safari branch from e34c4d6 to 903fe13 Compare April 12, 2018 13:46
Copy link
Contributor

@tech4him1 tech4him1 left a comment

Choose a reason for hiding this comment

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

Fixed in Safari 10.1 (BrowserStack), no regressions in Chrome.

@tech4him1 tech4him1 merged commit df2d5c2 into decaporg:master Apr 12, 2018
@maciejmatu
Copy link
Contributor Author

maciejmatu commented Apr 12, 2018

Awesome, I'm really happy it works 😀

erquhart added a commit that referenced this pull request Apr 20, 2018
erquhart added a commit that referenced this pull request Apr 20, 2018
brianlmacdonald pushed a commit to brianlmacdonald/netlify-cms that referenced this pull request May 23, 2018
* Fix button alignment on older safari

* Add @maciejmatu to contributors
brianlmacdonald pushed a commit to brianlmacdonald/netlify-cms that referenced this pull request May 23, 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.

UI missalignemnts in Safari
4 participants