bug: forge keyboard shortcuts interfere with gnome lock shortcut

nergdron commented 1 year ago

Description

the default Forge window switching keyboard shortcuts interfere with the builtin gnome shortcut to lock the screen.

Problem Explanation

pressing <Super> L, the builtin gnome shortcut to lock the screen, no longer works when forget is active. it seems likely this is because forge binds + I/J/K/L for window movement. however, changing those shortcuts to something else in the forget settings, such as <Shift><Super>L don't allow that shortcut to continue working either.

Expected Behavior

Forge extension shouldn't overwrite or break essential builtin gnome shortcuts without input from the user
Forge should provide non-conflicting shortcuts, or prompt the user if it wants to adopt the conflicting ones from forge or use the Gnome defaults

Reprodution

<Super> L with Forge active

It happens with any application?

[X] Yes, it applies to any application.

It happens only on one computer?

[X] No. I have tried it on more than one computer.

It happens only with some specific gnome configuration?

[X] I don't think that the configuration matters.

It happens only with some specific extension installed?

[X] I don't think that the installed extensions affect the bug/behavior.

Diagnostics

as long as forge is active

Fill in all information that applies:

Environment

Distro version : EndeavourOS rolling release
GNOME Shell version: 44.2
Forge source : extensions.gnome.org
Forge version : 70

Display Setup

Displays:

single 4k
Notebook 1920x1080 60hz

Extension Configuration

[/]
css-last-update=uint32 37
css-updated='1686711852801'
window-gap-hidden-on-single=false
window-gap-size=uint32 12
window-gap-size-increment=uint32 1

[keybindings]
con-split-horizontal=['<Super>z']
con-split-layout-toggle=['<Super>g']
con-split-vertical=['<Super>v']
con-stacked-layout-toggle=['<Shift><Super>s']
con-tabbed-layout-toggle=['<Shift><Super>t']
con-tabbed-showtab-decoration-toggle=['<Control><Alt>y']
focus-border-toggle=['<Super>x']
prefs-tiling-toggle=['<Super>w']
window-focus-down=['<Super>j']
window-focus-left=['<Super>h']
window-focus-right=['<Super>l']
window-focus-up=['<Super>k']
window-gap-size-decrease=['<Control><Super>minus']
window-gap-size-increase=['<Control><Super>plus']
window-move-down=['<Shift><Super>j']
window-move-left=['<Shift><Super>h']
window-move-right=['<Shift><Super>l']
window-move-up=['<Shift><Super>k']
window-resize-bottom-decrease=['<Shift><Control><Super>i']
window-resize-bottom-increase=['<Control><Super>u']
window-resize-left-decrease=['<Shift><Control><Super>o']
window-resize-left-increase=['<Control><Super>y']
window-resize-right-decrease=['<Shift><Control><Super>y']
window-resize-right-increase=['<Control><Super>o']
window-resize-top-decrease=['<Shift><Control><Super>u']
window-resize-top-increase=['<Control><Super>i']
window-snap-center=['<Control><Alt>c']
window-snap-one-third-left=['<Control><Alt>d']
window-snap-one-third-right=['<Control><Alt>g']
window-snap-two-third-left=['<Control><Alt>e']
window-snap-two-third-right=['<Control><Alt>t']
window-swap-down=['<Control><Super>j']
window-swap-last-active=['<Super>Return']
window-swap-left=['<Control><Super>h']
window-swap-right=['<Control><Super>l']
window-swap-up=['<Control><Super>k']
window-toggle-always-float=['<Shift><Super>c']
window-toggle-float=['<Super>c']
workspace-active-tile-toggle=['<Shift><Super>w']

Enabled Extensions

openweather-extension@jenslody.de
  Name: OpenWeather
  Description: Display weather information for any location on Earth in the GNOME Shell
  Path: /home/tessa/.local/share/gnome-shell/extensions/openweather-extension@jenslody.de
  URL: https://gitlab.com/skrewball/openweather
  Version: 121
  State: ENABLED

unredirect@vaina.lt
  Name: Disable unredirect fullscreen windows
  Description: Disables unredirect fullscreen windows in gnome-shell to workaround https://bugzilla.redhat.com/show_bug.cgi?id=767397 and https://bugzilla.gnome.org/show_bug.cgi?id=738719
  Path: /home/tessa/.local/share/gnome-shell/extensions/unredirect@vaina.lt
  URL: https://github.com/kazysmaster/gnome-shell-extension-disable-unredirect
  Version: 7
  State: ENABLED

user-theme@gnome-shell-extensions.gcampax.github.com
  Name: User Themes
  Description: Load shell themes from user directory.
  Path: /home/tessa/.local/share/gnome-shell/extensions/user-theme@gnome-shell-extensions.gcampax.github.com
  URL: https://gitlab.gnome.org/GNOME/gnome-shell-extensions
  Version: 51
  State: ENABLED

apps-menu@gnome-shell-extensions.gcampax.github.com
  Name: Applications Menu
  Description: Add a category-based menu for applications.
This extension is part of Classic Mode and is officially supported by GNOME. Please do not report bugs using the form below, use GNOME's GitLab instance instead.
  Path: /home/tessa/.local/share/gnome-shell/extensions/apps-menu@gnome-shell-extensions.gcampax.github.com
  URL: https://gitlab.gnome.org/GNOME/gnome-shell-extensions
  Version: 53
  State: ENABLED

forge@jmmaranan.com
  Name: Forge
  Description: Tiling and window manager for GNOME

Please report bugs/issues on https://github.com/forge-ext/forge/issues
  Path: /home/tessa/.local/share/gnome-shell/extensions/forge@jmmaranan.com
  URL: https://github.com/forge-ext/forge
  Version: 70
  State: ENABLED

appindicatorsupport@rgcjonas.gmail.com
  Name: AppIndicator and KStatusNotifierItem Support
  Description: Adds AppIndicator, KStatusNotifierItem and legacy Tray icons support to the Shell
  Path: /home/tessa/.local/share/gnome-shell/extensions/appindicatorsupport@rgcjonas.gmail.com
  URL: https://github.com/ubuntu/gnome-shell-extension-appindicator
  Version: 53
  State: ENABLED

WallpaperSwitcher@Rishu
  Name: Wallpaper Switcher
  Description: Extension to automatically Change wallpaper after a given interval
  Path: /home/tessa/.local/share/gnome-shell/extensions/WallpaperSwitcher@Rishu
  URL: https://github.com/rishuinfinity/WallpaperSwitcher
  Version: 9
  State: ENABLED

dash-to-dock@micxgx.gmail.com
  Name: Dash to Dock
  Description: A dock for the Gnome Shell. This extension moves the dash out of the overview transforming it in a dock for an easier launching of applications and a faster switching between windows and desktops. Side and bottom placement options are available.
  Path: /home/tessa/.local/share/gnome-shell/extensions/dash-to-dock@micxgx.gmail.com
  URL: https://micheleg.github.io/dash-to-dock/
  Original author: micxgx@gmail.com
  Version: 84
  State: ENABLED

cpufreq@konkor
  Name: cpufreq
  Description: System Monitor and Power Manager.

This is a lightweight system monitor and power management tool. It needs root permission to able changing governors.

Features:
⚫ Compatible with many hardware architectures;
⚫ CPU Frequency monitoring;
⚫ CPU Governor management;
⚫ CPU Frequency speed limits;
⚫ CPU Boost supporting;
⚫ CPU Core Power on/off;
⚫ Saving/Restoring settings...

For more information and how-to see README.md
  Path: /home/tessa/.local/share/gnome-shell/extensions/cpufreq@konkor
  URL: https://github.com/konkor/cpufreq
  Version: 54
  State: ENABLED

jmmaranan commented 1 year ago

Thanks for the report. This is related to #37

nergdron commented 1 year ago

ahhh yeah, this could be rolled into that one for sure. some sort of conflict resolution process where you could pick either gnome defaults or forge bindings would be an awesome solution to this issue.

nils-werner commented 1 year ago

Same with Super+H, which defaults to "hide window"

forge-ext / forge