Skip to content

Workaround Safari bug when zooming to < 100%#1389

Merged
waylan merged 1 commit intomkdocs:masterfrom
yeraydiazdiaz:fix-safari-bug
Jan 21, 2018
Merged

Workaround Safari bug when zooming to < 100%#1389
waylan merged 1 commit intomkdocs:masterfrom
yeraydiazdiaz:fix-safari-bug

Conversation

@yeraydiazdiaz
Copy link
Copy Markdown
Contributor

Fixes #1050

Tested on Safari 11.0.2 on Mac OS X Sierra.

Haven't tried with Cinder but I assume it's the same problem, Safari simply ignores the width: 25% of the col-md-3 and pushes the col-md-9 over it. Overriding the padding and width seems to work without affecting other browsers. It's a fairly hideous workaround but aren't they all.

@waylan waylan merged commit 6265416 into mkdocs:master Jan 21, 2018
@waylan
Copy link
Copy Markdown
Contributor

waylan commented Jan 21, 2018

@yeraydiazdiaz thanks for these. We don't really have anyone working on front end stuff right now, so any and all help in the area is much appreciated.

@lebigot
Copy link
Copy Markdown

lebigot commented Jan 21, 2018

Thanks @yeraydiazdiaz! Did you report this bug to Apple as well (unless already reported)?

@yeraydiazdiaz
Copy link
Copy Markdown
Contributor Author

@waylan no worries, thanks to Tom and all the contributors for your efforts :)

@lebigot I haven't, no, I'll see if I can test it on a newer version of Safari first

waylan added a commit to waylan/mkdocs that referenced this pull request Jul 10, 2018
The sidebar was under the left margin of the main content and not 
clickable. This moves the sidebar to the top so it is accessable. Not 
sure when this got broken. Maybe mkdocs#1387 or mkdocs#1389 or mkdocs#1395.
@waylan waylan mentioned this pull request Jul 10, 2018
waylan added a commit that referenced this pull request Jul 10, 2018
The sidebar was under the left margin of the main content and not 
clickable. This moves the sidebar to the top so it is accessable. Not 
sure when this got broken. Maybe #1387 or #1389 or #1395.
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