Opened 15 years ago

Last modified 14 years ago

#6343 closed defect

[Patch needs review] Relation windows don't remember their last position — at Version 5

Reported by: flaimo Owned by: Don-vip
Priority: normal Milestone:
Component: Core Version: tested
Keywords: Windows Cc:

Description (last modified by Don-vip)

I am working with two monitors on my windows machine. Whenever I open up a window for a relation it pops up on the first screen, no matter where it was located the last time it was closed. this is cumbersome to work with, since I always have to drag the windows over to the second screen.

win 7, build 4064

Change History (6)

comment:1 by flaimo, 15 years ago

Summary: Reation windows don't remeber their last positionRelation windows don't remember their last position

comment:2 by malenki, 15 years ago

Can't confirm this one for my setup:
Debian sid,
LXDE with OpenBox

java version "1.6.0_25"
Java(TM) SE Runtime Environment (build 1.6.0_25-b06)
Java HotSpot(TM) 64-Bit Server VM (build 20.0-b11, mixed mode)

comment:3 by flaimo, 15 years ago

can someone else with a win 7 system try this too?

in reply to:  3 comment:4 by Don-vip, 14 years ago

Keywords: Windows added
Owner: changed from team to Don-vip
Status: newassigned

Replying to flaimo:

can someone else with a win 7 system try this too?

Same with me on r4475. I'll look into it.

comment:5 by Don-vip, 14 years ago

Description: modified (diff)
Summary: Relation windows don't remember their last position[Patch needs review] Relation windows don't remember their last position

OK, here's a working patch for Windows 7. We're in stabilization state, so I won't commit something that could break display on other systems until the next tested version is out. In the meanwhile, I'd like to know if someone can test it under Linux and Mac OS X. And some comments on the approach: should we have two applySafe methods, like in this patch (one for a single screen, one for mullti screens), or should we just modify the existing applySafe method to allow multi screen display ? (I'm afraid of possible side effects as this method is called very often).

by Don-vip, 14 years ago

Attachment: patch.diff added
Note: See TracTickets for help on using tickets.