path: root/config
diff options
authorMike Blumenkrantz <>2013-10-07 06:49:33 +0100
committerMike Blumenkrantz <>2013-10-07 06:49:33 +0100
commitf1a65ee922c5f937cd9ab9318c5142e2e4dfccef (patch)
treedd469f43702308ff9474951c12aa0558c2e71812 /config
parent3c343db3494f7cbd77013ed993cea8eb7485350b (diff)
remove "border_raise_on_focus" config option
buckle up. for the first time in history, a config option is getting removed instead of added. the reasons for this removal are many, but let's go way back to the beginning and see why it was added: oh wait, we can't because the commit message (from 2006) is >> patches that i said were in - commit. (see my reply emails) >> also finish off a TODO item or 2 reading through the TODO items which were also crossed off in that commit, I'm assuming that this was the "option to NOT raise on focus in click to focus" item. == REASON 1 == the problem here is that there's another, BETTER option called "click raises window" (always_click_to_raise) which does the same thing, except it doesn't totally fuck you when you get a random X focus event, which happens more often than you might think. this means that, to avoid broken behavior which might cause your windows to spastically raise for a few frames in common cases (using winlist...) with click-to-focus, you have to know that this is the default-enabled option that's fucking you, and you have to remember to manually disable it every time. if you DON'T know that this is the option that's fucking you, and you just see windows randomly raising on their own, you'll probably either ignore it or file a bug, when this is supposed to be a "feature" that actually worked in reverse, since it was intended only for disabling. == REASON 2 == there's also auto-raise, which can be set to 0.0s, which is effectively the same thing since it also triggers on focus but can be configured not to fuck your window stack == REASON 3 == aaand finally, this option makes any sort of pointer focus model impossible to use, since your windows will constantly be raising all over as you move the mouse tl;dr: I'm removing it, e-dealwithit.gif
Diffstat (limited to 'config')
3 files changed, 0 insertions, 3 deletions
diff --git a/config/default/e.src b/config/default/e.src
index 43c91ec2a..458f643c1 100644
--- a/config/default/e.src
+++ b/config/default/e.src
@@ -160,7 +160,6 @@ group "E_Config" struct {
160 value "mouse_accel_denominator" int: 1; 160 value "mouse_accel_denominator" int: 1;
161 value "mouse_accel_threshold" int: 4; 161 value "mouse_accel_threshold" int: 4;
162 value "border_raise_on_mouse_action" int: 1; 162 value "border_raise_on_mouse_action" int: 1;
163 value "border_raise_on_focus" int: 1;
164 value "desk_flip_wrap" int: 0; 163 value "desk_flip_wrap" int: 0;
165 value "fullscreen_flip" int: 1; 164 value "fullscreen_flip" int: 1;
166 value "icon_theme" string: "Tango"; 165 value "icon_theme" string: "Tango";
diff --git a/config/mobile/e.src b/config/mobile/e.src
index ff0302487..a60c51705 100644
--- a/config/mobile/e.src
+++ b/config/mobile/e.src
@@ -148,7 +148,6 @@ group "E_Config" struct {
148 value "mouse_accel_denominator" int: 1; 148 value "mouse_accel_denominator" int: 1;
149 value "mouse_accel_threshold" int: 4; 149 value "mouse_accel_threshold" int: 4;
150 value "border_raise_on_mouse_action" int: 1; 150 value "border_raise_on_mouse_action" int: 1;
151 value "border_raise_on_focus" int: 1;
152 value "desk_flip_wrap" int: 0; 151 value "desk_flip_wrap" int: 0;
153 value "fullscreen_flip" int: 1; 152 value "fullscreen_flip" int: 1;
154 value "icon_theme" string: "Tango"; 153 value "icon_theme" string: "Tango";
diff --git a/config/standard/e.src b/config/standard/e.src
index f6d6e38fd..d834c8a28 100644
--- a/config/standard/e.src
+++ b/config/standard/e.src
@@ -149,7 +149,6 @@ group "E_Config" struct {
149 value "mouse_accel_denominator" int: 1; 149 value "mouse_accel_denominator" int: 1;
150 value "mouse_accel_threshold" int: 4; 150 value "mouse_accel_threshold" int: 4;
151 value "border_raise_on_mouse_action" int: 1; 151 value "border_raise_on_mouse_action" int: 1;
152 value "border_raise_on_focus" int: 0;
153 value "desk_flip_wrap" int: 0; 152 value "desk_flip_wrap" int: 0;
154 value "fullscreen_flip" int: 1; 153 value "fullscreen_flip" int: 1;
155 value "icon_theme" string: "Tango"; 154 value "icon_theme" string: "Tango";