Ticket #124 (new enhancement)

Opened 3 years ago

Last modified 11 months 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 3 years ago.
0002-Limit-range-of-monitor-to-avoid-crash.patch Download (0.9 KB) - added by regit 3 years ago.
0001-Add-monitor-choice-to-UI.patch Download (4.8 KB) - added by regit 3 years ago.
0002-Add-capability-to-set-up-monitor-in-multiple-monitor.patch Download (3.1 KB) - added by regit 3 years ago.
0003-Limit-range-of-monitor-to-avoid-crash.patch Download (0.9 KB) - added by regit 3 years ago.
multi-monitor.patch Download (18.0 KB) - added by pyriku 18 months ago.

Change History

Changed 3 years ago by regit

Changed 3 years ago by regit

Changed 3 years ago by regit

Changed 3 years ago by regit

Changed 3 years ago by regit

Changed 3 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 2 years ago by pingou

  • priority changed from minor to Fix available

Changed 18 months ago by pyriku

Changed 18 months 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 13 months ago by mahmoudimus

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

Changed 12 months 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 11 months 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.