06:49 <novakane> it seems that river doesn't send the zriver_output_status urgent_tags event now, at least testing with foot, did I missed some changes?

08:04 <aktina> is smartborder functionality planned in rivercarro layout generator ? (yes, i know rivercarro-wrapper exists)

09:16 <novakane> aktina: nope, because changing border on the layout generator side is a bit hacky and annoying to do currently

09:17 <novakane> what's rivercarro-wrapper?

12:27 <aktina> it is on riverwm's wiki, community layouts, and listed in it is rivercarro-wrapper, a hacky zsh script (that is how it is mentioned) to implement few stuffs like smartborders and stuff

12:29 <aktina> i would have copied it, but i cannot be sure that rivercarro might have the same commands in future, and if this wrapper too would get updated if rivercarro changes a bit

12:39 <ifreund> wow, that is impressively hacky

12:40 <ifreund> novakane: perhaps I broke it by accident, not sure

13:36 <novakane> aktina: huh yeah that's even more hacky than using river-control protocol, but yeah rivercarro is features complete at least till a new layout protocol/design

13:37 <novakane> ifreund: I hadn't the time to look more at it, but if someone can confirm I'm not the only one having this problem I'll look more into it

15:02 <aktina> then, i'll keep using vanilla rivercarro, and decide later if smart border is really necessary

16:52 <novakane> seems like this if statement is never true https://github.com/riverwm/river/blob/master/river/OutputStatus.zig#L146

16:52 <novakane> so I guess the view is never set as urgent, need to find why now

17:38 <ifreund> novakane: I confirmed the issue, looking into a fix

17:38 <ifreund> I think idle inhibit is broken as well by the same bug

17:40 <leon-p> yes it is, would have reported it, but TBH I forgot we even had it implemented in the first place and thought it was just missing :D

17:47 <ifreund> looks like I broke it in 05eac54b0

17:53 <ifreund> and should be fixed now, thanks novakane!

18:01 <novakane> ifreund: yeah it's fixed, cool thanks!

18:03 <novakane> leon-p: I didn't even saw it was broken, till I saw I couldn't make it works for the tags in zelbar, and I'm the one who implemented it lol