Re: [PATCH] xdg-shell: add set_max_size request

2016-04-05 Thread The Rasterman
On Mon, 4 Apr 2016 12:05:05 -0700 Bill Spitzak said: > I do not like clients having to update this value continuously as > conditions change. I would prefer any kind of design where the calculation > of the maximum size is deferred until it is actually used, ie at the point >

Re: [PATCH] xdg-shell: add set_max_size request

2016-04-05 Thread The Rasterman
On Mon, 4 Apr 2016 22:40:10 -0700 Bill Spitzak said: > I think sending stepping size or aspect is not needed, but steps will > work only if the client can add a constant. Ie the width can be n*A+B > where A and B are specified by the client. The X11 version did not allow >

Re: [PATCH] xdg-shell: add set_max_size request

2016-04-05 Thread Bill Spitzak
I do not like clients having to update this value continuously as conditions change. I would prefer any kind of design where the calculation of the maximum size is deferred until it is actually used, ie at the point that the user does whatever action attemts to make the window larger. Perhaps an

Re: [PATCH] xdg-shell: add set_max_size request

2016-04-05 Thread Bill Spitzak
I think sending stepping size or aspect is not needed, but steps will work only if the client can add a constant. Ie the width can be n*A+B where A and B are specified by the client. The X11 version did not allow a client to add a border that was not a multiple of the steps thick, which made

Re: [PATCH] xdg-shell: add set_max_size request

2016-04-05 Thread Olivier Fourdan
Hi Jasper, - Original Message - > "hint" just means "don't bother setting this flag, since it won't do > anything". > > If we want min/max size rules, I want them to be enforced. Compositors > SHOULD NOT attempt to configure windows above or below the requested > size. Right, so it gets

Re: [PATCH] xdg-shell: add set_max_size request

2016-04-05 Thread Jasper St. Pierre
"hint" just means "don't bother setting this flag, since it won't do anything". If we want min/max size rules, I want them to be enforced. Compositors SHOULD NOT attempt to configure windows above or below the requested size. On Mon, Apr 4, 2016 at 11:58 PM, Olivier Fourdan

Re: [PATCH] xdg-shell: add set_max_size request

2016-04-05 Thread Olivier Fourdan
Hi all, - Original Message - > On Mon, 04 Apr 2016 19:44:58 + "Jasper St. Pierre" > > said: > > > I think min/max hints are acceptable in xdg-shell. > > i agree. they are realistic things a apps have as constraints on their > content. > knowing in advance

Re: [PATCH] xdg-shell: add set_max_size request

2016-04-04 Thread The Rasterman
On Mon, 04 Apr 2016 19:44:58 + "Jasper St. Pierre" said: > I think min/max hints are acceptable in xdg-shell. i agree. they are realistic things a apps have as constraints on their content. knowing in advance what those constraints might be can make life for a

Re: [PATCH] xdg-shell: add set_max_size request

2016-04-04 Thread Jasper St. Pierre
I think min/max hints are acceptable in xdg-shell. On Mon, Apr 4, 2016, 12:33 PM Mike Blumenkrantz < michael.blumenkra...@gmail.com> wrote: > On Mon, Apr 4, 2016 at 3:30 PM Olivier Fourdan > wrote: > >> >> Hi Mike, >> >> - Original Message - >> > [...] >> > >> >

Re: [PATCH] xdg-shell: add set_max_size request

2016-04-04 Thread Mike Blumenkrantz
On Mon, Apr 4, 2016 at 3:30 PM Olivier Fourdan wrote: > > Hi Mike, > > - Original Message - > > [...] > > > > Yes, I know you are not currently advocating for it, but you've proved my > > point--others will see this go in and then they will push for it. Adding > >

Re: [PATCH] xdg-shell: add set_max_size request

2016-04-04 Thread Olivier Fourdan
Hi Mike, - Original Message - > [...] > > Yes, I know you are not currently advocating for it, but you've proved my > point--others will see this go in and then they will push for it. Adding > any form of size hints is a slipper slope which leads to more size hints > imo. My turn to

Re: [PATCH] xdg-shell: add set_max_size request

2016-04-04 Thread Mike Blumenkrantz
On Mon, Apr 4, 2016 at 1:17 PM Olivier Fourdan wrote: > Hi Mike, > > > Hm, you raise some interesting points. However, I think your argument is > > somewhat misled by your claim that "this case is unique". If there is an > > application which does not want to be larger than

Re: [PATCH] xdg-shell: add set_max_size request

2016-04-04 Thread Olivier Fourdan
Hi Mike, > Hm, you raise some interesting points. However, I think your argument is > somewhat misled by your claim that "this case is unique". If there is an > application which does not want to be larger than a certain size, why could > there not also be an application which does not want to be

Re: [PATCH] xdg-shell: add set_max_size request

2016-04-04 Thread Mike Blumenkrantz
Hm, you raise some interesting points. However, I think your argument is somewhat misled by your claim that "this case is unique". If there is an application which does not want to be larger than a certain size, why could there not also be an application which does not want to be smaller than a

Re: [PATCH] xdg-shell: add set_max_size request

2016-04-04 Thread Olivier Fourdan
Hi Mike, Thanks for starting the discussion! :) - Original Message - > Just to play devil's advocate, do we really want to start getting into size > hint type stuff? After this will be min size, then step size, then aspect, > ... I think these are different issues. min size, step size,

Re: [PATCH] xdg-shell: add set_max_size request

2016-04-04 Thread Mike Blumenkrantz
Just to play devil's advocate, do we really want to start getting into size hint type stuff? After this will be min size, then step size, then aspect, ... On Mon, Apr 4, 2016 at 5:14 AM Olivier Fourdan wrote: > Some application may wish to restrict their window in size, but

[PATCH] xdg-shell: add set_max_size request

2016-04-04 Thread Olivier Fourdan
Some application may wish to restrict their window in size, but xdg-shell has no mecanism for the client to advertise such a maximimum size. As a result, the compositor may try to maximize or fullscreen a window while the client would not allow for the requested size. Add a new request