D18320: Search-enable the application page

Dan Leinir Turthra Jensen noreply at phabricator.kde.org
Fri Jan 18 09:03:40 GMT 2019


leinir added a comment.


  In D18320#395251 <https://phabricator.kde.org/D18320#395251>, @ngraham wrote:
  
  > Thanks for the copious code comments. Definitely makes things easier to understand in the future.
  
  
  i do prefer to overcomment rather than undercomment... i've come back to things too many times and gone "huh, must be a reason" and then gone searching for ages to discover the reason which, of course, is never just straightforward - i want future-me to not squint disappointingly at past-me ;)

REPOSITORY
  R134 Discover Software Store

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

To: leinir, apol, ngraham, #discover_software_store
Cc: plasma-devel, jraleigh, bam, GB_2, 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/20190118/cd796873/attachment-0001.html>


More information about the Plasma-devel mailing list