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

Jan Kundrát <j...@kde.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|CONFIRMED                   |RESOLVED
         Resolution|---                         |FIXED
      Latest Commit|                            |https://commits.kde.org/tro
                   |                            |jita/434f677d4909e751ef59d2
                   |                            |c8b5e2899b05eae022

--- Comment #5 from Jan Kundrát <j...@kde.org> ---
Git commit d42deaec263c1a0cd00d0391f0b75b48c3471b0d by Jan Kundrát.
Committed on 05/02/2017 at 14:37.
Pushed by gerrit into branch 'master'.

GUI: Extracting e-mail addresses from a message body

Right now, we delegate link copying to WebKit, which means that it will
use URLs like "mailto:f...@example.org";. That is suboptimal from the
user's point of view; they typically just "want to have that e-mail
address".

This patch ensures that we do this fancy thing when the URL is really a
simple one -- pointing to exactly one recipient. As a fallback option,
all other cases are handled by the existing code which just copies the
URL as-is.

Please note that this does not fix the issue that the linked bugreport
complained about -- the header widget with e-mail addresses is a QLabel,
so we will have to do a slightly different dance for that.

Change-Id: I930f4eecd291af643c736701849d4c92fc8fc43d

M  +18   -3    src/Gui/SimplePartWidget.cpp
M  +1    -0    src/Gui/SimplePartWidget.h

https://commits.kde.org/trojita/d42deaec263c1a0cd00d0391f0b75b48c3471b0d

--- Comment #6 from Jan Kundrát <j...@kde.org> ---
Git commit 434f677d4909e751ef59d2c8b5e2899b05eae022 by Jan Kundrát.
Committed on 06/02/2017 at 11:29.
Pushed by gerrit into branch 'master'.

GUI: Clipboard copying of e-mail addresses from the header view

Previously, the context menu offered some options for accessing the
QLabel (this class is actually just a pretty QLabel)'s text, for
selecting all of it, and copying stuff.

The most important, IMHO, is to have access to the actual e-mail address
(as requested by the associated bugreport), and to the human readable
text for the sake of completeness.

The code is still a tad inconsistent because the List-Post header for
indicating an active mailing list does not currently use the
OneEnvelopeAddress class. Mailing list addresses will therefore remain
unaffected by this patch; their copy action is still going to copy a
complete mailto: URL.
Change-Id: I4e515045d0241e4c1064389aa777f58e72c7c999

M  +26   -2    src/Gui/OneEnvelopeAddress.cpp
M  +2    -0    src/Gui/OneEnvelopeAddress.h

https://commits.kde.org/trojita/434f677d4909e751ef59d2c8b5e2899b05eae022

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

Reply via email to