Skip to content
This repository has been archived by the owner on Jan 15, 2020. It is now read-only.

bar above fullscreen windows #25

Open
phisch opened this issue Aug 19, 2016 · 7 comments
Open

bar above fullscreen windows #25

phisch opened this issue Aug 19, 2016 · 7 comments

Comments

@phisch
Copy link

phisch commented Aug 19, 2016

Using the original lemonbar repository, a bar launched with -n BAR_NAME can be rendered below fullscreen windows using xdo below -t $(xdo id -n root) $(xdo id -a BAR_NAME).

Unfortunately, this doesn't seem to work with this fork.

Do you have any idea how to fix this?

@krypt-n
Copy link
Owner

krypt-n commented Aug 19, 2016

Well, the latest 9 commits from the lemonbar repo are not included in this branch (yet), including d680ea4 and 44a708b, which seem related to the -n Option. You could try applying these to this fork and see if they help.

I will look into the problem as soon as I have time for it.

@phisch
Copy link
Author

phisch commented Aug 19, 2016

Applying the latest commits did not solve the problem. I'll report back if i find a solution.

@phisch
Copy link
Author

phisch commented Aug 19, 2016

The problem was a break which was indented incorrectly. #26 fixes this issue.
Must have tested it with the wrong code. Forget what i was saying.

@phisch
Copy link
Author

phisch commented Aug 19, 2016

This bug is even more strange than i thought.

lemonbar -n "mybar" -p -g x100& xdo below -t $(xdo id -n root) $(xdo id -a mybar) starts a bar and uses xdo to render it behind full screen windows. Sometimes the bar will be behind full screen windows, but only on the first monitor. The second and third monitors will consistently display the bar above everything else. Most of the times the first monitor will display the bar above full screen windows.

Using the original repository the bar will consistently be behind full screen windows on all monitors.

@krypt-n
Copy link
Owner

krypt-n commented Aug 19, 2016

That seems like the kind of bug that d680ea4 fixes

@phisch
Copy link
Author

phisch commented Aug 19, 2016

This definitely fixes the issue with the multiple monitors.

@phisch
Copy link
Author

phisch commented Aug 19, 2016

Well, i guess i am retarded... turns out my lemonbar script was faulty. The bar does consistently move behind full screen windows. d680ea4 didn't really make it more easy to find out what is going on, since the behavior on different monitors was different, and i used multiple monitors to test.

Merging d680ea4 will fix the multi monitor issue though.

Sorry for the confusion!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants