Page MenuHomePhabricator

Moved paragraphs in diff use a tag with name, which is obsolete in HTML5
Open, LowestPublic

Description

The HTML of a moved paragraph in diff has something like <a name="movedpara_1_1_lhs"></a>. While still conforming, this is obsolete in HTML (https://html.spec.whatwg.org/multipage/obsolete.html#warnings-for-obsolete-but-conforming-features), so it should be replaced by a span with an id.

Event Timeline

As far as I can tell, the divs in diffs (at least in normal diffs, don't know about moved paragraphs) don't have an ID, so it might be tidier to append the ID to that one instead of creating a new span?