Review of "Wadokei 和時計" version 3

Details Page Preview

Japanese temporal hours (不定時法): the current toki kanji in the top bar, computed from sunrise/sunset at your location. The popup lists all twelve hours. Optional hourly bell chime (off by default; strikes the traditional bell count at each hour's center). Language, time format, dawn/dusk offset, and location are configurable in preferences. No notifications.

Extension Homepage
https://github.com/tedvask/wadokei

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) warning 1

Shexli found 1 issue 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:302
    this._infoItem = new PopupMenu.PopupMenuItem('', {reactive: false})
  • extension.js:296
            this._label = new St.Label({
                text: '…',
                y_align: Clutter.ActorAlign.CENTER,
            })
  • extension.js:318
    this._locItem = new PopupMenu.PopupMenuItem('', {reactive: false})
  • extension.js:303
    this._rangeItem = new PopupMenu.PopupMenuItem('', {reactive: false})
  • extension.js:317
    this._sunItem = new PopupMenu.PopupMenuItem('', {reactive: false})
  • extension.js:313
    row

All Versions

Version Status
3 Active
2 Rejected
1 Active

Previous Reviews on this Version

JustPerfection active
Thanks!