You can not select more than 25 topics
Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
209 lines
8.8 KiB
209 lines
8.8 KiB
2 years ago
|
From 3c62051c0a154ae987bb0126e8adb6cd86aa69a2 Mon Sep 17 00:00:00 2001
|
||
|
|
From: =?UTF-8?q?Florian=20M=C3=BCllner?= <fmuellner@gnome.org>
|
||
|
|
Date: Mon, 24 Feb 2020 16:17:05 +0100
|
||
|
|
Subject: [PATCH] dashToDock: Handle no-overview case
|
||
|
|
|
||
|
|
There is no longer an overview in GNOME Classic, so in order to be
|
||
|
|
used in that environment, the extension must deal with that case.
|
||
|
|
---
|
||
|
|
extensions/dash-to-dock/docking.js | 122 ++++++++++++++++-------------
|
||
|
|
1 file changed, 68 insertions(+), 54 deletions(-)
|
||
|
|
|
||
|
|
diff --git a/extensions/dash-to-dock/docking.js b/extensions/dash-to-dock/docking.js
|
||
|
|
index d35094b..2b8353a 100644
|
||
|
|
--- a/extensions/dash-to-dock/docking.js
|
||
|
|
+++ b/extensions/dash-to-dock/docking.js
|
||
|
|
@@ -233,7 +233,7 @@ var DockedDash = class DashToDock {
|
||
|
|
// Create a new dash object
|
||
|
|
this.dash = new MyDash.MyDash(this._settings, this._remoteModel, this._monitorIndex);
|
||
|
|
|
||
|
|
- if (!this._settings.get_boolean('show-show-apps-button'))
|
||
|
|
+ if (Main.overview.isDummy || !this._settings.get_boolean('show-show-apps-button'))
|
||
|
|
this.dash.hideShowAppsButton();
|
||
|
|
|
||
|
|
// Create the main actor and the containers for sliding in and out and
|
||
|
|
@@ -272,45 +272,11 @@ var DockedDash = class DashToDock {
|
||
|
|
this.dash.actor.add_constraint(this.constrainSize);
|
||
|
|
|
||
|
|
this._signalsHandler.add([
|
||
|
|
- Main.overview,
|
||
|
|
- 'item-drag-begin',
|
||
|
|
- this._onDragStart.bind(this)
|
||
|
|
- ], [
|
||
|
|
- Main.overview,
|
||
|
|
- 'item-drag-end',
|
||
|
|
- this._onDragEnd.bind(this)
|
||
|
|
- ], [
|
||
|
|
- Main.overview,
|
||
|
|
- 'item-drag-cancelled',
|
||
|
|
- this._onDragEnd.bind(this)
|
||
|
|
- ], [
|
||
|
|
// update when workarea changes, for instance if other extensions modify the struts
|
||
|
|
//(like moving th panel at the bottom)
|
||
|
|
global.display,
|
||
|
|
'workareas-changed',
|
||
|
|
this._resetPosition.bind(this)
|
||
|
|
- ], [
|
||
|
|
- Main.overview,
|
||
|
|
- 'showing',
|
||
|
|
- this._onOverviewShowing.bind(this)
|
||
|
|
- ], [
|
||
|
|
- Main.overview,
|
||
|
|
- 'hiding',
|
||
|
|
- this._onOverviewHiding.bind(this)
|
||
|
|
- ], [
|
||
|
|
- // Hide on appview
|
||
|
|
- Main.overview.viewSelector,
|
||
|
|
- 'page-changed',
|
||
|
|
- this._pageChanged.bind(this)
|
||
|
|
- ], [
|
||
|
|
- Main.overview.viewSelector,
|
||
|
|
- 'page-empty',
|
||
|
|
- this._onPageEmpty.bind(this)
|
||
|
|
- ], [
|
||
|
|
- // Ensure the ShowAppsButton status is kept in sync
|
||
|
|
- Main.overview.viewSelector._showAppsButton,
|
||
|
|
- 'notify::checked',
|
||
|
|
- this._syncShowAppsButtonToggled.bind(this)
|
||
|
|
], [
|
||
|
|
global.display,
|
||
|
|
'in-fullscreen-changed',
|
||
|
|
@@ -325,15 +291,6 @@ var DockedDash = class DashToDock {
|
||
|
|
this.dash,
|
||
|
|
'icon-size-changed',
|
||
|
|
() => { Main.overview.dashIconSize = this.dash.iconSize; }
|
||
|
|
- ], [
|
||
|
|
- // This duplicate the similar signal which is in owerview.js.
|
||
|
|
- // Being connected and thus executed later this effectively
|
||
|
|
- // overwrite any attempt to use the size of the default dash
|
||
|
|
- //which given the customization is usually much smaller.
|
||
|
|
- // I can't easily disconnect the original signal
|
||
|
|
- Main.overview._controls.dash,
|
||
|
|
- 'icon-size-changed',
|
||
|
|
- () => { Main.overview.dashIconSize = this.dash.iconSize; }
|
||
|
|
], [
|
||
|
|
// sync hover after a popupmenu is closed
|
||
|
|
this.dash,
|
||
|
|
@@ -341,6 +298,53 @@ var DockedDash = class DashToDock {
|
||
|
|
() => { this._box.sync_hover() }
|
||
|
|
]);
|
||
|
|
|
||
|
|
+ if (!Main.overview.isDummy) {
|
||
|
|
+ this._signalsHandler.add([
|
||
|
|
+ Main.overview,
|
||
|
|
+ 'item-drag-begin',
|
||
|
|
+ this._onDragStart.bind(this)
|
||
|
|
+ ], [
|
||
|
|
+ Main.overview,
|
||
|
|
+ 'item-drag-end',
|
||
|
|
+ this._onDragEnd.bind(this)
|
||
|
|
+ ], [
|
||
|
|
+ Main.overview,
|
||
|
|
+ 'item-drag-cancelled',
|
||
|
|
+ this._onDragEnd.bind(this)
|
||
|
|
+ ], [
|
||
|
|
+ Main.overview,
|
||
|
|
+ 'showing',
|
||
|
|
+ this._onOverviewShowing.bind(this)
|
||
|
|
+ ], [
|
||
|
|
+ Main.overview,
|
||
|
|
+ 'hiding',
|
||
|
|
+ this._onOverviewHiding.bind(this)
|
||
|
|
+ ], [
|
||
|
|
+ // Hide on appview
|
||
|
|
+ Main.overview.viewSelector,
|
||
|
|
+ 'page-changed',
|
||
|
|
+ this._pageChanged.bind(this)
|
||
|
|
+ ], [
|
||
|
|
+ Main.overview.viewSelector,
|
||
|
|
+ 'page-empty',
|
||
|
|
+ this._onPageEmpty.bind(this)
|
||
|
|
+ ], [
|
||
|
|
+ // Ensure the ShowAppsButton status is kept in sync
|
||
|
|
+ Main.overview.viewSelector._showAppsButton,
|
||
|
|
+ 'notify::checked',
|
||
|
|
+ this._syncShowAppsButtonToggled.bind(this)
|
||
|
|
+ ], [
|
||
|
|
+ // This duplicate the similar signal which is in owerview.js.
|
||
|
|
+ // Being connected and thus executed later this effectively
|
||
|
|
+ // overwrite any attempt to use the size of the default dash
|
||
|
|
+ //which given the customization is usually much smaller.
|
||
|
|
+ // I can't easily disconnect the original signal
|
||
|
|
+ Main.overview._controls.dash,
|
||
|
|
+ 'icon-size-changed',
|
||
|
|
+ () => { Main.overview.dashIconSize = this.dash.iconSize; }
|
||
|
|
+ ]);
|
||
|
|
+ }
|
||
|
|
+
|
||
|
|
this._injectionsHandler = new Utils.InjectionsHandler();
|
||
|
|
this._themeManager = new Theming.ThemeManager(this._settings, this);
|
||
|
|
|
||
|
|
@@ -370,14 +374,17 @@ var DockedDash = class DashToDock {
|
||
|
|
this._dashSpacer = new OverviewControls.DashSpacer();
|
||
|
|
this._dashSpacer.setDashActor(this._box);
|
||
|
|
|
||
|
|
- if (this._position == St.Side.LEFT)
|
||
|
|
- Main.overview._controls._group.insert_child_at_index(this._dashSpacer, this._rtl ? -1 : 0); // insert on first
|
||
|
|
- else if (this._position == St.Side.RIGHT)
|
||
|
|
- Main.overview._controls._group.insert_child_at_index(this._dashSpacer, this._rtl ? 0 : -1); // insert on last
|
||
|
|
- else if (this._position == St.Side.TOP)
|
||
|
|
- Main.overview._overview.insert_child_at_index(this._dashSpacer, 0);
|
||
|
|
- else if (this._position == St.Side.BOTTOM)
|
||
|
|
- Main.overview._overview.insert_child_at_index(this._dashSpacer, -1);
|
||
|
|
+ if (!Main.overview.isDummy) {
|
||
|
|
+ const { _controls, _overview } = Main.overview;
|
||
|
|
+ if (this._position == St.Side.LEFT)
|
||
|
|
+ _controls._group.insert_child_at_index(this._dashSpacer, this._rtl ? -1 : 0); // insert on first
|
||
|
|
+ else if (this._position == St.Side.RIGHT)
|
||
|
|
+ _controls._group.insert_child_at_index(this._dashSpacer, this._rtl ? 0 : -1); // insert on last
|
||
|
|
+ else if (this._position == St.Side.TOP)
|
||
|
|
+ _overview.insert_child_at_index(this._dashSpacer, 0);
|
||
|
|
+ else if (this._position == St.Side.BOTTOM)
|
||
|
|
+ _overview.insert_child_at_index(this._dashSpacer, -1);
|
||
|
|
+ }
|
||
|
|
|
||
|
|
// Add dash container actor and the container to the Chrome.
|
||
|
|
this.actor.set_child(this._slider);
|
||
|
|
@@ -412,7 +419,7 @@ var DockedDash = class DashToDock {
|
||
|
|
|
||
|
|
// Since Gnome 3.8 dragging an app without having opened the overview before cause the attemp to
|
||
|
|
//animate a null target since some variables are not initialized when the viewSelector is created
|
||
|
|
- if (Main.overview.viewSelector._activePage == null)
|
||
|
|
+ if (!Main.overview.isDummy && Main.overview.viewSelector._activePage == null)
|
||
|
|
Main.overview.viewSelector._activePage = Main.overview.viewSelector._workspacesPage;
|
||
|
|
|
||
|
|
this._updateVisibilityMode();
|
||
|
|
@@ -493,7 +500,8 @@ var DockedDash = class DashToDock {
|
||
|
|
this._settings,
|
||
|
|
'changed::show-show-apps-button',
|
||
|
|
() => {
|
||
|
|
- if (this._settings.get_boolean('show-show-apps-button'))
|
||
|
|
+ if (!Main.overview.isDummy &&
|
||
|
|
+ this._settings.get_boolean('show-show-apps-button'))
|
||
|
|
this.dash.showShowAppsButton();
|
||
|
|
else
|
||
|
|
this.dash.hideShowAppsButton();
|
||
|
|
@@ -1681,6 +1689,9 @@ var DockManager = class DashToDock_DockManager {
|
||
|
|
// set stored icon size to the new dash
|
||
|
|
Main.overview.dashIconSize = this._allDocks[0].dash.iconSize;
|
||
|
|
|
||
|
|
+ if (Main.overview.isDummy)
|
||
|
|
+ return;
|
||
|
|
+
|
||
|
|
// Hide usual Dash
|
||
|
|
Main.overview._controls.dash.actor.hide();
|
||
|
|
|
||
|
|
@@ -1707,6 +1718,9 @@ var DockManager = class DashToDock_DockManager {
|
||
|
|
}
|
||
|
|
|
||
|
|
_restoreDash() {
|
||
|
|
+ if (Main.overview.isDummy)
|
||
|
|
+ return;
|
||
|
|
+
|
||
|
|
Main.overview._controls.dash.actor.show();
|
||
|
|
Main.overview._controls.dash.actor.set_width(-1); //reset default dash size
|
||
|
|
// This force the recalculation of the icon size
|
||
|
|
--
|
||
|
|
2.25.0
|
||
|
|
|