On 7/13/21 2:04 PM, Hiltjo Posthuma wrote:
> On Mon, Jul 12, 2021 at 11:44:16PM +0200, Quentin Rameau wrote:
>> Some people are annoyed to have this new behaviour forced for some
>> application which use fake fullscreen.
>> ---
>> config.def.h | 1 +
>> dwm.c | 2 +-
>> 2 files changed, 2 insertions(+), 1 deletion(-)
>> ...
>
> For the archives: this was discussed on IRC (so not ignored or anything).
>
> If there are more people having issues with the behaviour introduced in commit
> 67d76bdc68102df976177de351f65329d8683064 then I'll gladly hear it.
>
I am the "some people" that Quentin mentioned above :)
I brought this up in the #suckless channel yesterday, when I was having
a problem with Firefox: When I entered into fullscreen mode, I could no
longer switch windows with my Alt+J/K keybindings. I use the
fakefullscreen patch, so for me fullscreen windows just expand as much
as possible.
Quentin (quinq), Ingvix and I discussed this on IRC. To me, this
behavior seems rather unintuitive and therefore should not be included
in mainline. To my knowledge, it isn't documented anywhere except the
commit message, which is:
> Do not allow focus to drift from fullscreen client via focusstack()
>
> It generally doesn't make much sense to allow focusstack() to navigate
> away from the selected fullscreen client, as you can't even see which
> client you're selecting behind it.
>
> I have had this up for a while on the wiki as a separate patch[0], but
> it seems reasonable to avoid this behaviour in dwm mainline, since I'm
> struggling to think of any reason to navigate away from a fullscreen
> client other than a mistake.
>
> 0: https://dwm.suckless.org/patches/alwaysfullscreen/
>
It may be that people aren't likely to want to switch windows when they
have a fullscreen window open -- again, assuming that they aren't using
fakefullscreen. But to assume that by default? To assume that the user
*must* have either pressed Alt+J/K by mistake? In what other context is
the user's inputs explicitly ignored?
The scenario mentioned by Ingvix on IRC was that it was annoying to
switch from a window they had forgotten was in fullscreen, and then have
to navigate back to their fullscreen window without seeing what windows
they have selected. Even if this wasn't easily remedied by just pressing
the opposite direction navigation keybinding, in my opinion this is the
sort of user error that should be addressed by a patch, not in dwm by
default. It seems a core tenet of suckless's development is to assume
that the user knows what they are doing. This behavior contradicts that.
Anyway, it's been more than a year since 67d76bd was merged, so I
understand if there is hesitance to revert it outright. I think that
this patch Quentin submitted is a good compromise. If merged, I will
update fakefullscreen to set the new lockfullscreen variable to 0.
---
Sebastian LaVine | https://smlavine.com
Received on Tue Jul 13 2021 - 20:33:31 CEST