Page MenuHomePhabricator

VisualEditor: Nested identical annotations can be additive, not just identical
Closed, ResolvedPublic

Description

<b><b>Foo</b></b> == <b>Foo</b> so it's not /so/ bad if VE corrects it (but it's still a bit dirty, but <small><small>Foo</small></small> != <small>Foo</small> (and the same with <big> and <sup> and …)


Version: unspecified
Severity: minor
URL: https://en.wikipedia.org/w/index.php?title=User:JohnCD/draft&diff=560430675&oldid=560430611

Details

Reference
bz49755

Event Timeline

bzimport raised the priority of this task from to High.Nov 22 2014, 2:02 AM
bzimport set Reference to bz49755.

*** Bug 49756 has been marked as a duplicate of this bug. ***

This causes round-trip failures.

Change 71716 had a related patch set uploaded by Esanders:
Allow annotations to be additive

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

The fix sets big/small/sup/sub to be additive.

Arguably we should make all annotations additive, as you could add 'style="font-size:110%"' to anything, although I haven't thought through how this might affect the toolbar interactions yet.

Change 71716 merged by jenkins-bot:
Allow annotations to be additive

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

(In reply to comment #4)

The fix sets big/small/sup/sub to be additive.

Arguably we should make all annotations additive, as you could add
'style="font-size:110%"' to anything, although I haven't thought through how
this might affect the toolbar interactions yet.

+1. This will also avoid dirty diffs on such content.

Change 72133 had a related patch set uploaded by Esanders:
Make all annotations additive

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

Change 72133 merged by jenkins-bot:
Make all annotations additive

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

Fixed on the VisualEditor side; the Parsoid upstream issue in Domino is patched but not yet released. Not marking as fixed until that's done.