<table><tr><td style="">rjvbb updated this revision to Diff 20877.<br />rjvbb edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-loi3f5qirz3jh6s/" rel="noreferrer">(Show Details)</a><br />rjvbb added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D8211" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>updated to match <a href="https://phabricator.kde.org/D4484" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">D4484</a></p></div></div><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;"><div style="padding: 8px 0;">...</div>Using QTextBrowser is implemented as a fallback, you have to disable WebEngine and WebKit support (or not have either component installed) to activate this mode. I would suggest to make this the default mode since it doesn't require extra dependencies, and focus on adding proper support for opening links in an external browser of choice at least for the QtHelp documentation. I currently have a working implementation that adds a contextmenu action to achieve this, using a remotely controlled instance of Qt's Assistant. I am looking into modifying the "Show documentation for" action in the context-help popups to trigger that external viewer also.<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);"><br />
</span></div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D8211?vs=20676&id=20877" rel="noreferrer">https://phabricator.kde.org/D8211?vs=20676&id=20877</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D8211" rel="noreferrer">https://phabricator.kde.org/D8211</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>kdevplatform/documentation/CMakeLists.txt<br />
kdevplatform/documentation/standarddocumentationview.cpp<br />
kdevplatform/documentation/standarddocumentationview.h<br />
kdevplatform/documentation/standarddocumentationview_p.h<br />
kdevplatform/documentation/standarddocumentationview_qtb.cpp<br />
plugins/qthelp/CMakeLists.txt<br />
plugins/qthelp/qthelpdocumentation.cpp<br />
plugins/qthelp/qthelpdocumentation.h</div></div></div><br /><div><strong>To: </strong>rjvbb, KDevelop<br /><strong>Cc: </strong>aaronpuchert, flherne, arichardson, apol, kdevelop-devel, geetamc, Pilzschaf, akshaydeo, surgenight, arrowdodger<br /></div>