[gnome-shell/gbsneto/folder-dialog-improvements: 12/30] appDisplay: Add folder dialog to overview group



commit d8abcff80461ce7302caf16045ada2a880bf6c08
Author: Georges Basile Stavracas Neto <georges stavracas gmail com>
Date:   Thu Jun 4 01:37:31 2020 -0300

    appDisplay: Add folder dialog to overview group
    
    Right now, the app folder dialog isn't really a dialog,
    since it is actually added to the AppDisplay. Furthermore,
    having it added in AppDisplay may mess up with its sizing
    calculations, since AppDisplay has a ClutterBinLayout and
    the folder dialog has a fairly large minimum size.
    
    Add the folder dialog to the overview group. Next commits
    will adjust various actors to be able to better handle it.
    
    https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/1301

 js/ui/appDisplay.js | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)
---
diff --git a/js/ui/appDisplay.js b/js/ui/appDisplay.js
index 8931ca4a16..ef36f8b675 100644
--- a/js/ui/appDisplay.js
+++ b/js/ui/appDisplay.js
@@ -728,7 +728,7 @@ class AppDisplay extends BaseAppView {
     }
 
     addFolderDialog(dialog) {
-        this.add_child(dialog);
+        Main.layoutManager.overviewGroup.add_child(dialog);
         dialog.connect('open-state-changed', (o, isOpen) => {
             this._eventBlocker.visible = isOpen;
 
@@ -1759,6 +1759,8 @@ var AppFolderDialog = GObject.registerClass({
         if (!this._isOpen)
             return;
 
+        this.get_parent().set_child_above_sibling(this, null);
+
         this._needsZoomAndFade = true;
         this.show();
 


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]