Review of "Show Desktop Button" version 6

Details Page Preview

Places a button to the left(default) of the App Menu to hide all windows, and the Overview if active, and show the current desktop. Panel location can be changed via prefs. The visible icon changes when you have toggled hiding/showing of the desktop. Can now be controlled via space/keyboard as well, when hovered, this can be achieved with ctrl/alt/tab/panel/arrows/space or enter. Eventually actual keybinding will happen(v22??). *icon credit madkristoff(thanks, this is _still_ the only icon created and submitted!)* (credit:madkristoff, jonnius, mbokil, simonthechipmunk, MGSE, erguille, mikechaberski, mathematicalcoffee, asan, spinus, Xes, gcampax, magcius). v21: cleanup v20 store toggle mode when panel location changed remove st.bin fix settings leak this is stable now! v20 Huge release lots of cleanup, now works with space or enter when hovered, also restores focus to last focused window. v19 localized thanks to jonnius, icon now toggles modes to indicate status of windows. v18 stylesheet removed, now uses system style, more cleanup & notations. v17 placement option returns, left right or center only for now. todo: icon setting, keybinding, better pleacement

Extension Homepage
https://github.com/l300lvl/Show-Desktop-Button

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

Previous Reviews on this Version

l300lvl posted a review
was told this would work for adding to the leftbox in 3.4, please correct me if im wrong here, i also made the same change to the 'favorites menu plus'
Jasper St. Pierre posted a review
Don't you also need to fix the rightbox code? Also, insert_child(actor, _children.length-1); can just be replaced with add_actor.
l300lvl posted a review
what would need to change with the rightbox that Im not aware of? So instead of insert_child_at_index I can use add_actor in both places? Would I then remove Main.panel._leftBox.add(button.actor); or would it be more like Main.panel._leftBox.add_actor(button.actor); Main.panel._leftBox.add(button.actor);