Opened 10 years ago
Closed 10 years ago
#29485 closed defect (bug) (fixed)
PageUp can cause jumpiness in editor scrolling
Reported by: | nacin | Owned by: | nacin |
---|---|---|---|
Milestone: | 4.0 | Priority: | normal |
Severity: | normal | Version: | 4.0 |
Component: | Editor | Keywords: | has-patch commit dev-reviewed |
Focuses: | Cc: |
Description
Initially reported by matt, and the two of us were the only people who could reproduce. (See IRC discussion to be linked.) Video: https://cloudup.com/igMKu-5ANFd.
Turns out our mceKeyup listener doesn't bail on pageup/pagedown. It should bail on special keys along the lines of [29535], though the only ones that should actually cause a problem are pageup/pagedown/home/end. Two different patches attached depending on the level of coverage we need.
Attachments (3)
Change History (10)
This ticket was mentioned in IRC in #wordpress-dev by nacin. View the logs.
10 years ago
#5
@
10 years ago
- Keywords commit added
Okay, now I understand how this is reproduced. Scroll down a post, click somewhere, then scroll up far enough until the cursor is out of the screen. Then start hitting special keys that *shouldn't* cause it to scroll back to the cursor.
This isn't just pagedown/pageup. It's also all modifier keys, including Windows/Mac (cmd) keys, function keys, etc... Basically, everything in [29535], just not being DRY about it.
Pretty sure this is all that is needed.