Horizontal scrollbar (.md-content) #108

Closed
makshh opened this Issue Jan 14, 2017 · 3 comments

Comments

Projects
None yet
2 participants
@makshh

makshh commented Jan 14, 2017

Description

Scrollbar

Expected behavior

-No scrollbar on .md-content
-Scrollbars on code blocks.

Actual behavior

-Scrollbar on .md-content when I have long text in code blocks.
-No scrollbars on code blocks.

Steps to reproduce the bug

Add long text to code block and scroll to bottom (desktop view).

Package versions

  • MkDocs: 0.16.1
  • Material for MkDocs: 1.0.1

System information

  • OS: MacOS
  • Browser: Chrome/Firefox

Possible solutions

  1. Remove float: left from .md-content__inner
  2. Add max-width: calc(100% - 4.8rem) to .md-content__inner
@squidfunk

This comment has been minimized.

Show comment
Hide comment
@squidfunk

squidfunk Jan 14, 2017

Owner

Thanks for reporting. That is indeed pretty bad. I just checked and it's possible to remove the float while still minimizing repaints. The overflow does the trick due to margin collapse.

Owner

squidfunk commented Jan 14, 2017

Thanks for reporting. That is indeed pretty bad. I just checked and it's possible to remove the float while still minimizing repaints. The overflow does the trick due to margin collapse.

@squidfunk

This comment has been minimized.

Show comment
Hide comment
@squidfunk

squidfunk Jan 14, 2017

Owner

I provided a fix in #109 which is merged into master. Will make a bugfix release tomorrow.

Owner

squidfunk commented Jan 14, 2017

I provided a fix in #109 which is merged into master. Will make a bugfix release tomorrow.

@squidfunk squidfunk self-assigned this Jan 14, 2017

@squidfunk squidfunk added the bug label Jan 14, 2017

@squidfunk squidfunk added this to the 1.0.2 milestone Jan 14, 2017

@squidfunk

This comment has been minimized.

Show comment
Hide comment
@squidfunk

squidfunk Jan 15, 2017

Owner

Released 1.0.2 today which fixes the issue.

Owner

squidfunk commented Jan 15, 2017

Released 1.0.2 today which fixes the issue.

@squidfunk squidfunk closed this Jan 15, 2017

@squidfunk squidfunk added the regression label Feb 8, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment