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

Igor Poboiko <igor.pobo...@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|                            |https://commits.kde.org/kjo
                   |                            |ts/4f861fa9211b31b1e810696b
                   |                            |724d8459e2251a7c
             Status|REPORTED                    |RESOLVED
         Resolution|---                         |FIXED

--- Comment #3 from Igor Poboiko <igor.pobo...@gmail.com> ---
Git commit 4f861fa9211b31b1e810696b724d8459e2251a7c by Igor Poboiko.
Committed on 26/04/2020 at 21:09.
Pushed by poboiko into branch 'master'.

[KJotsEdit] Don't mess with margins when pasting rich text

Summary:
Original code contained weird html parsing, removing `margin-*: *px;' style
properties from raw HTML.
However, when inserted, `QTextDocument` sees no margins and decides to set them
to the default 12px,
which breaks even copy-pasting of non-formatted text (see attached).

Instead just use standard `QTextDocument` manipulation methods to remove
margins. It fixes both pasting
formatted and non-formatted text.

Test Plan:
Before: {F8259687}
After: {F8259688}

Reviewers: dvratil, mlaurent

Reviewed By: mlaurent

Subscribers: kde-pim

Tags: #kde_pim

Differential Revision: https://phabricator.kde.org/D29177

M  +15   -29   src/kjotsedit.cpp

https://commits.kde.org/kjots/4f861fa9211b31b1e810696b724d8459e2251a7c

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to