09:47 <leon-p> do we really re-layout for every focus change? That seems a bit excessive

10:06 <LarstiQ> I suppose you might have a lens-zoom type layout bringing the focused window to the front/center?

10:07 <LarstiQ> though currently that requires some state keeping/querying of rivercontrol that's tedious

11:32 <LarstiQ> scheme has identifiers with spaces in their names!? oho

11:57 <leon-p> I don't think it does?

11:59 <LarstiQ> seems an r7rs-ism

12:00 <LarstiQ> https://standards.scheme.org/official/r7rs.pdf 2.1, "Alternatively, an identifier can be represented by a se- quence of zero or more characters enclosed within vertical lines (|), analogous to string literals. Any character, in- cluding whitespace characters, but excluding the backslash and vertical line characters, can appear verbatim in such an identifier."

12:00 <LarstiQ> I've switched to reading r5rs instead

12:43 <waleee> you can go wild with identifiers in zig in a similar vein, @"anything goes inside here"

16:29 <ifreund> leon-p: yes, re-layouting every focus change is not necessary but it make the code simpler

16:29 <ifreund> there's a nice approach to this that clay uses which I mean to steal for river at some point

16:31 <ifreund> https://git.sr.ht/~kennylevinsen/clay/tree/e63b331c20a22e7634741aa0d9f5f7248a6242bb/item/src/clay/clay.c#L68-95

22:00 <tiosgz> ifreund: fyi github has a "sign in to view" policy on hidden comments, which makes it impossible for me to catch up

22:01 <tiosgz> it's okay now when you need it organised, but if it isn't a problem, i'd welcome if you uncollaped them once everything is done

22:02 <ifreund> oh really? that's bs

22:02 <ifreund> tiosgz: in any case, the latest string of comments were just me fixing output-management related bugs without having more then one output available to test

22:03 <tiosgz> ah, thanks :)

22:03 <ifreund> the only remaining blocker is debugging

22:03 <ifreund> er, debugging novakane's issue

22:05 <ifreund> simon already opened a wlroots MR to fix the crash with refresh rate 0, I just need to review/test it

23:23 <parsnip64> hey :) does anybody else who uses scaling experience large gtk interfaces? firefox is normal size but the save window is huge

23:48 <szgy> Hi y'all, I'm having issues with mapping the PrintScreen key of my keyboard.

23:49 <szgy> The issue I'm having is that I'm using grim (to take the screenshot) with slurp (to select the screen region)

23:49 <szgy> but since slurp is used as a command substitution thingy its run when I set the mapping with riverctl instead when I actually press the key...

23:49 <NickH> Have you confirmed the problem is with the binding? Eg test the command independently to the binding.

23:50 <szgy> yeah yeah. That works just fine

23:51 <NickH> Perhaps pastebin the relevant part of your ~/.config/river/init

23:51 <szgy> its `riverctl map Normal None Print spawn "grim -g $(slurp)"`

23:52 <NickH> Use single quotes instead of double

23:52 <szgy> that did it!

23:52 <szgy> thanks!

23:53 <NickH> FYI, with what you orignally had, slurp was being run when the init file was executed, not when then binding was run.

23:54 <NickH> s/then/the

23:54 <szgy> yeah yeah, I was aware of that, didn't know that bash sort of escapes command subtitution when between single quotes

23:55 <NickH> Ahh, I see now you knew what was happening, just didn't know how to fix it.

23:56 <szgy> mhH. not entirely solved the problem. I mean, that is solved, but it doesn't seem to take the screenshot

23:57 <szgy> also tried with `riverctl spawn 'grim -g $(slurp)'`

23:57 <szgy> slurp is run fine, but no screenshot is taken...