Last modified: 2014-07-15 17:28:39 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 T69985, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 67985 - VisualEditor: Cursor positioning broken after about-grouped inline alien
VisualEditor: Cursor positioning broken after about-grouped inline alien
Status: RESOLVED FIXED
Product: VisualEditor
Classification: Unclassified
ContentEditable (Other open bugs)
unspecified
All All
: Highest major
: VE-deploy-2014-07-17
Assigned To: Ed Sanders
:
: 68037 (view as bug list)
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-07-14 12:39 UTC by Ed Sanders
Modified: 2014-07-15 17:28 UTC (History)
5 users (show)

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


Attachments

Description Ed Sanders 2014-07-14 12:39:30 UTC
From bug 58221:

Tilman Bayer 2014-07-14 04:16:20 UTC

Another example:

In https://en.wikipedia.org/w/index.php?oldid=616866203&veaction=edit , place the cursor left of "F" in the word "Former" in the first paragrah, and hit the delete key. The "F" is deleted correctly, but the cursor jumps two to the left (between "s" and ".").

Reproduced in Firefox and Chromium under Ubuntu 14.04.
Comment 1 Ed Sanders 2014-07-14 12:40:29 UTC
Anywhere we do logical cursoring you get off by two errors, e.g. delete/backspace/arrow keys (after clicking)
Comment 2 Ed Sanders 2014-07-14 12:53:23 UTC
The problem here appears to be the IPA template which returns two <span>s which are about-grouped (about="#mwtX"). I imagine the cursoring logic is expecting an alien to consist of only one node.
Comment 3 Gerrit Notification Bot 2014-07-14 15:37:40 UTC
Change 146079 had a related patch set uploaded by Esanders:
Prevent double counting of DM nodes in getNodeAndOffset

https://gerrit.wikimedia.org/r/146079
Comment 4 Gerrit Notification Bot 2014-07-14 17:05:51 UTC
Change 146079 merged by jenkins-bot:
Prevent double counting of DM nodes in getNodeAndOffset

https://gerrit.wikimedia.org/r/146079
Comment 5 Gerrit Notification Bot 2014-07-14 17:31:36 UTC
Change 146137 had a related patch set uploaded by Jforrester:
Prevent double counting of DM nodes in getNodeAndOffset

https://gerrit.wikimedia.org/r/146137
Comment 6 Gerrit Notification Bot 2014-07-14 17:31:56 UTC
Change 146138 had a related patch set uploaded by Jforrester:
Prevent double counting of DM nodes in getNodeAndOffset

https://gerrit.wikimedia.org/r/146138
Comment 7 Gerrit Notification Bot 2014-07-14 18:45:28 UTC
Change 146137 merged by jenkins-bot:
Prevent double counting of DM nodes in getNodeAndOffset

https://gerrit.wikimedia.org/r/146137
Comment 8 Gerrit Notification Bot 2014-07-14 18:45:40 UTC
Change 146138 merged by jenkins-bot:
Prevent double counting of DM nodes in getNodeAndOffset

https://gerrit.wikimedia.org/r/146138
Comment 9 Gerrit Notification Bot 2014-07-14 18:46:42 UTC
Change 146159 had a related patch set uploaded by Jforrester:
Update VE core submodule to e3fa7ec8

https://gerrit.wikimedia.org/r/146159
Comment 10 Gerrit Notification Bot 2014-07-14 18:47:55 UTC
Change 146161 had a related patch set uploaded by Jforrester:
Update VE core submodule to cfdb3a54

https://gerrit.wikimedia.org/r/146161
Comment 11 Gerrit Notification Bot 2014-07-14 18:58:38 UTC
Change 146159 abandoned by Jforrester:
Update VE core submodule to e3fa7ec8

https://gerrit.wikimedia.org/r/146159
Comment 12 Gerrit Notification Bot 2014-07-14 18:58:44 UTC
Change 146161 abandoned by Jforrester:
Update VE core submodule to cfdb3a54

https://gerrit.wikimedia.org/r/146161
Comment 13 Gerrit Notification Bot 2014-07-14 19:00:57 UTC
Change 146165 had a related patch set uploaded by Jforrester:
Update VE core submodule to e3fa7ec8

https://gerrit.wikimedia.org/r/146165
Comment 14 Gerrit Notification Bot 2014-07-14 19:04:58 UTC
Change 146166 had a related patch set uploaded by Jforrester:
Update VE core submodule to cfdb3a54

https://gerrit.wikimedia.org/r/146166
Comment 15 Gerrit Notification Bot 2014-07-14 23:06:05 UTC
Change 146165 merged by jenkins-bot:
Update VE core submodule to e3fa7ec8

https://gerrit.wikimedia.org/r/146165
Comment 16 Gerrit Notification Bot 2014-07-14 23:08:30 UTC
Change 146166 merged by Mwalker:
Update VE core submodule to cfdb3a54

https://gerrit.wikimedia.org/r/146166
Comment 17 James Forrester 2014-07-14 23:42:18 UTC
Now fixed. Thank you, all.
Comment 18 James Forrester 2014-07-15 17:28:39 UTC
*** Bug 68037 has been marked as a duplicate of this bug. ***

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


Navigation
Links