fthx / babar

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

Keyboard layout window not listed #18

Closed wooque closed 3 years ago

wooque commented 3 years ago

When you click on "Show Keyboard Layout" in layout switcher widget it opens up new window that is not shown in list.

wooque commented 3 years ago

I investigated and window type is DIALOG, so maybe add it to whitelist also.

fthx commented 3 years ago

Thanks. I just was starting to execute lg to pick window type! Could you add DIALOG to whitelist? I made this whitelist to ease adding some types, so that's the moment to do that. ;-)

If you don't experience some downsides after a few days, we could add this type. I will test this too.

fthx commented 3 years ago

Please note that I usually do not define many layouts, so I never saw this... And it's intended that you cannot minimize the layout dialog: I test if the window can be minimized before doing this.

_on_button_press(widget, event, w_box, ws_index, w) {
        // left-click: toggle window
        if (event.get_button() == 1) {
            if (w.has_focus() && !Main.overview.visible) {
                if (w.can_minimize()) {
                    w.minimize();
                }
            } else {    
                w.activate(global.get_current_time());
            }
            if (Main.overview.visible) {
                Main.overview.hide();
            }
            if (!w.is_on_all_workspaces()) {
                WM.get_workspace_by_index(ws_index).activate(global.get_current_time());
            }
        }
wooque commented 3 years ago

yes, I added DIALOG type and will report if any problems arise from that change

fthx commented 3 years ago

Can we close this?

wooque commented 3 years ago

well I didn't experience any problems from adding DIALOG to whitelist types, so I think it's safe to add

fthx commented 3 years ago

Ok, it will be included in the next upload. I don't make this now because we have some GS 40 CSS changes in discussion.