Skip to content

fix: fix css flex/height issues in safari #248

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

Merged
merged 2 commits into from
Jul 6, 2020

Conversation

bmeverett
Copy link
Contributor

What:
Fix #238

Why:
To improve how the site looks and behaves in Safari to make it align with Chrome.

How:
Update css classes to work on Safari. I'm not the best at css, but I removed/added some classes to make Safari show more like Chrome. I checked Chrome while making these changes as well and it didn't seem to be affected. Happy to look at a different/better way of doing things, or if there was something I missed

Safari
Screen Shot 2020-07-06 at 8 44 36 AM

Chrome
Screen Shot 2020-07-06 at 8 44 40 AM

Checklist:

  • Tests n/a
  • Ready to be merged

Copy link
Member

@smeijer smeijer left a comment

Choose a reason for hiding this comment

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

Thanks 👍 Looking good!

Two small requests so we can make it more future proof. Mostly so that we won't be confused when working on this in the future.

Can you apply those changes?

@bmeverett
Copy link
Contributor Author

Thanks @smeijer! I've applied the fixes you requested. Thanks for tidying that up.

Copy link
Member

@smeijer smeijer left a comment

Choose a reason for hiding this comment

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

Thanks!

@smeijer smeijer changed the title fix: css issue in safari (#238) fix: fix css flex/height issues in safari Jul 6, 2020
@smeijer smeijer merged commit 2c67fc5 into testing-library:develop Jul 6, 2020
@smeijer
Copy link
Member

smeijer commented Jul 6, 2020

@all-contributors please add @bmeverett for code

@allcontributors
Copy link
Contributor

@smeijer

I've put up a pull request to add @bmeverett! 🎉

@bmeverett bmeverett deleted the bug/safari-css branch July 6, 2020 19:32
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.

Safari doesn't use full screen space
2 participants