collinss / Cinnamon-Sticky-Notes

A sticky note applet for the Cinnamon DE
6 stars 2 forks source link

Popup menu item "Remove this note" isn't shown #5

Closed yrtimiD closed 7 years ago

yrtimiD commented 9 years ago

Cinnamon 2.2.16 Sticky Notes 1.2

When performing right click on note only 3 menu items are shown:

As result - no way to delete note at all.

yrtimiD commented 9 years ago

I'm not familiar with cinnamon applets programming, but saw that in buildMenu() function two types of menu items are used: PopupMenu.PopupMenuItem and Applet.MenuItem

All "invisible" items are of Applet.MenuItem type. I replaced "Remove" menu item to be:

    let remove = new PopupMenu.PopupMenuItem("Remove this note");
    this.menu.addMenuItem(remove);
    remove.connect("activate", Lang.bind(this, function() {
        this.emit("destroy", this);
    }));

and this seems to work.

vdeville commented 8 years ago

Thank, add this after copy menu entry and work !

collinss commented 7 years ago

I believe this issue is now fixed.