fthx / babar

Task bar. GNOME Shell extension
GNU General Public License v3.0
41 stars 14 forks source link

Babar thinks "DING" is a window #8

Closed bernardelli closed 3 years ago

bernardelli commented 3 years ago

Firstly, thank you for the simple task bar project and this current remake. I love saving screen space.

I use a extension for better desktop icons, DING. Babar recognizes it as a window at the sticky workspace. Clicking on the right gear below shows/hides the desktop icons.

image

I am not sure, though, where the left gear comes from. I will try to find out later.

bernardelli commented 3 years ago

Just found out that the first gear only appears when I am using two monitors. But I'm still not sure what's going on.

fthx commented 3 years ago

Thanks for reporting. I look at this now, I think that checking more carefully the windows that have to be shown as icons will solve this. I hope that Looking Glass will tell me what type is DING window.

FYI: The gear is the default fallback icon, if window app does not have one. It's useful too when GS create a window object but its properties are not yet accessible... Would you need a setting for this fallback icon or you don't care?

fthx commented 3 years ago

Ok, done. You can find the commit here: https://github.com/fthx/babar/commit/d3b874ef8d9c81f1f8c479855cceaf4eeb3d3148 You can also download the latest version here, before reviewing by GNOME team: https://extensions.gnome.org/review/download/22315.shell-extension.zip

I just added DESKTOP WindowType in blacklist. And it worked, as far as I can test. Feel free to reopen if it did not solve your issue.

fthx commented 3 years ago

Ok, I saw that you changed icon size and that the gear did not follow that: https://github.com/fthx/babar/commit/bd6033277612e3174e0cadd7d3ccab2bcebdc80b You can find the corrected extension here: https://extensions.gnome.org/review/download/22318.shell-extension.zip

bernardelli commented 3 years ago

The problem is gone! 🎉

fthx commented 3 years ago

FYI: https://github.com/fthx/babar/issues/6#issuecomment-781943033 New options: hide dash & ww thumbnails, buttons for moving window to another ws. I don't know how to make drag'n'drop, so it's better than nothing. :-)

patarapolw commented 3 years ago

Can I choose what to hide?

Currently, I need not only DING, but also Conky.

fthx commented 3 years ago

What is precisely your issue? Conky is displayed as a BaBar icon? If Conky is displayed as a NORMAL typed window, I cannot do anything and this should be considered as a Conky bug.

patarapolw commented 3 years ago

Screenshot from 2021-05-20 16-08-37

I am only opening Brave browser.

First icon is Conky. It's in Applications also. Third icon is DING; which I didn't use before I tried today...

And DING moves when I change workspace.

Forgot to mention. I am on GNOME 40.

fthx commented 3 years ago

Ok, so something has changed in DING too. I try that in my Fedora 34 VM and I'll be back.

Could you check the window type of Conky (using lg) ?

patarapolw commented 3 years ago

own_window_type = 'normal',

What is lg?

fthx commented 3 years ago

Looking Glass of GNOME Alt+F2, lg

fthx commented 3 years ago

image

use the pipette

fthx commented 3 years ago

Ok, DING is a NORMAL window (value: 0), so I cannot do anything. Maybe a hack, I'll see.

patarapolw commented 3 years ago

DING disappears in Xorg, but appear in Wayland.

Screenshot from 2021-05-20 16-25-47 Screenshot from 2021-05-20 16-28-52

I am currently in Xorg.

fthx commented 3 years ago

Well, it will be hard, because the DING window name is: image

fthx commented 3 years ago

So, I think DING has to revert to its DESKTOP typed window. What's the name of conky's window?

fthx commented 3 years ago

Does DING window appear in GNOME Dash?

patarapolw commented 3 years ago

Both GNOME and Conky, no, I think. (In Wayland)

Screenshot from 2021-05-20 16-41-05

I still hope for a Quick Fix, in extension.js, maybe.

Conky title is conky (<MY_LAPTOP_NAME>)

fthx commented 3 years ago

Hmm, maybe I can investigate on some window properties (fullscreen but not minimizable e.g.) to filter displayed BaBar icons.

patarapolw commented 3 years ago

And, it tried connect a second screen, DING duplicates, with some odd, but different title.

fthx commented 3 years ago

Could you check that for DING and Conky windows? : image

patarapolw commented 3 years ago

Screenshot from 2021-05-20 17-23-05

Top is Conky. The bottom is DING, I think.

BTW, temporary solution.

    // create window button ; ws = workspace, w = window
    _create_window_button(ws_index, w) {
        // windows on all workspaces have to be displayed only once
        if (!w.is_on_all_workspaces() || ws_index == 0) {
            if (/^conky \(.+\)$/.test(w.title)) {
                return
            }

            if (/^@!\d+,\d+;BDH$/.test(w.title)) {
                return
            }
fthx commented 3 years ago

Nope, I'll include a is_skip_taskbar filter. I'll check that very soon.

fthx commented 3 years ago

https://github.com/fthx/babar/commit/5445219a0fb0c5140a55afb9f4878cc7ef968704

Feel free to reopen if you still experience the issue.

fthx commented 3 years ago

It's online now, you should get a GNOME notification to update BaBar soon.