In the applications window, clicking Add To Menu on an app exports it and the heading says "App Exported!" - but the row still shows Add To Menu. The button that offers the reverse, Remove From Menu, only appears after closing the window and opening it again: which of the two buttons a row gets is decided once, when the list is built, from whether ~/.local/share/applications/<box>-<app>.desktop exists at that moment, and nothing updates it afterwards. So right after adding an app to the menu there is no way to take it out again from the same row, and a second click on the still-visible Add To Menu just exports it a second time.
Expected: the row reflects the menu straight away - one button that reads Add To Menu or Remove From Menu depending on whether the exported desktop file is there now, and flips after each click. Reading the state back from the host directory after the command, rather than assuming the export worked, also means the button never claims something the menu doesn't have.
I have a fix for this together with the row layout from #232, since it is the same few lines; PR to follow.
In the applications window, clicking Add To Menu on an app exports it and the heading says "App Exported!" - but the row still shows Add To Menu. The button that offers the reverse, Remove From Menu, only appears after closing the window and opening it again: which of the two buttons a row gets is decided once, when the list is built, from whether
~/.local/share/applications/<box>-<app>.desktopexists at that moment, and nothing updates it afterwards. So right after adding an app to the menu there is no way to take it out again from the same row, and a second click on the still-visible Add To Menu just exports it a second time.Expected: the row reflects the menu straight away - one button that reads Add To Menu or Remove From Menu depending on whether the exported desktop file is there now, and flips after each click. Reading the state back from the host directory after the command, rather than assuming the export worked, also means the button never claims something the menu doesn't have.
I have a fix for this together with the row layout from #232, since it is the same few lines; PR to follow.