Review of "Vertical overview" version 2

Details Page Preview

Bringing back vertically stacked workspaces

Extension Homepage
https://github.com/RensAlthuis/vertical-overview

No comments.

Diff Against

Files

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

All Versions

Version Status
10 Active
9 Active
8 Active
7 Rejected
6 Active
5 Inactive
4 Inactive
3 Rejected
2 Rejected
1 Rejected

Previous Reviews on this Version

corvettecole rejected
hey there, make sure you don't have any extra connects that you didn't disconnect. It looks like when you call bindSettings() you connect a bunch of signals to a settings object. These signals are never removed and the settings object is never explicitly deleted. If you hold the settings object as a reference, then you can disconnect the signals and delete it in disable()