D17050: Allow clicking of links in app descriptions

Dan Leinir Turthra Jensen noreply at phabricator.kde.org
Tue Nov 20 13:52:15 GMT 2018


leinir created this revision.
leinir added reviewers: Discover Software Store, apol.
leinir added a project: Discover Software Store.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
leinir requested review of this revision.

REVISION SUMMARY
  Prior to this patch, links found in KNewStuff content descriptions were ignored and both unclickable and unselectable. With this patch, clicking a link will launch the default browser pointed at that link. Currently this is limited to KNS content, but it would be reasonably trivial to add to other content types as well, but thought it would be best to discuss on one type before doing them all at once.
  
  This patch further fixes a minor oversight in the BBCode neutraliser in the KNS backend (which would previously eat chunks it was not supposed to, such as parts of URLs).
  
  BUG: 401176

REPOSITORY
  R134 Discover Software Store

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

AFFECTED FILES
  discover/qml/ApplicationPage.qml
  libdiscover/backends/KNSBackend/KNSResource.cpp

To: leinir, #discover_software_store, apol
Cc: plasma-devel, masilva, ragreen, ixoos, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20181120/2deb139f/attachment.html>


More information about the Plasma-devel mailing list