New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Match empty window properties (e. g. no title set) #3308

Open
exic opened this Issue Jun 11, 2018 · 7 comments

Comments

Projects
None yet
5 participants
@exic

exic commented Jun 11, 2018

I'm submitting a…

[ ] Bug
[x] Feature Request
[ ] Documentation Request
[ ] Other (Please describe in detail)

Current Behavior

We cannot check for "title unset" or something similar to match a window.

Expected Behavior

I can check for title="^$" or title=null to match weird windows.

Reproduction Instructions

Environment

Output of i3 --moreversion 2>&-:

Binary i3 version:  4.15.0.1-161-g19735b5e (2018-05-04, branch "gaps-next") © 2009 Michael Stapelberg and contributors
Running i3 version: 4.15.0.1-161-g19735b5e (2018-05-04, branch "gaps-next") (pid 3720)
Loaded i3 config: $HOME/.config/i3/config (Last modified: Mon 11 Jun 2018 10:47:18 AM CEST, 726 seconds ago)

The i3 binary you just called: /usr/bin/i3
The i3 binary you are running: i3

- Linux Distribution & Version: Ubuntu 18.04
- Are you using a compositor (e.g., xcompmgr or compton): compton

I've asked about this here, describing the use case: https://www.reddit.com/r/i3wm/comments/87qlqr/finding_criteria_for_heidisql_window_with_net_wm/

Looking at the code I found that it's simply not possible atm.

@i3bot i3bot added the enhancement label Jun 11, 2018

@Airblader

This comment has been minimized.

Member

Airblader commented Jun 11, 2018

I would argue that a client should preferably set a title. Why not fix the client to provide useful information?

@exic

This comment has been minimized.

exic commented Jun 11, 2018

That's what I'd prefer too. You might encounter software that you can't fix though and would still want to handle the window?

In my case its probably a wine issue since no windows user ever saw this weird untitled window, and I'm unsure where to file an issue to get this fixed. :-/

@Airblader Airblader added the 4.15 label Jun 11, 2018

@Airblader

This comment has been minimized.

Member

Airblader commented Jun 11, 2018

You might encounter software that you can't fix though and would still want to handle the window?

We officially only support open-source software, so I'd argue that this case should never reasonably happen (and even in closed-source software you can still contact the developers behind it).

Regarding your current problem with wine: does this affect all windows opened by wine or is it just specific programs launched through wine (i.e., does wine generally not set titles)?

I'm not too opposed to the proposed idea here, but I would like to probe the situation a bit more before making a call.

@exic

This comment has been minimized.

exic commented Jun 12, 2018

It only happened using HeidiSQL so far, other windows are fine.

These are the xprop outputs (as in above mentioned post):
Crappy useless window: https://pastebin.com/nmsthuLm
Main window: https://pastebin.com/hu4JZ6TP

To fix my issue it actually doesn't need to be "match empty window properties", if I could for example match _NET_WM_STATE_SKIP_TASKBAR to put it on scratchpad, that would work fine too.

I agree that I should check wine/heidisql to get a permanent fix for the problem. It's just that very few people use HeidiSQL with Wine and a tiling manager, where this hidden window gets in the way. I've been on AwesomeWM before. There, it was an only one pixel large window. Here's a screenshot how it looks on i3, the black box is what I'm talking about:

weird

@archenemies

This comment has been minimized.

archenemies commented Sep 21, 2018

Well maybe I'll try asking this one over here: "is there a pattern to 'match every window'? Even if it doesn't have a title or a class? Maybe that would be a good thing to have."

@orestisf1993

This comment has been minimized.

Member

orestisf1993 commented Sep 21, 2018

[workspace=.*] should work.

@archenemies

This comment has been minimized.

archenemies commented Sep 21, 2018

Thank you, that indeed works for me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment