Opened 12 years ago
Closed 12 years ago
#29485 closed defect (bug) (fixed)
PageUp can cause jumpiness in editor scrolling
| Reported by: | nacin | Owned by: | nacin |
|---|---|---|---|
| Priority: | normal | Milestone: | 4.0 |
| Component: | Editor | Version: | 4.0 |
| Severity: | normal | Keywords: | has-patch commit dev-reviewed |
| Cc: | Focuses: |
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.
12 years ago
#5
@
12 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.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
Pretty sure this is all that is needed.