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

Using max-height does not work in IE11 #154

Closed
jacksonv1lle opened this issue Mar 22, 2018 · 3 comments
Closed

Using max-height does not work in IE11 #154

jacksonv1lle opened this issue Mar 22, 2018 · 3 comments
Labels

Comments

@jacksonv1lle
Copy link

This is a really great scrollbar. It satisfies all my needs except when I try to use max-height on IE11. I don't want to specify a height on my scrollable panels because I need them to collapse when there is no content. Is there any way for this to work?

@Grsmto
Copy link
Owner

Grsmto commented Mar 22, 2018

Hi @jacksonv1lle , I think I experienced this issue as well. Could you let me know if applying this css solves the issue?

.simplebar-scroll-content {
    max-height: inherit!important;
}

Thanks for your report!

@jacksonv1lle
Copy link
Author

@Grsmto Thank you sir. That has worked

@Grsmto
Copy link
Owner

Grsmto commented Mar 23, 2018

I will implement this fix in the plugin for the next version! Thanks!

@Grsmto Grsmto added the bug label Mar 23, 2018
@Grsmto Grsmto closed this as completed in 13b112c Mar 24, 2018
Grsmto pushed a commit that referenced this issue Apr 9, 2018
* master:
  Add an scrollMaxSize option (fix #125
  Refactor the way options are retrieved to be more scalable (no more mapping from data attributes)
  Direction option is now handled through options for performances reason (ref #162)
  Fix RTL support not positioning scrollbar properly (fix #153)
  Add a "caveats" section to the README (fix #91)
  2.6.1
  Remove !important rule on props that don't absolutely need it
  Overflow css rules shouldn't be defined as !important so user can opt-out from x/y scroll by overriding these easily
  Add a new option to force track to be visible (same behaviour as overflow: scroll, show track, hide scrollbar) (ref #129)
  Set SimpleBar to inherit from parent's max-height + fix issue in IE11 (fix #154)
  add dist files for 2.6.0
  2.6.0

# Conflicts:
#	dist/simplebar.css
#	dist/simplebar.js
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

2 participants