Last modified: 2014-03-14 00:45:25 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 T64172, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 62172 - VisualEditor: [Regression wmf16] Getting "Your edit may have been corrupted – please review before saving" message when page contains gallery/formula
VisualEditor: [Regression wmf16] Getting "Your edit may have been corrupted ...
Status: RESOLVED FIXED
Product: VisualEditor
Classification: Unclassified
Editing Tools (Other open bugs)
unspecified
All All
: High normal
: VE-deploy-2014-03-06
Assigned To: Roan Kattouw
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-03-03 23:47 UTC by Rummana Yasmeen
Modified: 2014-03-14 00:45 UTC (History)
7 users (show)

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


Attachments
Screenshot (72.65 KB, image/png)
2014-03-03 23:47 UTC, Rummana Yasmeen
Details

Description Rummana Yasmeen 2014-03-03 23:47:16 UTC
Created attachment 14731 [details]
Screenshot

Steps to reproduce:

Yet Unknown.I am getting it by following no specific steps.
Comment 1 Rummana Yasmeen 2014-03-04 01:21:22 UTC
Steps to reproduce:  (found later )

1.Open a page with VE
2.Add a formula or a gallery
3.Save the page

Observed Result:
 "Your edit may have been corrupted – please review before saving" appears on the save dialog box
Comment 2 Ed Sanders 2014-03-04 14:40:51 UTC
Confirmed on http://en.wikipedia.beta.wmflabs.org/wiki/Editcorrupted?veaction=edit with not actual corruption, but can't reproduce locally.
Comment 3 Roan Kattouw 2014-03-05 23:38:07 UTC
This is not because of galleries of formulas, but because of Moriel's fixes for default images sizes in the DM. So it manifests only for images with default size.

The corruption occurs because the default image size fixes overwrite Parsoid's width/height (which is reasonable) but then also send the overwritten values back to Parsoid (which is not reasonable). We should store the original values and send those back in the HTML unless the user changed something.

Additionally, the default size fixes are causing us to incorrectly size images whose original size is less than 220px. An 16x16px icon, for instance, is 16x16 even though the default bounding box is 220, because default sizing never grows an image above its original size. In this case Parsoid's width/height are actually correct, but we overwrite that with the default size, and so 16x16px icons end up rendering as 220x220px in VE and look horribly pixelated.
Comment 4 Gerrit Notification Bot 2014-03-06 01:45:37 UTC
Change 117144 had a related patch set uploaded by Catrope:
Work around DOM corruption for default-sized images

https://gerrit.wikimedia.org/r/117144
Comment 5 Gerrit Notification Bot 2014-03-06 02:10:13 UTC
Change 117144 merged by jenkins-bot:
Work around DOM corruption for default-sized images

https://gerrit.wikimedia.org/r/117144
Comment 6 Rummana Yasmeen 2014-03-06 23:56:10 UTC
Verified the fix on betalabs  on this page: http://en.wikipedia.beta.wmflabs.org/wiki/Editcorrupted
Comment 7 Rummana Yasmeen 2014-03-07 00:38:36 UTC
verified the fix on test2
Comment 8 Gerrit Notification Bot 2014-03-07 18:43:15 UTC
Change 117445 had a related patch set uploaded by Catrope:
Work around DOM corruption for default-sized images

https://gerrit.wikimedia.org/r/117445
Comment 9 Gerrit Notification Bot 2014-03-07 20:09:46 UTC
Change 117445 merged by jenkins-bot:
Work around DOM corruption for default-sized images

https://gerrit.wikimedia.org/r/117445
Comment 10 Rummana Yasmeen 2014-03-14 00:45:25 UTC
Verified the fix in production:https://en.wikipedia.org/wiki/User:RYasmeen_%28WMF%29/sandbox

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


Navigation
Links