search

richardgv / skippy-xd

A full-screen Exposé-style standalone task switcher for X11.
GNU General Public License v2.0
342 stars 78 forks source link

Doesn't work with i3wm #14

Closed shellkr closed 11 years ago

shellkr commented 11 years ago

I don't know if this should count as a feature request or bug.. I installed skippy-xd package from Arch Linux AUR to see if it would work in i3wm. It didn't. It gave me...

FATAL: WM not NETWM or GNOME WM Spec compliant.

With skippy-xd-git witch is currently built from the 20.e8e3638 commit gave..

config_get_double(general, updateFreq, 10,000000): Value "10.0" is not a valid floating-point number. main(): WM not NETWM or GNOME WM Spec compliant. Troubles ahead. running once then quitting... skippy_run(): No client windows found.

So I guess it is not compatible yet. Will it be in the future?

richardgv commented 11 years ago

Let's say this WM is giving me tons of troubles in compton that I do not wish to touch it anymore...

I will check what I could do about this. Give me some time, please.

If you are using a European locale (where comma is used instead of dot in a floating-point number), I will fix the config_get_double(general, updateFreq, 10,000000): Value "10.0" is not a valid floating-point number. issue soon.

shellkr commented 11 years ago

Cool, no worries... Let me know if I can help. And yes, I am using the Swedish locale (sv_SE.utf8).

richardgv commented 11 years ago

I got skippy-xd reading window list correctly under i3, but the operation is incorrect presumably because i3 is grabbing the keyboard... Tomorrow is a workday, so... Perhaps you would have to wait until next weekend, unfortunately.

shellkr commented 11 years ago

That was fast working! Expected it to take longer than that. As I said no worries. Just let me know when you want it tested. ;)

richardgv commented 11 years ago

Weird, it somehow is working today... Anyway, my work is inside this patch now < https://gist.github.com/richardgv/5696281 >. I don't really have time to check everything I changed, so expect bugs.

shellkr commented 11 years ago

I got it working after I applied the patch. The output did not render right though. Some of my terminals look right but not all and not Firefox. It gives an output little like if you connect a video camera to a TV and then point the camera on the TV. A kind of optical feedback.

On this screen I am on workspace one which has three terminals. As you can see it's only the terminals on that workspace that looks right. http://i.imgur.com/NWED6C2.jpg

If I switch to workspace two with which only contains Firefox. Here Firefox looks good but not any of the terminals which are on other workspaces. http://i.imgur.com/cDzAJvD.jpg

So it probably has with the workspaces somehow. As applications on the workspace I currently am on work good but not the one on other workspaces.

richardgv commented 11 years ago

Sorry for the problem. I guess I understood what is causing this. I wasn't checking for unmapped windows, and i3 isn't providing us hints whether a window is on current desktop. I will take care of this when I'm back home.

richardgv commented 11 years ago

I've updated the patch. Hopefully it will work for you. < https://gist.github.com/richardgv/5696281 >

shellkr commented 11 years ago

Working nice with the last patch. No more "optical feedback" issues. Are it supposed to just show the applications of the current workspace or can it do all?

Will you include this in the master branch soon? Just so I can contact the maintainer of the package in AUR to update it.

richardgv commented 11 years ago

Are it supposed to just show the applications of the current workspace or can it do all?

Current desktop. Because windows on other desktops are typically unmapped (somewhat like, iconified), and we can't get content of unmapped windows (which is why you see so much garbage).

Will you include this in the master branch soon? Just so I can contact the maintainer of the package in AUR to update it.

Thanks, but the problem is I changed too much in the WIP patch and it needs some review. And some features I planned to add aren't finished yet. I got a job and could only work in weekends and sometimes at night, so... I guess it won't be really fast, sorry...

shellkr commented 11 years ago

Too bad. It could have been a nice feature if possible.

I hear you. I have children and the times when I could sit many hours in a row is long gone. ;) I find it admirable how fast you fixed this problem. So kudos to that. For me it is fixed anyway. Just let me know if you need me to do more testing further along.