From: Arnout E. <no...@bz...> - 2011-10-20 23:44:20
|
On Thu, Oct 20, 2011 at 11:52:00PM +0100, David Given wrote: > On 20/10/11 22:31, Arnout Engelen wrote: > [...] > > To read client window properties from lua, use ioncore.x_get_window_property > > (http://notion.sourceforge.net/notionconf/node7.html#fn:ioncore.x_get_window_property) > > and check out the atom-related functions. > > Documentation! Thanks very much for the link --- I'd totally failed to > find that before. Not sure how to make it more prominent - did you search for it at all? where? > > To get your desired behavior of having all windows float by default, I'm not > > sure I know of an elegant way to do that right now. Indeed having all matching > > winprops 'stack' their options seems like a neat idea, but it might have quite > > a bit of impace - we should carefully consider it. > > In fact, I've just changed ioncore_winprop.lua to support this. It now > matches against all the defwinprop statements, in order, and accumulates > settings from each one. > > ...and it actually *works* --- windows all float by default, except > Chrome, which gets tiled. > > Of course, the stock Notion only has two defwinprop statements by > default, neither of which are active on my system, so it's hardly a good > test. But the patch is enclosed... Yes, this would indeed merge the winprops - thanks for diving in. Still, we should carefully consider a couple things: - Language power: I think we have established that this approach allows us to neatly express some configurations that we couldn't express with the old semantics. Can we also think of configurations that could be expressed in the old semantics but not in the new ones? Can we do anything about those? - When defining more and more rules, does the configuration remain readable (and thus maintainable)? Will unexpected side-effects lead to confusion? - Can we think of a way to introduce this without breaking the carefully- constructed winprop configurations of our current users (and all ion3 users)? Kind regards, Arnout |