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

Use vertical scroll for horizontal scrollbar if only horizontal scrollbar is present #64

Merged
merged 1 commit into from Sep 30, 2013

Conversation

jtmalinowski
Copy link
Contributor

It might be slightly more convenient to be able to use vertical scroll wheel for horizontal scrollbar if vertical scrollbar is not present, what do you think?

@hatashiro
Copy link
Contributor

Sounds good! But how about making it optional? Conventionally, the features that is not supported by native scrollbar have been adopted with optional parameters.

You can find the example of the optional feature here: #48.

And the min files should not be included. Please remove them from the diff.

Thank you!

@jtmalinowski
Copy link
Contributor Author

Done, updated docs too. Let me know if it's alright.

@hatashiro
Copy link
Contributor

Seems good!

And for the last one, I'm very sorry but can you add an example code in 'examples' directory?
There're example codes for optional parameters to help people know how to use the option.

Thank you.

hatashiro pushed a commit that referenced this pull request Sep 30, 2013
Use vertical scroll for horizontal scrollbar if only horizontal scrollbar is present.
@hatashiro hatashiro merged commit b92ece2 into mdbootstrap:master Sep 30, 2013
@hatashiro
Copy link
Contributor

Never mind. I've added an example while refactoring the code :D

Thank you very much!

@jtmalinowski
Copy link
Contributor Author

Was going to take a stab at it today... Anyway, awesome!

hatashiro pushed a commit that referenced this pull request Apr 29, 2014
Use vertical scroll for horizontal scrollbar if only horizontal scrollbar is present.
hatashiro pushed a commit that referenced this pull request Sep 24, 2014
Use vertical scroll for horizontal scrollbar if only horizontal scrollbar is present.
@mdbootstrap mdbootstrap locked as resolved and limited conversation to collaborators Mar 14, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants