Review of "Wetterkurve" version 5

Details Page Preview

A compact three-day weather chart for up to three saved locations

Extension Homepage
https://github.com/vibecodingwean/wetterkurve

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:580
            this._bigIcon = new St.Icon({
                gicon: this._weatherIcon('unknown'),
                icon_size: 42,
                style_class: 'mw-big-icon',
            })
  • extension.js:636
    this._chart = new ForecastChart()
  • extension.js:523
            this._locationBar = new St.BoxLayout({
                style_class: 'mw-location-bar',
                x_expand: true,
            })
  • extension.js:527
            this._locationTabs = new St.BoxLayout({
                style_class: 'mw-location-tabs',
                x_expand: true,
            })
  • extension.js:499
            this._panelIcon = new St.Icon({
                gicon: this._weatherIcon('unknown'),
                icon_size: 17,
                style_class: 'mw-panel-icon',
            })
  • extension.js:586
            this._refreshButton = new St.Button({
                label: '↻',
                style_class: 'mw-refresh-button',
                can_focus: true,
                accessible_name: this._t('refreshWeather'),
            })
  • extension.js:532
            this._removeLocationButton = new St.Button({
                label: '−',
                style_class: 'mw-location-remove-button',
                can_focus: true,
                visible: false,
            })
  • extension.js:543
            this._searchBox = new St.BoxLayout({
                vertical: true,
                style_class: 'mw-search-box',
                x_expand: true,
                visible: false,
            })
  • extension.js:549
            this._searchEntry = new St.Entry({
                hint_text: this._t('searchLocation'),
                can_focus: true,
                style_class: 'mw-search-entry',
                x_expand: true,
            })
  • extension.js:561
            this._searchResults = new St.BoxLayout({
                vertical: true,
                style_class: 'mw-search-results',
                x_expand: true,
            })

All Versions

Version Status
5 Unreviewed
4 Rejected
3 Rejected
2 Rejected
1 Rejected