Last modified: 2013-05-07 04:45: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 T49712, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 47712 - VisualEditor: Trailing whitespace at end of paragraph or table cell dropped on round-trip
VisualEditor: Trailing whitespace at end of paragraph or table cell dropped o...
Status: RESOLVED FIXED
Product: VisualEditor
Classification: Unclassified
Data Model (Other open bugs)
unspecified
All All
: High major
: VE-deploy-2013-05-13
Assigned To: Ed Sanders
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-04-26 08:19 UTC by Nicolas Raoul
Modified: 2013-05-07 04:45 UTC (History)
9 users (show)

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


Attachments

Description Nicolas Raoul 2013-04-26 08:19:22 UTC
Ubuntu/Firefox
Go to http://en.wikipedia.org/wiki/Alfresco_%28software%29
At the end of a line, add a space and remove it
Click "Save"
No changes should have happen, but there actually are changes, in the infobox:

| caption = Alfresco Share / Repository Browser <br/> (Community Edition)
became
| caption =Alfresco Share / Repository Browser <br></table> (Community Edition)
Comment 1 Nicolas Raoul 2013-04-26 08:25:40 UTC
Other faulty roundtrips:

http://en.wikipedia.org/wiki/Neritina_rubricata
Removes space at end of line

http://en.wikipedia.org/wiki/Timeline_of_portable_computers
--> || becomes -->||

Should I continue reporting space removals like these 2 ones?
Or are these spaces removed on purpose?
Comment 3 James Forrester 2013-04-26 16:08:40 UTC
This is an instance of the jQuery bug 47737 - marking as a duplicate.

*** This bug has been marked as a duplicate of bug 47737 ***
Comment 4 Nicolas Raoul 2013-04-26 16:10:21 UTC
Do you mean all of the problem mentioned on this page are due to this bug?
Cheers!
Nicolas
Comment 5 Gabriel Wicke 2013-04-26 16:16:02 UTC
I believe the whitespace changes are a separate issue. Reopening so that those don't get lost.
Comment 6 James Forrester 2013-04-26 16:36:40 UTC
(In reply to comment #4)
> Do you mean all of the problem mentioned on this page are due to this bug?

Please open separate bug tickets for each bug in future. :-)
Comment 7 ssastry 2013-04-27 15:03:26 UTC
I tested the following 4 pages and am recording my findings here.  Once we go through the other pages, we should close this bug and create new bugs where necessary.

On the following two pages, I can reproduce in Firefox, but the edit is clean in Chrome.  So, this is definitely Firefox specific, not sure if it is JQuery-related or something else.

* http://en.wikipedia.org/wiki/Timeline_of_portable_computers
* http://en.wikipedia.org/wiki/Neritina_rubricata

On the following two pages, I can reproduce the spurious diffs in both Firefox and Chrome even though I didn't edit the relevant areas. I haven't looked at the diffs yet.  So, this is quite likely some kind of whitespace change in the DOM that VE sends Parsoid.

* http://en.wikipedia.org/wiki/Timeline_of_portable_computers -- It appears that the table is being marked as edited even though I didn't edit the table.

* http://en.wikipedia.org/wiki/Anne_Hendershott -- the list is being marked as edited because of some whitespace normalization when VE loads/saves the DOM.
Comment 8 Gerrit Notification Bot 2013-05-01 23:01:52 UTC
Related URL: https://gerrit.wikimedia.org/r/61919 (Gerrit Change I42daaf142e548e5b221ff0a52df0ad24ec6a4fd0)
Comment 9 James Forrester 2013-05-06 20:19:04 UTC
We're now three bugs fixed into this stack. Closing. Others should be opened as other bugs.
Comment 10 Gabriel Wicke 2013-05-07 03:26:02 UTC
http://en.wikipedia.org/wiki/Anne_Hendershott still shows the same symptoms. Is this fix not yet deployed?
Comment 11 James Forrester 2013-05-07 04:45:47 UTC
(In reply to comment #10)
> http://en.wikipedia.org/wiki/Anne_Hendershott still shows the same symptoms.
> Is
> this fix not yet deployed?

Sorry, the meaning of "VE-deploy-2013-05-13" is meant to be pretty clear, but evidently isn't! It'll go out with 1.22wmf4, so on the English Wikipedia on 20 May.

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


Navigation
Links