In T185708#3949390, @TheDJ wrote:
In T185708#3950163, @Deskana wrote:To my knowledge, there is no particular reason it is this way. I agree it should be changed.
• Deskana | |
Feb 6 2018, 7:47 PM |
F13186510: image.png | |
Feb 7 2018, 10:59 AM |
F13186227: image.png | |
Feb 7 2018, 10:51 AM |
F13124738: Screen Shot 2018-02-06 at 17.15.13.png | |
Feb 6 2018, 7:47 PM |
The padding is for showing the move markers:
In the historical diffs (and on Vector) there is already a margin, so we could apply a negative margin to the whole element.
Removing the margin would be a problem in Monobook though:
These left margin markers may not exist after the next design iteration though (T169325)
We could also track if there were any moves at all and only show the margin in those cases, similar to what we do with the right sidebar.
Change 408779 had a related patch set uploaded (by Esanders; owner: Esanders):
[VisualEditor/VisualEditor@master] DiffElement: Only show left margin if moves present
Change 408779 merged by jenkins-bot:
[VisualEditor/VisualEditor@master] DiffElement: Only show left margin if moves present
Change 409632 had a related patch set uploaded (by Jforrester; owner: Jforrester):
[mediawiki/extensions/VisualEditor@master] Update VE core submodule to master (c39a4a69b)
Change 409632 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor@master] Update VE core submodule to master (c39a4a69b)