Skip to content

Commit 0c949f7

Browse files
authored
Merge branch 'main' into leolost/fix-notification-segfault
2 parents bc0d46d + debdb5a commit 0c949f7

File tree

3 files changed

+22
-16
lines changed

3 files changed

+22
-16
lines changed

data/gala.gschema.xml

Lines changed: 11 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@
6262
</description>
6363
</key>
6464
<key type="s" name="panel-main-menu-action">
65-
<default>''</default>
65+
<default>'io.elementary.wingpanel --toggle-indicator=app-launcher'</default>
6666
<summary>Panel main menu action</summary>
6767
<description>Sets the command to run when the panel-main-menu keybinding is pressed.</description>
6868
</key>
@@ -72,10 +72,15 @@
7272
<description></description>
7373
</key>
7474
<key type="s" name="overlay-action">
75-
<default>''</default>
75+
<default>'io.elementary.wingpanel --toggle-indicator=app-launcher'</default>
7676
<summary></summary>
7777
<description></description>
7878
</key>
79+
<key type="s" name="interactive-screenshot-action">
80+
<default>'flatpak run io.elementary.screenshot'</default>
81+
<summary>Interactive screenshot action</summary>
82+
<description>Sets the command to run when the interactive-screenshot keybinding is pressed.</description>
83+
</key>
7984
<key type="b" name="move-maximized-workspace">
8085
<default>false</default>
8186
<summary>Automatically move maximized windows to a new workspace</summary>
@@ -144,6 +149,10 @@
144149
<default><![CDATA[['Print']]]></default>
145150
<summary>Take a screenshot</summary>
146151
</key>
152+
<key name="interactive-screenshot" type="as">
153+
<default><![CDATA[['<Super>Print']]]></default>
154+
<summary>Launch the interactive screenshot tool</summary>
155+
</key>
147156
<key name="window-screenshot" type="as">
148157
<default><![CDATA[['<Alt>Print']]]></default>
149158
<summary>Take a screenshot of a window</summary>

src/ShellClients/HideTracker.vala

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -213,13 +213,10 @@ public class Gala.HideTracker : Object {
213213
}
214214

215215
private void toggle_display (bool should_hide) {
216-
unowned var window_actor = (Meta.WindowActor) panel.window.get_compositor_private ();
217-
218-
// Window actor receives pointer events while hidden on X11: https://github.com/elementary/gala/issues/2083
219216
#if HAS_MUTTER45
220-
hovered = panel.window.has_pointer () && window_actor.visible;
217+
hovered = panel.window.has_pointer ();
221218
#else
222-
hovered = window_has_pointer () && window_actor.visible;
219+
hovered = window_has_pointer ();
223220
#endif
224221

225222
if (should_hide && !hovered && !panel.window.has_focus ()) {

src/WindowManager.vala

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -253,6 +253,7 @@ namespace Gala {
253253
display.add_keybinding ("switch-input-source-backward", keybinding_settings, Meta.KeyBindingFlags.IGNORE_AUTOREPEAT, (Meta.KeyHandlerFunc) handle_switch_input_source);
254254

255255
display.add_keybinding ("screenshot", keybinding_settings, Meta.KeyBindingFlags.IGNORE_AUTOREPEAT, (Meta.KeyHandlerFunc) handle_screenshot);
256+
display.add_keybinding ("interactive-screenshot", keybinding_settings, Meta.KeyBindingFlags.IGNORE_AUTOREPEAT, (Meta.KeyHandlerFunc) handle_screenshot);
256257
display.add_keybinding ("window-screenshot", keybinding_settings, Meta.KeyBindingFlags.IGNORE_AUTOREPEAT, (Meta.KeyHandlerFunc) handle_screenshot);
257258
display.add_keybinding ("area-screenshot", keybinding_settings, Meta.KeyBindingFlags.IGNORE_AUTOREPEAT, (Meta.KeyHandlerFunc) handle_screenshot);
258259
display.add_keybinding ("screenshot-clip", keybinding_settings, Meta.KeyBindingFlags.IGNORE_AUTOREPEAT, (Meta.KeyHandlerFunc) handle_screenshot);
@@ -405,10 +406,12 @@ namespace Gala {
405406
private void launch_action (string action_key) {
406407
try {
407408
var action = behavior_settings.get_string (action_key);
408-
if (action != null && action != "") {
409+
if (action != null) {
409410
Process.spawn_command_line_async (action);
410411
}
411-
} catch (Error e) { warning (e.message); }
412+
} catch (Error e) {
413+
warning (e.message);
414+
}
412415
}
413416

414417
private void on_monitors_changed () {
@@ -511,6 +514,9 @@ namespace Gala {
511514
case "screenshot":
512515
screenshot_screen.begin ();
513516
break;
517+
case "interactive-screenshot":
518+
launch_action ("interactive-screenshot-action");
519+
break;
514520
case "area-screenshot":
515521
screenshot_area.begin ();
516522
break;
@@ -1040,13 +1046,7 @@ namespace Gala {
10401046
current.@delete (Gtk.get_current_event_time ());
10411047
break;
10421048
case ActionType.OPEN_LAUNCHER:
1043-
try {
1044-
Process.spawn_command_line_async (
1045-
behavior_settings.get_string ("panel-main-menu-action")
1046-
);
1047-
} catch (Error e) {
1048-
warning (e.message);
1049-
}
1049+
launch_action ("panel-main-menu-action");
10501050
break;
10511051
case ActionType.WINDOW_OVERVIEW:
10521052
if (window_overview == null) {

0 commit comments

Comments
 (0)