1. Please remove unnecessary files: - `stylesheet.css` - `po/toggleAudio@blackeyeprojects.de.pot` [EGO Review Guidelines: unnecessary files](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#don-t-include-unnecessary-files) 2. Also null out in disable: ```js this._mixerControlFacade = null; ``` [EGO Review Guidelines: Destroy](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#destroy-all-objects) 3. When you store instance of objects in the properties of the default class you are exporting, it won't let the garbage collector do its job (line 15-25 `prefs.js`). You have two options: 1. Make them local to the functions. 2. Null them out in the window close request: ```js window.connect('close-request', () => { this._settings = null; }); ``` 4. Move `lib/util/DebugHelper.js` to a `prefs` folder or find a solution to make it obvious for the reviewers to know that file is prefs only module. 5. Timeout should be removed on destroy (line 112, 144 `lib/MixerControlFacade.js`): [EGO Review Guidelines: Timeout](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#remove-main-loop-sources)

About #3: You can also move everything (other than `fillPreferencesWindow()`) to another class and only create the instance of that class in `fillPreferencesWindow()`. That's the better solution.

Commit: Fix for Reviewpoint 1,2,4 https://github.com/Blackstareye/toggleAudio-blackeyeprojects.de/commit/613fa5ab260edc75c3489fb6b3aa3ea7b8a7da24 Commit: Fix for Reviewpoint 3 https://github.com/Blackstareye/toggleAudio-blackeyeprojects.de/commit/c0a6ccd349eb86b06c9992bb3ea6dcb4957df881 Commit: Fix for Reviewpoint 5 https://github.com/Blackstareye/toggleAudio-blackeyeprojects.de/commit/d3b9ce0edbf72f9f17b499eb3e6609c066457736 next version uploaded