#32 extension: Do not relayout on scale changes
Opened 3 years ago by garnacho. Modified 3 years ago
garnacho/background-logo-extension garnacho/overview-animation-glitches  into  master

file modified
-7
@@ -25,13 +25,6 @@ 

  class IconContainer extends St.Widget {

      _init(params) {

          super._init(params);

- 

-         this.connect('notify::scale-x', () => {

-             this.queue_relayout();

-         });

-         this.connect('notify::scale-y', () => {

-             this.queue_relayout();

-         });

      }

  

      vfunc_get_preferred_width(forHeight) {

IconContainer considers the scale for its size request, so rather than just removing the relayout, can we get rid of the custom actor altogether?

This makes the background logo position flicker as we enter the
overview. This actor is part of the BackgroundActor actor hierarchy,
and can be left to scale neatly with it.

Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1943938

IconContainer considers the scale for its size request, so rather than just removing the relayout, can we get rid of the custom actor altogether?

Metadata