SavingGoogleCode / tint2

tint2 is a lightweight panel/taskbar.
GNU General Public License v2.0
0 stars 0 forks source link

Allow setting the window title (or role) #460

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
pekwm (and others) matches windows by WM_CLASS, WM_WINDOW_ROLE, and WM_NAME.  
If running multiple copies of tint2 there is no way to be selective about which 
one is being acted upon by specific window manager policies.

Allowing within the config to change the window title or role would allow this 
selectivity.

Original issue reported on code.google.com by lstr...@gmail.com on 30 Jan 2015 at 7:49

GoogleCodeExporter commented 9 years ago

Original comment by goo...@craigoakes.com on 31 Jan 2015 at 4:15

GoogleCodeExporter commented 9 years ago

Original comment by mrovi9...@gmail.com on 1 Feb 2015 at 5:56

GoogleCodeExporter commented 9 years ago

Original comment by mrovi9...@gmail.com on 1 Feb 2015 at 11:48

GoogleCodeExporter commented 9 years ago

Original comment by goo...@craigoakes.com on 2 Feb 2015 at 4:17

GoogleCodeExporter commented 9 years ago
r690 looks good, but there will be a small memory leak when parsing the config 
file the second time, due to the strdup followed by no free in cleanup_panel().

On the other hand we must be careful not to free the static string "tint2" set 
in default_panel().

I suggest that in default_panel() we set panel_window_name = strdup("tint2"); 
free it in config before overwriting, and finally also free it in 
cleanup_panel().

Original comment by mrovi9...@gmail.com on 8 Feb 2015 at 6:48

GoogleCodeExporter commented 9 years ago
Issue 469 has been merged into this issue.

Original comment by mrovi9...@gmail.com on 9 Feb 2015 at 7:57

GoogleCodeExporter commented 9 years ago

Original comment by goo...@craigoakes.com on 9 Feb 2015 at 3:06