Last modified: 2014-10-06 15:52:33 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T64812, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 62812 - VisualEditor: setResizableHandlesSizeAndPosition() shouldn't measure things on every keypress
VisualEditor: setResizableHandlesSizeAndPosition() shouldn't measure things o...
Status: RESOLVED FIXED
Product: VisualEditor
Classification: Unclassified
ContentEditable (Other open bugs)
unspecified
All All
: High enhancement
: VE-deploy-2014-07-24
Assigned To: Ed Sanders
: performance
Depends on:
Blocks: ve-performance
  Show dependency treegraph
 
Reported: 2014-03-19 00:26 UTC by Roan Kattouw
Modified: 2014-10-06 15:52 UTC (History)
6 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Roan Kattouw 2014-03-19 00:26:42 UTC
In afterDocumentKeyPress, setResizableHandlesSizeAndPosition() is called and measures and repositions all ResizableNode resize handles, even though they're not even visible. This is wasteful.
Comment 1 Roan Kattouw 2014-10-06 09:48:48 UTC
This was fixed at some point. setResizableHandlesSizeAndPosition() is now attached to the 'position' event on focus and detached on blur.
Comment 2 James Forrester 2014-10-06 15:52:33 UTC
Fixed in gerrit Ia6cf8de0da0.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links