openbox icon indicating copy to clipboard operation
openbox copied to clipboard

Allow adding/removing desktops by index

Open sidequestboy opened this issue 12 years ago • 5 comments
trafficstars

See Bugzilla 5974 The commit message describes the functionality added.

Keys "last" and "current" should work exactly the same as before.

sidequestboy avatar Aug 15 '13 20:08 sidequestboy

Gah, I force-updated this commit, and it erased all of your code review. Edit: actually, it still exists here sneaky Github

Anyway, this re-commit addresses the issues you raised (namely styling and behaviour of adding with "current" keyword). That behaviour is accomplished by a parameter gboolean follow in screen_add_desktop.

sidequestboy avatar Aug 31 '13 01:08 sidequestboy

This is actually buggy with raising windows on desktop remove right now - don't bother looking over it yet.

sidequestboy avatar Aug 31 '13 23:08 sidequestboy

Ok, ping me again when it's ready :) Thanks!

danakj avatar Sep 01 '13 15:09 danakj

This code works for me, including the old "current" behaviour. Take a look at your leisure.

sidequestboy avatar Sep 24 '13 14:09 sidequestboy

ping

sidequestboy avatar Feb 14 '14 03:02 sidequestboy