[okular] [Bug 510945] okular fails to render strike through formatted text of markdown documents (2)

[Sune Vuorela]

https://bugs.kde.org/show_bug.cgi?id=510945

Sune Vuorela <sune at vuorela.dk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|https://invent.kde.org/grap |https://invent.kde.org/grap
                   |hics/okular/-/commit/656683 |hics/okular/-/commit/2080ad
                   |8bb259622f023476af17f753ae4 |79ab08d17c5f7f244bad36c108a
                   |a9b3530                     |69bd7f1

--- Comment #8 from Sune Vuorela <sune at vuorela.dk> ---
Git commit 2080ad79ab08d17c5f7f244bad36c108a69bd7f1 by Sune Vuorela.
Committed on 14/11/2025 at 09:38.
Pushed by sune into branch 'release/25.12'.

Do not process HTML with QDomDocument

`QDomDocument` was used to replace `<del>` tags in Discount's output with `<s>`
tags.

`QDomDocument::setContent()` parses XML only. This usually works, because
Discount's HTML is usually valid XML. However, since it is in "Smartypants"
mode, Discount generates named character references in response to certain
inputs, e.g. `(c)` -> `©` and `...` -> `…`. These are valid HTML,
but most are not predefined in standard XML, and so QDomDocument refuses to
parse them.

This MR uses `QString::replace()` in place of `QDomDocument`.

I know it's generally frowned upon to process HTML by such simple approaches,
but within the constraints of HTML which Discount generates, I can't see a way
that this could go wrong.


(cherry picked from commit 6566838bb259622f023476af17f753ae4a9b3530)

9acf9eeb Do not process HTML with QDomDocument
1337effa Removed wrapper tags from Markdown tests
8238e4e7 Whitespace-only changes to some Markdown tests

Co-authored-by: Ben Morris <bugs at benmorris.org.uk>

M  +6    -18   autotests/markdowntest.cpp
M  +4    -28   generators/markdown/converter.cpp

https://invent.kde.org/graphics/okular/-/commit/2080ad79ab08d17c5f7f244bad36c108a69bd7f1

-- 
You are receiving this mail because:
You are the assignee for the bug.