[Differential] [Commented On] D1554: Add GUI feedback if no system QtHelp file is available

cordlandwehr (Andreas Cord-Landwehr) noreply at phabricator.kde.org
Tue May 10 19:17:13 UTC 2016


cordlandwehr added inline comments.

INLINE COMMENTS
  documentation/qthelp/qthelpconfig.cpp:391 My intention was to make clear that this is a configuration page for the API documentations one can install for different frameworks/languages. "QtHelp Documentation" is correct from the implementation details, yet a user won't care if it is in QCH files or different containers. Furthermore, I did not use "Documentation" since that easily can be confused with documentation about KDevelop.
  Actually, IMO PHP Docs is API documentation and fits will under this caption.
  Which caption would you prefer?
  documentation/qthelp/qthelpqtdoc.cpp:120 I see your point. The logic however was already there and worked :)
  I will have a look *why* it works this way and add a comment explaining this.

REPOSITORY
  rKDEVELOP KDevelop

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

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: cordlandwehr, kfunk, KDevelop
Cc: apol, kfunk, kdevelop-devel, Pilzschaf, akshaydeo, patrickelectric, surgenight, joshiakshay, arrowdodger


More information about the KDevelop-devel mailing list