croick created this revision. croick added reviewers: Plasma: Workspaces, apol, mwolff. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. croick requested review of this revision.
REVISION SUMMARY - independently of the config setting in drkonqirc propose debugging with KDevelop if a session is available - default debugger is only displayed if ShowDebugButton=true in [DrKonqi] - button is the first action button to be displayed - fix adding/removing of the button to the button box TEST PLAN - set ShowDebugButton=false, have no KDevelop session running - force a crash (using src/tests/crashtest for instance) - DrKonqi shows no debug button - start a KDevelop session - the button appears with the KDevelop session selectable - quit DrKonqi, set ShowDebugButton=true - crash again, the button now always is visible, including the default debugger as a selection REPOSITORY R871 DrKonqi BRANCH nicebutton REVISION DETAIL https://phabricator.kde.org/D11234 AFFECTED FILES src/drkonqidialog.cpp To: croick, #plasma_workspaces, apol, mwolff Cc: plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart