Closed
Bug 266513
Opened 20 years ago
Closed 20 years ago
Dependent windows don't persist the correct position
Categories
(Core Graveyard :: GFX: Gtk, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: neil, Assigned: roc)
References
()
Details
(Keywords: regression)
Attachments
(1 file)
1.19 KB,
patch
|
blizzard
:
review+
blizzard
:
superreview+
|
Details | Diff | Splinter Review |
Dependent windows, such as the Open Location dialog, normally perist their position relative to their parent. However since the given checkin the saved positions are offset apparently by the size of the window decorations (on my system, by 6 pixels horizontally and 20 vertically). I verified that backing out that part of the patch restores the behaviour.
Comment 1•20 years ago
|
||
That part of which patch?
Assignee | ||
Comment 2•20 years ago
|
||
Is this fixed by the change to fix the GTK1 warnings that I checked in yesterday?
Reporter | ||
Comment 3•20 years ago
|
||
(In reply to comment #2) >Is this fixed by the change to fix the GTK1 warnings that I checked in yesterday? No it is not. However, popups no longer deactivate their parent :-)
Assignee | ||
Comment 4•20 years ago
|
||
Reverting just the putative optimization to GetWindowPos fixes the bug.
Assignee: blizzard → roc
Status: NEW → ASSIGNED
Assignee | ||
Updated•20 years ago
|
Attachment #164133 -
Flags: superreview?(blizzard)
Attachment #164133 -
Flags: review?(blizzard)
Reporter | ||
Comment 5•20 years ago
|
||
Comment on attachment 164133 [details] [diff] [review] fix Thanks for the fix :-)
Updated•20 years ago
|
Attachment #164133 -
Flags: superreview?(blizzard)
Attachment #164133 -
Flags: superreview+
Attachment #164133 -
Flags: review?(blizzard)
Attachment #164133 -
Flags: review+
Reporter | ||
Comment 6•20 years ago
|
||
/me hopes roc doesn't mind if I check this in for him ;-)
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 7•20 years ago
|
||
thanks! sorry
Updated•16 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•