Review of "Dynamic Notch" version 1

Details Page Preview

Top-center notch/pill with a clock that expands into a compact, flat native-style tabbed layout: Tab 1 holds a small dense calendar, a compact notification list, and running-application icons with an Activities Overview button; Tab 2 is a minimal timer that sizes the panel to its own content. Icon-only quick actions; no cards or tiles. GNOME notifications are mirrored into the island (stock banners replaced by an island peek + list). Configurable panel background with smooth Overview transitions. GNOME Shell 51.

Extension Homepage
https://github.com/faishalmhmd/dynamic-notch

No comments.

FAQ

Files

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

Shexli (experimental) warning 3

Shexli found 3 issues that may need reviewer attention.

EGO-L-002 warning

objects created by extension should be destroyed in disable()

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

Destroy all objects

  • extension.js:697
    this._topBar = topBar
  • extension.js:1358
    label
  • extension.js:1310
    label

EGO-L-005 warning

owned object references should be released in disable()

Owned references that are cleaned up in `disable()` should also be released with `null` or `undefined`.

Destroy all objects

  • extension.js:697
    this._topBar = topBar

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

  • extension.js:969
    actor.connect('enter-event', () => this._showTooltip(actor, text))
  • extension.js:970
    actor.connect('leave-event', () => this._hideTooltip())

All Versions

Version Status
3 Unreviewed
2 Rejected
1 Rejected

Previous Reviews on this Version

faishalmhmd auto- rejected
Auto-rejected because of new version 2 was uploaded