D29087: [KJotsTreeView] Fix "copy link address" action

Daniel Vrátil noreply at phabricator.kde.org
Wed Apr 22 13:07:28 BST 2020


dvratil requested changes to this revision.
dvratil added inline comments.
This revision now requires changes to proceed.

INLINE COMMENTS

> kjotstreeview.cpp:179
> +    const QString title = idx.data().toString();
> +    const QString link = i18nc("clipboard contents; %1 is internal link to the page (i.e. kjots://org.kjots.page/1), %2 is page title",
> +                               "<a href=\"%1\">%2</a>", idx.data(KJotsModel::UrlRole).toString(), title);

Does this actually need to be localized? There's nothing to localize, really :)

REPOSITORY
  R573 KJots

REVISION DETAIL
  https://phabricator.kde.org/D29087

To: poboiko, dvratil
Cc: kde-pim, fbampaloukas, dcaliste, dvasin, rodsevich, winterz, vkrause, mlaurent, knauss, dvratil
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-pim/attachments/20200422/a15c1f85/attachment.html>


More information about the kde-pim mailing list