Page MenuHomePhabricator

Floated image in visual diffs on history page overlapping content
Closed, ResolvedPublic1 Estimated Story Points

Description

In https://de.wikipedia.org/w/index.php?title=Wikipedia:Technik/Werkstatt&type=revision&diff=174179377&oldid=174177794&diffmode=visual, the added image floats into the content. There should be a clear: both; somewhere to prevent this.
Note that (as far as I can see) this only happens when floating content is added to the very last section, as otherwise the indicator that the following content is omitted in the diff will do the clearing.

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript
Esanders renamed this task from Clear floats in visual diff to Visual diffs on history page overlapping content.Feb 20 2018, 2:51 PM
Esanders renamed this task from Visual diffs on history page overlapping content to Floated image in visual diffs on history page overlapping content.Feb 20 2018, 3:03 PM

Change 412926 had a related patch set uploaded (by Esanders; owner: Esanders):
[mediawiki/extensions/VisualEditor@master] Add clearfix to diff page's visual diff element

https://gerrit.wikimedia.org/r/412926

Change 412926 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor@master] Add clearfix to diff page's visual diff element

https://gerrit.wikimedia.org/r/412926

Deskana set the point value for this task to 1.

Change 644899 had a related patch set uploaded (by Esanders; owner: Esanders):
[VisualEditor/VisualEditor@master] Upstream diffElement clearfix

https://gerrit.wikimedia.org/r/644899

Change 644899 merged by jenkins-bot:
[VisualEditor/VisualEditor@master] Upstream diffElement clearfix

https://gerrit.wikimedia.org/r/644899

Change 644925 had a related patch set uploaded (by Bartosz Dziewoński; owner: Bartosz Dziewoński):
[mediawiki/extensions/VisualEditor@master] Update VE core submodule to master (b615c2e21)

https://gerrit.wikimedia.org/r/644925

Change 644925 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor@master] Update VE core submodule to master (b615c2e21)

https://gerrit.wikimedia.org/r/644925