Last modified: 2013-07-31 01:09:47 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 T54014, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 52014 - VisualEditor: floating toolbar causes scrolling of large pages to be very slow
VisualEditor: floating toolbar causes scrolling of large pages to be very slow
Status: RESOLVED FIXED
Product: VisualEditor
Classification: Unclassified
Editing Tools (Other open bugs)
unspecified
All All
: High normal
: VE-deploy-2013-08-15
Assigned To: Krinkle
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-07-25 05:12 UTC by Tim Starling
Modified: 2013-07-31 01:09 UTC (History)
5 users (show)

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


Attachments

Description Tim Starling 2013-07-25 05:12:52 UTC
In Firefox, scrolling a large VE page is very slow, with visible pauses between repaints, predominantly because of the floating toolbar. Calling disableFloating() on the toolbar from the JS console causes scrolling to be roughly as fast as scrolling the non-VE page view.

Timo says he knows how to fix this.
Comment 1 Inez Korczyński 2013-07-25 05:38:07 UTC
@Krinkle: How are you going to fix this? :-)
Comment 2 Tim Starling 2013-07-25 05:41:13 UTC
(In reply to comment #1)
> @Krinkle: How are you going to fix this? :-)

<Krinkle> I already did, in a way. Earlier this week I re-used that logic for mw.notification (which now also has a floating mode, previously mw.notify messages weren't visible if you scroll down the page, which is a problem for VE). However I noticed a few things that could be optimised that I did 'right' from the get go for mw.notify. I should be able to apply that to ve as well
<Krinkle> most notably the fact that floating calculation for ve toolbar calls offset() each time
<Krinkle> which is a repaint for something that is unlikely to ever change
<Krinkle> I realised it when I was doing mwnotify but never remembered to fix it in VE
Comment 3 Tim Starling 2013-07-29 00:22:06 UTC
https://gerrit.wikimedia.org/r/#/c/76241/
Comment 4 Gerrit Notification Bot 2013-07-29 04:27:57 UTC
Change 76241 had a related patch set uploaded by Jforrester:
ve.ui.Toolbar: Refactor floating logic for performance

https://gerrit.wikimedia.org/r/76241
Comment 5 Gerrit Notification Bot 2013-07-30 22:31:49 UTC
Change 76241 merged by jenkins-bot:
ve.ui.Toolbar: Refactor floating logic for performance

https://gerrit.wikimedia.org/r/76241
Comment 6 James Forrester 2013-07-31 01:09:47 UTC
Merged and deployed.

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


Navigation
Links