Review "Status Title Bar" version 6 - GNOME Shell Extensions

Review of "Status Title Bar" version 6

Details Page Preview

Status Title Bar

Shows the title of windows in the status bar. FOR BEST RESULTS, USE THE Extend left box EXTENSION TO MAKE THE TITLE TAKE THE WHOLE AVAILABLE SPACE IN THE STATUS BAR

Extension Homepage: https://github.com/emerinohdz/StatusTitleBar

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
22	Active
21	Active
20	Active
19	Active
18	Active
17	Rejected
16	Active
15	Active
14	Active
13	Active
12	Rejected
11	Rejected
10	Active
9	Active
8	Active
7	Active
6	Rejected
5	Active
4	Active
3	Active
2	Rejected
1	Rejected

Previous Reviews on this Version

gcampax posted a review May 20, 2012

You need to disconnect a bunch of signals in your AppMenuButton (essentially, all you connect to global objects). (Btw, if you just copied the code from panel.js, and that's not disconnecting the signals in destroy(), you have a recipe for trouble, as signals will be invoked on a destroyed actor and at least cause criticals)

emerino posted a review May 20, 2012

Ok, I actually haven't reviewed the code my self, it was contributed by another user (although I believe problems come from my actual code). I will fix those errors and do the same for the poweralttab extension. Have a good day!