krypt-n / bar

Fork of bar ain't recursive - A lightweight xcb based bar. This is a hot mess, do not use
MIT License
174 stars 46 forks source link

bar above fullscreen windows #25

Open phisch opened 8 years ago

phisch commented 8 years ago

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 commented 8 years ago

Well, the latest 9 commits from the lemonbar repo are not included in this branch (yet), including d680ea4256637bc89d59342cf6ac6c6f5fe62dec and 44a708b7a4a266ca8e91eb8a6a1b34ffa4a7a164, 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 commented 8 years ago

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

phisch commented 8 years ago

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 commented 8 years ago

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 commented 8 years ago

That seems like the kind of bug that d680ea4256637bc89d59342cf6ac6c6f5fe62dec fixes

phisch commented 8 years ago

This definitely fixes the issue with the multiple monitors.

phisch commented 8 years ago

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!