Skip to content

Added mirror of displays #22

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

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Added mirror of displays #22

wants to merge 3 commits into from

Conversation

desdic
Copy link

@desdic desdic commented Aug 12, 2019

Small patch for creating mirror of displays

doc/grobi.conf Outdated
outputs_present: [eDP1, DP2]
outputs_same_as:
- output: DP2
same_as: eDP1
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hm, this rule looks odd. You are trying to configure two displays showing the same content, right? How about we add configure_mirror instead of using configure_row and outputs_same_as? This would simplify the rule a lot:

outputs_present: [eDP1, DP2]
configure_mirror: [eDP1, DP2]

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like the name configure_mirror :) but if I add them to a list (like outputs_present) it would indicate that you could specify more than 2 displays but that is not really possible when mirroring. I can do the change if you still prefer it to be like output_present

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hm, are you sure that's the case for all graphic cards? I can imagine it's a common limitation (only two outputs in mirror mode), but I guess there are devices where three is also possible.

I'd prefer configure_mirror, you can then remove the code for outputs_same_as, which makes the configuration simpler :)

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll have a look at it later and do a fix :)

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed and I rebased it the newly added features

@seqizz
Copy link

seqizz commented Sep 9, 2021

Sorry for bumping, is there a way to mirror the screens on currently released version, or should I use this MR?

@DamienCassou
Copy link

I would also appreciate this feature

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.

4 participants