Skip to content
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

Not displaying properly on dual monitors #10

Open
sbmueller opened this issue Dec 14, 2015 · 17 comments
Open

Not displaying properly on dual monitors #10

sbmueller opened this issue Dec 14, 2015 · 17 comments
Assignees

Comments

@sbmueller
Copy link

It seems that your script is not working for dual monitors. The lock is displayed in the gap between the monitors while the type feedback indicator is displayed twice in the center of each screen. Would be great to have support for dual monitors!

@meskarune meskarune added the bug label Dec 30, 2015
@meskarune meskarune self-assigned this Dec 30, 2015
@meskarune
Copy link
Owner

I don't have dual monitors to test on, but I'll see what I can do. Thanks for reporting this!

@insanebits
Copy link

Can confirm, would be nice to have text on both screens

@ghost
Copy link

ghost commented Jan 17, 2016

I experience this bug as well, I'll see if I can fix it on my own and then submit a pull request.

edit: PR submitted.

@sbmueller
Copy link
Author

I think somone has made effort to solve this: https://github.com/guimeira/i3lock-fancy-multimonitor so we can close this for now.

@meskarune
Copy link
Owner

I have a fix for this in the works, so I'm going to go ahead and reopen, but thanks for the link! I'm happy other people have been able to use my idea/code 💃

@meskarune meskarune reopened this Apr 3, 2016
@meskarune
Copy link
Owner

Ok, preliminary dual monitor support is up in the dualmonitor branch. \o/

I still need to fix up dark/light detection so it works for each screen and make all the text take up the same amount of space so I can center it for all languages. Unfortunately this could cause some line wrapping for really long text, but that's very unlikely to happen unless someone edits the text. (or I add a custom text option...)

@meskarune
Copy link
Owner

In the dual monitor branch I switched to token light lock icon which has a shadow behind it and shows up on dark and light backgrounds. I am working on making text with light grey fill and a dark stroke and/or shadow so it will also stand out on both dark/light backgrounds. This reduces the run time as there isn't a check for light/dark. I need to work on text centering more. I think there is a way to detect pixel length of text so the centering is more accurate.

@meskarune
Copy link
Owner

Here are screenshots for the dual monitors script with Open-Sans-Light set as the font. I'm using text shadow to make the text readable for light/dark and using a grey icon with shadow. Any suggestions/recommendations would be appreciated.

dark background preview

light background preview

@sbmueller
Copy link
Author

Looks awesome 👐

@jjmcdn
Copy link
Contributor

jjmcdn commented May 12, 2016

Just tested this branch out on my dual monitor setup (I don't use it very often or I would've tried it out sooner) and what's in the current dualmonitors works great for me.

@leshow
Copy link

leshow commented Jul 24, 2016

has this been merged into master yet?

@meskarune
Copy link
Owner

Not yet @leshow there are still some things I need to figure out. Primarily if there is some way to do color detection or if I should drop that feature? I'm not sure what the right way to handle it is. I really need to update the dual monitors branch with features added in the main branch.

@zeorin
Copy link

zeorin commented Jan 5, 2017

My pull request does colour detection.

@szalishchuk
Copy link

@meskarune any chance for this being merged into master this year? I'm not sure whether I should install master onto a laptop and manually dualmonitors branch onto my desktops, or move forward with a fork. What would be your recommendation? Modifying PKGBUILD to install from custom branches on multiple machines is kinda painful.

@bvn13
Copy link

bvn13 commented Jun 25, 2020

Hi! @meskarune what kind of help do you need? I have two monitors and I want to help.

@bvn13
Copy link

bvn13 commented Jun 25, 2020

I have checked https://github.com/meskarune/i3lock-fancy/pull/87/files PR - it works fine.

@bvn13
Copy link

bvn13 commented Jun 25, 2020

@patrick96 Your version for dual monitors works fine. I have only one question. Script beeps twice on executing. Do you have any thoughts what may the reason of it?

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

No branches or pull requests

8 participants