SergKolo / launcher-list-indicator

Indicator for Ubuntu with Unity desktop , which allows switching between profiles for the Unity launcher
MIT License
21 stars 2 forks source link

"Remove entry" issues #1

Closed hotice closed 8 years ago

hotice commented 8 years ago

When the "Remove entry" indicator menu item is used, a popup to name the profile is displayed. If I name the profile with the exact same name as an existing profile (either the one I want to remove or another one), all profile entries disappear from the indicator. The ~/.launcher_profiles.json file continues to have the other profiles, they just don't show up in the indicator.

Also, if after clicking "Remove entry", I enter a profile name that's not already in use, nothing occurs (both the indicator and the configuration file remain unchanged).

SergKolo commented 8 years ago

@hotice Does it display any traceback issues when running from terminal ? I had the same issue in mouse-profile-indicator, which was fixed, and ported the fix into this indicator as well. The latest commit was 944f56d389aff8acc94629d81148ba42469dfe41 , run git status and see if you got the latest one. With the latest commit I had changed the remove entry dialog to say "Which entry to remove?" instead of "Name the profile".

hotice commented 8 years ago

Ah, since it was just released, I didn't check the latest git. Indeed, it's fixed in git.