Ticket #122 (closed defect: fixed)

Opened 4 years ago

Last modified 15 months ago

Dual-monitor problem.

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

Description

I think this is not related to the other dual monitor problem that is targeted for 0.5.0.

This patch fixes the problem that apears when default monitor is not left monitor. Guake always uses the left monitor (position 0,0) but gets the width of the default monitor which can be on right. More details in commit message ;)

I have reworked the get_final_window_size function but there is one part of the code commented like "avoiding X Window system error" that I am not sure what's the purpose. It looks ilogical to me, it get's the height of the screen in pixels and then compares it to the height configured by the user which is in percents.

I don't think it makes sense to compare percents with pixels, something like if 60 > 1050. And even if we can compare them I can't see what we would achive with that. Let's say screen size is 50px what will we get is that 60% is bigger than 50px and then we will make the height equal to screen height so height will be 50%.

I think this part of the code is remaining from some older version where guake kept the height preference in pixels, and then it made sense to compare if the height specified by the user is bigger then monitor height.

Attachments

0001-Fix-dual-monitor-problem.patch Download (3.5 KB) - added by SnapShot 4 years ago.
ticket122.jpg Download (155.1 KB) - added by mihneasim 17 months ago.
issue in 0.4.2

Change History

Changed 4 years ago by SnapShot

Changed 4 years ago by lincoln

  • status changed from new to closed
  • resolution set to fixed

Very nice solution, thanks SnapShot? =)

Changed 17 months ago by mihneasim

issue in 0.4.2

Changed 17 months ago by mihneasim

  • status changed from closed to reopened
  • resolution fixed deleted
  • milestone changed from 0.4.0 to 0.5.0

This issue is still present in 0.4.2

Changed 15 months ago by ulidtko

  • status changed from reopened to closed
  • resolution set to fixed

This patch seems to be applied already. I'm closing the ticket.

Note: See TracTickets for help on using tickets.