From f2bd697d64a13233889b8d0b6aa9cc716e14ab60 Mon Sep 17 00:00:00 2001 From: lethosor Date: Mon, 13 May 2019 19:21:57 -0400 Subject: [PATCH] mousequery: give explicit feedback when enabling/disabling sub-features The behavior of this plugin is somewhat unintuitive - "mousequery edge" disables the edge-scrolling feature instead of enabling it. This should avoid confusion without breaking compatibility with existing init scripts. --- plugins/mousequery.cpp | 13 +++++++++++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/plugins/mousequery.cpp b/plugins/mousequery.cpp index d9bba78b6..5aab0b584 100644 --- a/plugins/mousequery.cpp +++ b/plugins/mousequery.cpp @@ -836,26 +836,35 @@ static command_result mousequery_cmd(color_ostream &out, vector & param else if (cmd[0] == 'p') { plugin_enabled = (state == "enable"); + out << "mousequery: plugin " << (plugin_enabled ? "enabled" : "disabled") << endl; } else if (cmd[0] == 'r') { rbutton_enabled = (state == "enable"); + out << "mousequery: rbutton " << (rbutton_enabled ? "enabled" : "disabled") << endl; } else if (cmd[0] == 't') { tracking_enabled = (state == "enable"); - if (!tracking_enabled) + if (!tracking_enabled) { + out << "mousequery: edge scrolling disabled" << endl; active_scrolling = false; + } + out << "mousequery: tracking " << (tracking_enabled ? "enabled" : "disabled") << endl; } else if (cmd[0] == 'e') { active_scrolling = (state == "enable"); - if (active_scrolling) + if (active_scrolling) { + out << "mousequery: tracking enabled" << endl; tracking_enabled = true; + } + out << "mousequery: edge scrolling " << (active_scrolling ? "enabled" : "disabled") << endl; } else if (cmd[0] == 'l') { live_view = (state == "enable"); + out << "mousequery: live view " << (live_view ? "enabled" : "disabled") << endl; } else if (cmd == "drag") {