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

Documentation clarification of the selector for scrollingContainer #1496

Closed
sferoze opened this issue Jun 7, 2017 · 1 comment
Closed

Documentation clarification of the selector for scrollingContainer #1496

sferoze opened this issue Jun 7, 2017 · 1 comment

Comments

@sferoze
Copy link
Contributor

sferoze commented Jun 7, 2017

From the documentation on the scrolling container

scrollingContainer

Default: null

Specifies which container has the scrollbars (i.e. overflow-y: auto), if changed with CSS from the default ql-editor. Necessary to fix scroll jumping bugs when Quill is set to auto grow its height, and another ancestor container is responsible from the scrolling.

But it does not clarify what can be passed in. Can it be the actual element or only a string? From testing I see it can only be a string.

@jhchen Would passing 'html, body' work as the selector for scrollingContainer?

@jhchen
Copy link
Member

jhchen commented Jun 8, 2017

It can take a DOM element or a CSS selector so yes html, body should work.

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

No branches or pull requests

2 participants