nilix/openbox
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix per-app monitor setting

Browse source 
A couple of things were wrong, the parser added 1 to the value despite
expecting the user to give values in the range of 1 to
screen_num_monitors, rc.xml documented the values to start from 0 and
finally the monitor value wasn't copied over at all when matching the
client.
This commit is contained in:
Mikael Magnusson 2009-02-20 17:41:34 +01:00
parent d0829d4416
commit 271e0fd9c8
2 changed files with 3 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
data/rc.xml
View file

@ -682,7 +682,7 @@
<y>200</y>
<monitor>1</monitor>
# specifies the monitor in a xinerama setup.
# 0 is the first head, or 'mouse' for wherever the mouse is
# 1 is the first head, or 'mouse' for wherever the mouse is
</position>
<focus>yes</focus>

3
openbox/config.c
View file

@ -128,6 +128,7 @@ void config_app_settings_copy_non_defaults(const ObAppSettings *src,
copy_if(type, (ObClientType)-1);
copy_if(decor, -1);
copy_if(shade, -1);
copy_if(monitor, -1);
copy_if(focus, -1);
copy_if(desktop, 0);
copy_if(layer, -2);
@ -267,7 +268,7 @@ static void parse_per_app_settings(ObParseInst *inst, xmlDocPtr doc,
if (!g_ascii_strcasecmp(s, "mouse"))
settings->monitor = 0;
else
settings->monitor = parse_int(doc, c) + 1;
settings->monitor = parse_int(doc, c);
g_free(s);
}