Use DEL / INS tags for Diff comparions changes rather than SPAN.diffchange

Created on 5 December 2016, about 8 years ago
Updated 31 January 2023, almost 2 years ago

Problem/Motivation

To use INS and DEL tags to highlight insertions and deletions from Diff comparions, rather than SPAN tags.

As a conclusion from the discussion - #685210: Use DEL / INS instead of SPAN β†’ , all modern applications should be using proper HTML 5 syntax. There is a screenshot attached with the diff markup.

Proposed resolution

Detect type of action and use an INS or DEL tag within DiffEngine's HWLDFWordAccumulator::_flushGroup().

Remaining tasks

User interface changes

API changes

Data model changes

πŸ“Œ Task
Status

Needs work

Version

10.1 ✨

Component
MarkupΒ  β†’

Last updated about 2 months ago

No maintainer
Created by

πŸ‡¨πŸ‡­Switzerland drobnjak

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • πŸ‡ΊπŸ‡ΈUnited States smustgrave

    This issue is being reviewed by the kind folks in Slack, #needs-review-queue-initiative. We are working to keep the size of Needs Review queue [2700+ issues] to around 400 (1 month or less), following Review a patch or merge request β†’ as a guide.

    This type of change feels like, even though it's a task, it will need a test case. So maybe it's a feature request?

Production build 0.71.5 2024