Ticket #124 (new enhancement)

Opened 4 years ago

Last modified 2 years ago

Add monitor selection to guake

Reported by: regit Owned by: somebody
Priority: Fix available Milestone: 0.5.0
Component: guake Version:
Keywords: Cc:

Description

The following patches add monitor selection to guake. It adds the capability to choose on which monitor guake will appear.

Attachments

0001-Add-capability-to-set-up-monitor.patch Download (137.2 KB) - added by regit 4 years ago.
0002-Limit-range-of-monitor-to-avoid-crash.patch Download (0.9 KB) - added by regit 4 years ago.
0001-Add-monitor-choice-to-UI.patch Download (4.8 KB) - added by regit 4 years ago.
0002-Add-capability-to-set-up-monitor-in-multiple-monitor.patch Download (3.1 KB) - added by regit 4 years ago.
0003-Limit-range-of-monitor-to-avoid-crash.patch Download (0.9 KB) - added by regit 4 years ago.
multi-monitor.patch Download (18.0 KB) - added by pyriku 3 years ago.

Change History

Changed 4 years ago by regit

Changed 4 years ago by regit

Changed 4 years ago by regit

Changed 4 years ago by regit

Changed 4 years ago by regit

Changed 4 years ago by SnapShot

Thanks regit for your patches. This was one of the planned features for 0.5.0.

Your patch adds new page in the preferences dialog, just as we planned to implement it (we additionally need to move other display options to the new tab page).

Two issues that I see with the patches are:

  • your second patch adds back the code commented with "# avoiding X Window system error" which was intentionally removed in one of our latest commits.
  • IMHO it is better to use ComboBox? instead of SpinButton? for monitor choice.

Our intention was to initially do some other base changes to the guake in 0.5.0 development, like moving to GtkBuilder? and UIManager, and possibly some build system changes, before adding new features, so your patches may need to wait a little until we start working on 0.5.0 (which is very soon).

Changed 4 years ago by pingou

  • priority changed from minor to Fix available

Changed 3 years ago by pyriku

Changed 3 years ago by pyriku

Hi

I created a new patch based on the posted by regit, but working on the development version.

It also improves some things:

Changed 2 years ago by mahmoudimus

Thanks for the patch pyriku! I tried it and it works beautifully :) Thank you very much for this.

Changed 2 years ago by Trisfall

Just want to say thanks - this worked for me too.

Although it would be helpful if somewhere in the FAQ or something it told you how to install a patch, because I spent over 3 hours just trying to figure that out (I'm also completely new to Linux so, yeah).

Changed 2 years ago by amog2011

Your patch adds new page in the preferences dialog, just as we planned to implement it (we additionally need to move other display options to the new tab page).

air jordan
Note: See TracTickets for help on using tickets.