Set the tooltip bounds container to document.body if undefined #944

benbro · 2016-09-09T20:33:54Z

jhchen · 2016-09-09T20:53:06Z

core/quill.js

@@ -298,7 +298,6 @@ class Quill {
  }
 }
 Quill.DEFAULTS = {
-  bounds: document.body,


Can you keep bounds: null in the list instead so DEFAULTS still has all the possible options listed?

jhchen · 2016-09-09T21:36:56Z

Thanks!

Set the tooltip bounds container to document.body if undefined

38f0359

jhchen reviewed Sep 9, 2016
View reviewed changes

Set bounds to null in the defults

2f0fde8

jhchen merged commit 55a8bab into slab:develop Sep 9, 2016

benbro deleted the bounds branch September 9, 2016 21:49

This was referenced Sep 13, 2016

Cannot load quill in polymer component #904

Closed

Cannot read property 'getBoundingClientRect' of null #969

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Set the tooltip bounds container to document.body if undefined #944

Set the tooltip bounds container to document.body if undefined #944

benbro commented Sep 9, 2016

jhchen Sep 9, 2016

jhchen commented Sep 9, 2016

Set the tooltip bounds container to document.body if undefined #944

Set the tooltip bounds container to document.body if undefined #944

Conversation

benbro commented Sep 9, 2016

jhchen Sep 9, 2016

Choose a reason for hiding this comment

jhchen commented Sep 9, 2016