Bug 6368 - Add ability to set window type (desktop, dock) via app settings
Status: NEW
Alias: None
Product: Openbox
Classification: Unclassified
Component: general
Version: git-master
Hardware: PC Linux
: P3 enhancement
Assignee: Dana Jansens
QA Contact:
URL:
Depends on:
Blocks:
 
Reported: 2015-03-06 04:24 EST by Paul G
Modified: 2015-03-23 07:10:33 EDT
1 user (show)

See Also:


Attachments
Diff against mikachu's master (10.23 KB, patch)
2015-03-06 04:24 EST, Paul G
Correct diff against mikachu's master (8.79 KB, patch)
2015-03-06 04:28 EST, Paul G
Support setting window type to desktop or dock internally (4.05 KB, patch)
2015-03-23 07:08 EDT, Paul G

Description Paul G 2015-03-06 04:24:36 EST
Created attachment 3513 [details]
Diff against mikachu's master

This enhancement allows you to use an application as a desktop or dock even if it was not designed for this, eg using a terminal as a desktop/transparent wallpaper. See: http://article.gmane.org/gmane.comp.window-managers.openbox/6893

Patch adds:
  * <set_window_prop><type>(desktop,dock)</type></set_window_prop> application setting
  * comments in default rc.xml to this effect
  * additional debugging info for app matches (incidental, can be removed if undesirable)

Testing:
  * Window type set to _DESKTOP: tested, works as expected
  * Window type set to _DOCK: tested, matches what I know of expected behaviour for docks (eg always above); behaviour unknown to me may not work
Comment 1 Paul G 2015-03-06 04:28:19 EST
Created attachment 3514 [details]
Correct diff against mikachu's master

Oops, wrong diff last time,fixed
Comment 2 Paul G 2015-03-23 07:08:33 EDT
Created attachment 3528 [details]
Support setting window type to desktop or dock internally

Updated patch as per dana's comments:
 * Removed all references to _OVERRIDE
 * Backed out everything that changed _NET_WM_WINDOW_TYPE, we're only setting window type in internal state now
Comment 3 Paul G 2015-03-23 07:10:33 EDT
(In reply to Paul G from comment #2)
> Created attachment 3528 [details]
> Support setting window type to desktop or dock internally
> 
> Updated patch as per dana's comments:
>  * Removed all references to _OVERRIDE
>  * Backed out everything that changed _NET_WM_WINDOW_TYPE, we're only
> setting window type in internal state now

* Backed out app matching debug info, since it's not relevant to this particular patch.