jake-phy / WindowIconList

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

Super+number shortcut opens new window every time #34

Closed iknowcss closed 10 years ago

iknowcss commented 11 years ago

First off, thank you for all of your hard work. I really like this applet and I appreciate what you have done so far.

When I use the Super+number shortcut that you added in version 2.0, I get some undesired behavior. I will use the Terminal application as my example.

I have the terminal as one of my pinned favorites. To open it, I type Super+2. When I type Super+2 the first time, I get a new terminal window. When I click the terminal icon group several times, the window minimizes and maximizes as expected. However, when I type Super+2, I always get a new instance of the terminal application. I want to get the same terminal window I opened before.

Now I know that things can get a bit hairy when there are multiple terminal windows open already. I think the best behavior would be for the group preview to pop up and allow the user to type Super+number to cycle through the different windows (I believe this is what Win7 does).

Is this possible?

Thanks again! Cody

iknowcss commented 11 years ago

Update: I believe I have worked out a small snippet of code that will address part of my suggestion. I have created a pull request for the change.

https://github.com/jake-phy/WindowIconList/pull/35

Thanks! Cody

jhasse commented 10 years ago

The remaining part of your suggestion is now a duplicate of #32, since clicking on a window and using Super+Number now behave the same ;)

aikeru commented 10 years ago

This seems to be resolved. Can this issue be closed?

jake-phy commented 10 years ago

Well I don't have the cycling through the windows implemented I don't think.

iknowcss commented 10 years ago

When I wrote this up I just wanted super+number not to open a new window. I suggested alternative behavior as an example. If that's an important feature I recommend creating a separate issue. Either way I consider this issue resolved.

jake-phy commented 10 years ago

Cool Thanks.