GNOME Shell Extensions

Review of "AppIndicator and KStatusNotifierItem Support" version 8

Details Page Preview

AppIndicator and KStatusNotifierItem Support

by 3v1n0

Adds AppIndicator, KStatusNotifierItem and legacy Tray icons support to the Shell

Extension Homepage
https://github.com/ubuntu/gnome-shell-extension-appindicator
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
59 Active
58 Active
57 Active
56 Active
55 Rejected
54 Rejected
53 Active
52 Active
51 Active
50 Active
49 Active
48 Active
47 Active
46 Active
45 Active
44 Active
43 Active
42 Active
41 Active
40 Active
39 Rejected
38 Rejected
37 Active
36 Active
35 Active
34 Active
33 Active
32 Active
31 Active
30 Active
29 Active
28 Active
27 Active
26 Active
25 Rejected
24 Active
23 Active
22 Active
21 Active
20 Active
19 Active
18 Active
17 Active
16 Rejected
15 Active
14 Active
13 Active
12 Rejected
11 Active
10 Active
9 Active
8 Active
7 Active
6 Active
5 Active
4 Rejected
3 Rejected
2 Rejected
1 Active

Previous Reviews on this Version

gcampax active
clutter_image_set_data() is not supposed to work, it's lacking a length argument - maybe previously it worked as a null terminated string, but that's obviously wrong for an image. set_bytes() OTOH is the right thing, as gjs implicitly converts byte arrays to and from GBytes
rgcjonas posted a review
It's interesting because set_data blows up in Ubuntu 13.04 with gnome3 ppa (gjs 1.36; instant segmentation fault). set_bytes however throws a TypeError in Ubuntu 12.10 (gjs 1.34), while set_data seems to work fine. Since clutter_image_set_data is incorrect, how should I handle that in GJS 1.34?
gcampax posted a review
You can't, you need gjs 1.36 to handle GBytes transparently. Use the GdkPixbuf as a GIcon instead, and use a StIcon derived from it.