Window list add to panel launchers #921

Closed
wants to merge 3 commits into from

4 participants

@dalcde

Add an "Add/Remove to panel launchers" item in the menu

Code cleanup included

@AlbertJP
Linux Mint member

@dalcde I can't move panel launchers around anymore with your code.

Edit. I had "panel launchers draggable" turned off in cinnamon settings. Your code works fine.

@AlbertJP
Linux Mint member

Thanks for indenting the file correctly, but does that really belong in a pull request about panel launchers?

@dalcde

I'm just afraid that if I do it in a separate pull request, the merges might conflict. It is not much work, so if this doesn't get merged, I can re-do it in a separate pull request.

@AlbertJP
Linux Mint member

The indent/whitespace fixes are on other lines of the file, meaning they should merge correctly if in a separate pull request. However, I don't know how clem thinks about this.

@clefebvre
Linux Mint member

A bit overkill. That feature belongs to the panel launchers themselves, to drag and drop and to the menu.

Closing the pull request.

@clefebvre clefebvre closed this Aug 29, 2012
@LiamDawe

I disagree clef i think its useful and Unity/KDE both enable you to add a launcher from an open window.

@dalcde dalcde deleted the dalcde:window-list-add-to-panel-launchers branch Dec 21, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment