Uploaded image for project: 'eZ Publish / Platform'
  1. eZ Publish / Platform
  2. EZP-28044

As an Developer I want support for striked text in RichText

    Details

    • Type: Story Story
    • Status: Closed
    • Priority: High High
    • Resolution: None
    • Affects Version/s: None
    • Fix Version/s: 1.12.0
    • Component/s: None
    • Labels:
      None

      Description

      To be able to implement OE features listed in EZP-28030, striked <s> text needs to be supported.

      Currently only <del> is supported by RichText, but this means deleted text and has a different contextual meaning.

      After a bit searching it seems we should do the following mapping for these tags so we can also continue to support <del> which might be useful in the future.

      • <s> => <emphasis role="strikedthrough">
      • <del> => <emphasis role="strikedthrough" revisionflag="deleted">

      This also means that the meaning of strikedthrough text already migrated from eZ Publish will now correctly be displayed as strike in the editor and view, instead of being converted to <del> which editor did not support and which has other meaning.

        Issue Links

          Activity

          Show
          André Rømcke added a comment - PR: https://github.com/ezsystems/ezpublish-kernel/pull/2103
          Hide
          André Rømcke added a comment -

          Merged: https://github.com/ezsystems/ezpublish-kernel/commit/2056095133b2f4d26f29d974ee18df7c996d74e1

          QA: Don't think there is a need to do testing here, that is more relevant on the connected OE story once that is sent to Testing.

          Show
          André Rømcke added a comment - Merged: https://github.com/ezsystems/ezpublish-kernel/commit/2056095133b2f4d26f29d974ee18df7c996d74e1 QA: Don't think there is a need to do testing here, that is more relevant on the connected OE story once that is sent to Testing.

            People

            • Assignee:
              Unassigned
              Reporter:
              André Rømcke
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: