Last modified: 2013-05-20 23:51:48 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 T49672, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 47672 - VisualEditor: Sequential floated images are stacking
VisualEditor: Sequential floated images are stacking
Status: RESOLVED FIXED
Product: VisualEditor
Classification: Unclassified
ContentEditable (Other open bugs)
unspecified
All All
: Normal normal
: VE-deploy-2013-05-27
Assigned To: Inez Korczyński
http://en.wikipedia.org/wiki/Principa...
:
Depends on:
Blocks: 48426
  Show dependency treegraph
 
Reported: 2013-04-25 21:39 UTC by Owen Blacker
Modified: 2013-05-20 23:51 UTC (History)
4 users (show)

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


Attachments

Description Owen Blacker 2013-04-25 21:39:15 UTC
Looking at [[Principality of Stavelot-Malmedy]] in the Visual Editor, immediately after the heading "Development and the High Middle Ages" there are two images floated right.

In Read mode, these images are given `float: right; clear: right;`. In VE mode, however, the `clear: right;` appears to be missing, so the images are stacking up.

In case it makes a difference, I'm running Chrome 26.0.1410.64 m, on 64-bit Windows 7 Home Premium (Build 7601, Service Pack 1).
Comment 1 James Forrester 2013-04-25 21:43:37 UTC
Confirmed - the DM layer for alien items (like images) apparently respects the CSS and is meant to pass it through, but CE isn't?
Comment 2 Roan Kattouw 2013-04-25 22:03:14 UTC
This is because Parsoid outputs completely different HTML for images (using <figure> and what not) that doesn't match the MediaWiki CSS.

This will be fixed once we change the ce.MWThumbImageNode rendering to match the MW rendering rather than the Parsoid rendering, and move image support out of the experimental list.
Comment 3 James Forrester 2013-05-20 23:50:44 UTC
This was fixed by Gerrit change #64367 which is now merged, but is 'experimental' and
so will not go out for a short while. Marking as fixed within wmf5, though.

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


Navigation
Links