gnome-shell-extension-simple-system-monitor icon indicating copy to clipboard operation
gnome-shell-extension-simple-system-monitor copied to clipboard

Resolve the feedback received during the submission of v15 to GNOME extensions

Open LGiki opened this issue 7 months ago • 0 comments

  1. lookupByUUID() is a bad practice (line 323 extension.js). You can send this from the entry point to the class needing it when it's possible (dependency injection).

  2. Use export class instead of export var.

  3. Make it local to fillPreferencesWindow() (line 73 prefs.js).

  4. Don't store any instance in global scope since that cannot get garbage collected after window close (line 70 prefs.js).

  5. Don't use the same property name for the timeout ids (line 410 and 535 extension.js).

  6. Please use TextDecoder instead of byteArray (line 34 extension.js): https://developer.mozilla.org/en-US/docs/Web/API/TextDecoder

LGiki avatar Nov 14 '23 01:11 LGiki