GNOME Shell Extensions

Review of "Blur my Shell" version 73

Details Page Preview

Blur my Shell

by aunetx

Adds a blur look to different parts of the GNOME Shell, including the top panel, dash and overview. You can support my work by sponsoring me on: - github: https://github.com/sponsors/aunetx - ko-fi: https://ko-fi.com/aunetx If you use Ubuntu, you may want to change the radius of the corner effect for Dash-to-Dock in preferences, else you may see unpleasantly mixed radii in your dock. Note: if the extension shows an error after updating, please make sure to restart your session to see if it persists. This is due to a bug in gnome shell, which I can't fix by myself.

Extension Homepage
https://github.com/aunetx/blur-my-shell
No comments.

Diff Against

Files

Note: Binary files aren't shown on the web site. To see all files, please download the extension zipfile.

Shexli (experimental) error 2 warning 2

Shexli found 4 issues that may need reviewer attention.

EGO-P-004 error

GSettings schema XML filename must match schema id

GSettings schema filename must match `<schema-id>.gschema.xml`.

GSettings Schemas

  • schemas/org.gnome.shell.extensions.blur-my-shell.gschema.xml 
    id='org.gnome.shell.extensions.blur-my-shell.overview' path='/org/gnome/shell/extensions/blur-my-shell/overview/'
  • schemas/org.gnome.shell.extensions.blur-my-shell.gschema.xml 
    id='org.gnome.shell.extensions.blur-my-shell.appfolder' path='/org/gnome/shell/extensions/blur-my-shell/appfolder/'
  • schemas/org.gnome.shell.extensions.blur-my-shell.gschema.xml 
    id='org.gnome.shell.extensions.blur-my-shell.panel' path='/org/gnome/shell/extensions/blur-my-shell/panel/'
  • schemas/org.gnome.shell.extensions.blur-my-shell.gschema.xml 
    id='org.gnome.shell.extensions.blur-my-shell.dash-to-dock' path='/org/gnome/shell/extensions/blur-my-shell/dash-to-dock/'
  • schemas/org.gnome.shell.extensions.blur-my-shell.gschema.xml 
    id='org.gnome.shell.extensions.blur-my-shell.applications' path='/org/gnome/shell/extensions/blur-my-shell/applications/'
  • schemas/org.gnome.shell.extensions.blur-my-shell.gschema.xml 
    id='org.gnome.shell.extensions.blur-my-shell.screenshot' path='/org/gnome/shell/extensions/blur-my-shell/screenshot/'
  • schemas/org.gnome.shell.extensions.blur-my-shell.gschema.xml 
    id='org.gnome.shell.extensions.blur-my-shell.lockscreen' path='/org/gnome/shell/extensions/blur-my-shell/lockscreen/'
  • schemas/org.gnome.shell.extensions.blur-my-shell.gschema.xml 
    id='org.gnome.shell.extensions.blur-my-shell.window-list' path='/org/gnome/shell/extensions/blur-my-shell/window-list/'
  • schemas/org.gnome.shell.extensions.blur-my-shell.gschema.xml 
    id='org.gnome.shell.extensions.blur-my-shell.coverflow-alt-tab' path='/org/gnome/shell/extensions/blur-my-shell/coverflow-alt-tab/'
  • schemas/org.gnome.shell.extensions.blur-my-shell.gschema.xml 
    id='org.gnome.shell.extensions.blur-my-shell.popup' path='/org/gnome/shell/extensions/blur-my-shell/popup/'
  • schemas/org.gnome.shell.extensions.blur-my-shell.gschema.xml 
    id='org.gnome.shell.extensions.blur-my-shell.hidetopbar' path='/org/gnome/shell/extensions/blur-my-shell/hidetopbar/'
  • schemas/org.gnome.shell.extensions.blur-my-shell.gschema.xml 
    id='org.gnome.shell.extensions.blur-my-shell.dash-to-panel' path='/org/gnome/shell/extensions/blur-my-shell/dash-to-panel/'

EGO-L-001 warning

extension must not create GObject instances or modify shell before enable()

Resource creation or signal/source setup was found outside `enable()`.

Only use initialization for static resources

  • components/panel.js:27 
    Main.panel.connect('destroy', () => isMainPanelAlive = false)

EGO-L-003 warning

signals connected by extension should be disconnected in disable()

Signals assigned in `enable()` are missing matching disconnect calls in `disable()` or its helper methods.

Disconnect all signals

  • components/dash_to_dock.js:42 
            this.bg_allocation_id = this.background_group?.connect('notify::allocation', () => {
            this.schedule_update();
        })
  • components/dash_to_dock.js:49 
    this.dash_destroy_id = dash.connect('destroy', () => this.remove_dash_blur(false))
  • components/dash_to_dock.js:52 
    this.dash_blur.connect('remove-dashes', () => this.remove_dash_blur())
  • extension.js:110 
                this._connection.connect(
                Main.layoutManager,
                'startup-complete',
                () => this._enable_components()
            )
  • extension.js:94 
            this._connection.connect(Main.sessionMode, 'updated',
            () => this._on_session_mode_changed(Main.sessionMode)
        )

EGO-M-004 error

metadata shell-version must only include plausible stable releases and at most one development release

Field `shell-version` contains invalid values, more than one development release, or implausible future releases.

metadata.json must be well-formed

  • metadata.json 
    shell-version=['46', '47', '48', '49', '50', '51']

All Versions

Version Status
73 Rejected
72 Active
71 Active
70 Active
69 Active
68 Active
67 Active
66 Active
65 Active
64 Active
63 Active
62 Active
61 Active
60 Active
59 Active
58 Active
57 Rejected
56 Rejected
55 Active
54 Active
53 Active
52 Active
51 Active
50 Active
49 Rejected
48 Rejected
47 Active
46 Active
45 Active
44 Active
43 Active
42 Active
41 Active
40 Rejected
39 Active
38 Active
37 Active
36 Active
35 Active
34 Active
33 Active
32 Active
31 Active
30 Rejected
29 Active
28 Active
27 Active
26 Active
25 Active
24 Active
23 Active
22 Active
21 Active
20 Active
19 Active
18 Active
17 Rejected
16 Active
15 Active
14 Active
13 Active
12 Active
11 Active
10 Active
9 Active
8 Active
7 Active
6 Active
5 Active
4 Active
3 Active
2 Active
1 Active

Previous Reviews on this Version

JustPerfection rejected
Didn't review. There are many lines of unnecessary lines: https://gjs.guide/extensions/review-guidelines/best-practices.html#avoid-unnecessary-try-catch-wrappers Also fix EGO-L-001 mentioned by Shexli.