aunetx / blur-my-shell

Extension that adds a blur look to different parts of the GNOME Shell, including the top panel, dash and overview
https://extensions.gnome.org/extension/3193/blur-my-shell/
GNU General Public License v3.0
1.58k stars 88 forks source link

Error loading extension after locking screen #224

Closed pesader closed 2 years ago

pesader commented 2 years ago

Whenever my laptop goes to the lockscreen (either by Suspend or Lock) the "Overview components style" preference is ignored and the components go back to their original opaque appearance.

In addition to that, Extension Manager shows that the Blur My Shell could not be loaded (although everything seems to be working, except the Overview components styling). The error it displays is "this.effect.actor is null".

Logging out and logging in again brings everything back to normal.

Opaque overview components
image
Error loading extension
image

And thanks a lot for developing Blur My Shell. I requested the overview components styling back in issue #174 and you made it look better than I imagined it. Great work! :star_struck:

retrixe commented 2 years ago

I'm having the same issue, all the blurs are intact, but the dash and search cards are broken (dash screenshot is in original issue).

Issues: image image

The error: image image

I was able to patch this by editing ~/.local/share/gnome-shell/extensions/blur-my-shell@aunetx/components/panel.js line 250 like so:

-        if (this.prefs.PANEL_STATIC_BLUR.get())
+        if (this.prefs.PANEL_STATIC_BLUR.get() && this.effect.actor != null)
zayats1 commented 2 years ago

I'm having the same issue, all the blurs are intact, but the dash and search cards are broken (dash screenshot is in original issue).

Issues: image image

The error: image image

I was able to patch this by editing ~/.local/share/gnome-shell/extensions/blur-my-shell@aunetx/components/panel.js line 250 like so:

-        if (this.prefs.PANEL_STATIC_BLUR.get())
+        if (this.prefs.PANEL_STATIC_BLUR.get() && this.effect.actor != null)

create pr

qwertychouskie commented 2 years ago

Can confirm, getting same issue.

pesader commented 2 years ago

I opened a PR, crediting @retrixe.

aunetx commented 2 years ago

This has been merged, thanks!