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

Multi-monitor support #68

Closed
wants to merge 1 commit into from
Closed

Multi-monitor support #68

wants to merge 1 commit into from

Conversation

zeorin
Copy link

@zeorin zeorin commented Jan 5, 2017

I used guimeira/i3lock-fancy-multimonitor as inspiration (because I wasn't satisfied with it--couldn't change the text), and changed the lock script to support multiple monitors. Fixes #10.

Should also work for more than 2 monitors.

@AladW
Copy link
Collaborator

AladW commented Feb 2, 2017

Should be split to several, overseeable commits. The general approach also needs improving if this is to be merged to the main branch, such as no keeping xrandr output as a big string and parsing it with bash_rematch and no extraneous use of eval either.

@michaelplews
Copy link

@zeorin I'm using this pull request for my day to day and it's been working great. I wanted to adjust the code to where the overlay can only be applied to one monitor, perhaps with a flag and option such as -m=0 for the primary display or -m=1 for the secondary display.

I was looking at your code to find where you iterate over the list of attached monitors but couldn't see it, could you point me in the right direction?

@AladW
Copy link
Collaborator

AladW commented Apr 22, 2017

No response, closing.

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

Successfully merging this pull request may close these issues.

3 participants