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

Position on primary monitor #15

Closed
MostFrabjous opened this issue Oct 26, 2023 · 7 comments
Closed

Position on primary monitor #15

MostFrabjous opened this issue Oct 26, 2023 · 7 comments
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@MostFrabjous
Copy link

!5 was implemented and now I'm able to choose which monitor to put the terminal on. But when the configured monitor is detached an reattached, quake-terminal does not switch back to it. I'd like to see the terminal popping up at my external display when it's attached, otherwise the internal display (i.e. the active display at a time).

Pleas allow quake-terminal to be configured to the active monitor, as this is alredy handled by Gnome and should work fine.

@diegodario88 diegodario88 added enhancement New feature or request help wanted Extra attention is needed labels Oct 27, 2023
@diegodario88
Copy link
Owner

Hey @Henry78, could you please share more about your workflow? Since I don't have another display to test it out, I'll need to ask you a few questions.

You're using two monitors, with one being the secondary display where the Quake terminal configuration is located. If you unplug the secondary monitor and the Quake terminal is now displayed on your main monitor, does it still show on the main screen when you plug the external monitor back in?

How frequently does this issue occur?

It would be great if Quake-terminal could be configured to the active monitor, especially since this functionality is already Mhandled by Gnome and should work seamlessly.

Could you provide more details on this, please? Specifically, what do you mean by "allow it to be configured to the active monitor..."?

@MostFrabjous
Copy link
Author

How frequently does this issue occur?

Basically every time I detach my external (primary) display and the internal laptop screen becomes the only/last one. In this (detached) state, the configuration of quake-terminal is changed (unclear why/how), so the ID of the internal display shows up at "Monitor Screen" option. As a result the Terminal keeps opening on the internal screen after re-attaching the external display.

Could you provide more details on this, please? Specifically, what do you mean by "allow it to be configured to the active monitor..."?

Sorry, 'primary display' is term used by Gnome. You can setup any connected display as Primary (where the panel is shown). If the display is unplugged and reconnected, Gnome remembers that this was the primary.

There's another extension, I think it was ddterm, that has to option to configure the Primary Display over any specific one in the settings. That means, quake-terminal would have and additional Primary option at the "Monitor Screen" setting. This should make the Terminal to pop up at the screen that's the Primary at any moment.

@petre2dor
Copy link
Contributor

I am interested in contributing to this improvement as I would like to use this functionality myself.
As a timid first step, I opened a PR (#19) to improve the Monitor Screen field in Preferences.

@petre2dor
Copy link
Contributor

petre2dor commented Nov 5, 2023

My general plan would be to:

  1. align the language of this extension with the language used by Gnome Display Settings screen
  2. change the default behavior so that the Quake Terminal opens on the Primary Display when it does not have any saved settings for the current Display configuration
  3. whenever the user changes the preferred Display, then save that configuration and apply it whenever the same display configuration is encountered

@diegodario88 does this plan sound decent to you? If implemented you your standards, you would be willing to accept these changes? I will try to create PRs with the smallest possible changes.

Cheers!

@diegodario88
Copy link
Owner

Hey @petre2dor,

Thanks for sharing your plan for the extension. I'm on board with your proposed changes, and I appreciate your intention to keep the PRs as minimal as possible. This way, we can ensure a smoother integration.

Cheers, and thanks for your dedication to improving the extension! 🚀👍

@petre2dor
Copy link
Contributor

@Henry78 I added "Show on the primary Display" and "Show on the current Display" options in this PR: #20
If you have the time, please test my changes and give me some feedback.

Cheers!

@MostFrabjous
Copy link
Author

MostFrabjous commented Nov 9, 2023

Tested with Laptop & external monitor. Both settings, 'Primary' and 'Current' worked very well. Tests included plugging in and out the external monitor a few times.

This issue is resolved in #20

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants