feat: Firefox picture-in-picture should always float

[nvergottini]

Is your feature request related to a problem? Please describe.
Forge tiles Firefox picture-in-picture like a regular window, which defeats the purpose of PIP. You can toggle float mode with the keyboard shortcut, but that is awkward and shouldn't be necessary. Setting always float affects all Firefox Windows, so that isn't an option.

Describe the solution you'd like
Make Firefox PIP always float.

Describe alternatives you've considered
I was using Pop Shell, which always allows Firefox PIP to float, but it isn't currently supported on Gnome 45. I was looking at Forge as a permanent alternative to Pop Shell since Pop Shell requires an overlay install on Fedora Silverblue.

Additional context
None

[p1gp1g]

Pop shell can uses regex to match wmTitle and it uses this regex to apply the floating exception to all firefox windows except the main one

[jmmaranan]

Forge uses a very simple .include() function. And I think adding wmTitle: Mozilla Firefox, wmClass: firefox as a PR should fix it.

[p1gp1g]

But that's the opposite that is looking for, all the windows except the ones with "Mozilla Firefox" in its name should float. And that is not a full match, only those which end with this.

[jmmaranan]

The windows.json entries in Forge, floats the windows that matches wmClass and wmTitle along with mode: float.

[p1gp1g]

Do you mean that it is possible to do something like this ?

{"wmClass": "firefox", "mode": "float"},
{"wmClass": "firefox", "wmTitle":"Mozilla Firefox", "mode": "tile"},

[jmmaranan]

This worked for me:

{ "wmClass": "firefox", "wmTitle": "About Mozilla Firefox", "mode": "float" },
{ "wmClass": "firefox", "wmTitle": "Picture-in-Picture", "mode": "float" }

[p1gp1g]

I see. Maybe a way to do a not would be useful. For instance titles starting with ! to do something like

{"wmClass": "firefox", "wmTitle":"!Mozilla Firefox", "mode": "float"},

[jmmaranan]

Yes, adding the negate syntax should be simple. But to solve this current issue, need to update the default config

[jmmaranan]

Hi @nvergottini - I know this has taken long but do you know if you can try the PR branch or do I have to ask to publish to extensions site, first for you to test?
If you do not mind @p1gp1g , can you also please confirm that the one/two entries does fix it?

[nvergottini]

Hi @nvergottini - I know this has taken long but do you know if you can try the PR branch or do I have to ask to publish to extensions site, first for you to test? If you do not mind @p1gp1g , can you also please confirm that the one/two entries does fix it?

Unfortunately, I cannot test at the moment. I ended up reinstalling my laptop with Debian 12 for unrelated reasons. I tried installing the fix-firefox-pip branch, but it shows as outdated, so I cannot enable. I'm guessing because Debian 12 is using Gnome 43. I have Silverblue on my desktop, but I'm on vacation until next week, so I don't have access to it.

[p1gp1g]

The following rules works if firefox is in english. Unfortunately, these rules should be written for all available language. It would be way more efficient to introduce the not for this issue too. * (And some other app have too many different titles for non-main window)

{"wmClass": "firefox","wmTitle": "About Mozilla Firefox","mode": "float"},
{"wmClass": "firefox","wmTitle": "Picture-in-Picture","mode": "float"},
{"wmClass": "firefox","wmTitle": "Page Info","mode": "float"}

* I can open a PR for this.

BTW, is it possible to reload the config file without logging out/logging in ?