Ticket #201 (new defect)

Opened 5 months ago

Last modified 2 months ago

Terminal appears on wrong monitor if primary screen is right of secondary one

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

Description

get_final_window_rect, assumes window_rect.x is 0, which may not always be the case. If the starting X coordinate is > 0 it gets reset to 0 by the calculations. The X coordinates should be added to, not set directly:

if halignment == ALIGN_CENTER:

window_rect.x += (total_width - window_rect.width) / 2

if halignment == ALIGN_LEFT:

window_rect_x += 0

elif halignment == ALIGN_RIGHT:

window_rect.x += total_width - window_rect.width

Attachments

Change History

Changed 5 months ago by maweaver

Sorry I created a typo in my fix. The only change should have been changing the '=' to '+='s, and the fourth line should read

window_rect.x += 0

not

window_rect_x += 0

Changed 5 months ago by pingou

It would be nice if you could provide a git format-patch for your patch :-)

Thanks for contributing

Changed 5 months ago by maweaver

Hopefully I did that right, I'm not too familiar with git.

Changed 2 months ago by emanuelez

I tried changing the file manually on my machine according to the patch and it seems to work fine. Release it soon! :)

Note: See TracTickets for help on using tickets.