jake-phy / WindowIconList

GNU General Public License v2.0
75 stars 26 forks source link

document or address the need to restart Cinnamon after installing #29

Closed tmetro closed 9 years ago

tmetro commented 11 years ago

I tried installing this applet last October using a 1.x version. I installed the files using the typical technique for applets, then added it to the panel. A little bit of empty space was added to the panel and that's it.

I assumed the applet was broken, unchecked it from the applets dialog and forgot about it.

Just recently I tried it again. This time I noticed the installation instructions and I used your configure.py tool to install it.

Once again when the applet was added to the panel, it still did nothing but add some whitespace. I figured it was still broken, and planned to file a bug. But this time I didn't uncheck it in the applets dialog.

On my next reboot (which happens rarely) I was surprised to see it working on my panel. Apparently a restart of Cinnamon is required.

I did notice that configure.py included a button to restart Cinnamon, but no indication that this was absolutely required. I figured it might be there just for convenience.

Restarting Cinnamon is a "data destructive" operation - all child apps are killed off. So not exactly a practical thing to do unless you are prepared to shut everything down.

Ideally, you should redesign your applet to avoid needing a Cinnamon restart.

But I can understand that may not be trivial, so until that can be accomplished, I recommend:

  1. clarifying in your documentation and in configure.py that a Cinnamon restart is definitely required. Explain to the user what they will or won't see until that happens.
  2. an interim measure would be to modify the applet so that it puts up a placeholder icon on the panel until Cinnamon is restarted. This is certainly doable, as pretty much all applets display an icon immediately upon being checked in the applet dialog. The placeholder icon could show your applets logo and when clicked display a message about Cinnamon requiring a restart before it will work. The user should be able to reposition this placeholder, so the new position will be used by your applet after the restart.
aikeru commented 9 years ago

I think this has been resolved.

jake-phy commented 9 years ago

Yes this is resolved.