Ticket #122 (reopened defect)

Opened 3 years ago

Last modified 4 weeks 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 3 years ago.
ticket122.jpg Download (155.1 KB) - added by mihneasim 4 weeks ago.
issue in 0.4.2

Change History

Changed 3 years ago by SnapShot

Changed 3 years ago by lincoln

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

Very nice solution, thanks SnapShot? =)

Changed 4 weeks ago by mihneasim

issue in 0.4.2

Changed 4 weeks 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

Note: See TracTickets for help on using tickets.