Re: [PATCH v3] xwm: Choose icon closest to target size

2018-03-23 Thread Scott Moreau
Hi Derek, On Fri, Mar 23, 2018 at 11:34 AM, Derek Foreman wrote: > Thanks for this. This particular feature is getting on my nerves > because it seemed like a simple thing and has result in a lot of breakage. > > Would like to see it repaired or removed by late next

Re: [PATCH v3] xwm: Choose icon closest to target size

2018-03-23 Thread Derek Foreman
Thanks for this. This particular feature is getting on my nerves because it seemed like a simple thing and has result in a lot of breakage. Would like to see it repaired or removed by late next week. This seems to be a good step towards repair, comments inline below. On 2018-03-22 12:47 AM,

[PATCH v3] xwm: Choose icon closest to target size

2018-03-21 Thread Scott Moreau
Xwayland clients can offer multiple icon sizes in no particular order. Previously xwm was selecting the first one unconditionally. This patch selects the icon that matches the size closest to the target size. The target size is hard coded to 16 since there is only one theme and the data used to