Closed
Bug 31713
Opened 25 years ago
Closed 25 years ago
Offscreen form element's render in the upper left corner of the content area
Categories
(Core :: XUL, defect, P1)
Core
XUL
Tracking
()
M16
People
(Reporter: kmcclusk, Assigned: saari)
References
()
Details
(Whiteboard: [PDT-])
Form elements (text area's, listboxes) render their contents at (0,0) if
they are initially offscreen. This is a regression caused by the conditioning
code used to fix bug 26834.
It can be fixed, by adding a conditional around the code used to fix bug 26834
as follows:
if (mIsTopWidgetWindow) {
if(dc) {
if (::GetDeviceCaps(dc, TECHNOLOGY) == DT_RASDISPLAY)
{
RECT workArea;
::SystemParametersInfo(SPI_GETWORKAREA, 0, &workArea, 0);
if(mBounds.x >= workArea.right)
aX = mBounds.x = workArea.left;
if(mBounds.y >= workArea.bottom)
aY = mBounds.y = workArea.top;
}
::ReleaseDC(mWnd, dc);
}
Not sure if adding this conditional causes bug 26834 to reappea
Reporter | ||
Comment 1•25 years ago
|
||
Marking PDT+, beta1. Approved (Rickg)
Keywords: beta1
Whiteboard: PDT+
Comment 2•25 years ago
|
||
I cheked in the fix for this this morning into the tip, saari you need to get it
into the branch asap.
Comment 3•25 years ago
|
||
reassigning to pink for checkin (saari is out with what sounds like a migraine)
Assignee: saari → pinkerton
Priority: P3 → P1
Target Milestone: M14
Comment 4•25 years ago
|
||
why me? why can't rod check this in on the branch?
Assignee: pinkerton → rods
Updated•25 years ago
|
Whiteboard: PDT+ → PDT+ will verify with 03/15 builds
Comment 8•25 years ago
|
||
Okay, this (mostly) works for the _commercial_ beta builds:
2000031506 win98
2000031507 macos 8.6
2000031512 linux rh6.0
... with one exception -- the native scrollbar(s) for offscreen <select>
listboxes will momentarily paint in the top left corner of the canvas on win98
and linux (mac appears to be OK). This is particularly bad on linux.
Does this bug cover the native scrollbars (ref. bug #18895)? (shouldn't it?)
Whiteboard: PDT+ will verify with 03/15 builds → PDT+ -- question for rods
Comment 9•25 years ago
|
||
This was a Windows bug only. If you are still seeing this, then reopen it and
reassign it to saari who is the original author of the regression.
Comment 10•25 years ago
|
||
Okay, reopening and passing to saari.
Note: this is mostly a linux problem, it's less severe on win32, and I don't
see it at all on mac. See also http://bugzilla.mozilla.org/query.cgi on linux
for another example of this problem.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment 11•25 years ago
|
||
-> saari
Assignee: rods → saari
Status: REOPENED → NEW
Whiteboard: PDT+ -- question for rods → PDT+
Comment 12•25 years ago
|
||
This should not cover native scrollbars, as we do not support them,. If that's
the only problem you're seeing then this bug should probably be closed. In any
case, this should no longer be a showstopper for PR1 (which has no native
scrollbars); removing PDT+ for reconsideration.
Whiteboard: PDT+
Comment 13•25 years ago
|
||
Actually, the scrollbars for HTML <select> widgets are still native (see bug
#18895) and it is specifically these scrollbars that flash as the page is
loaded. No other widgets are affected.
Comment 15•25 years ago
|
||
The flashing gray boxes (where the native scrollbars first appear) is a bug I
think waqar already has which will be fixed by GfxLists, if that is all this bug
is about then it is a dup.
Assignee | ||
Updated•25 years ago
|
Status: NEW → ASSIGNED
Target Milestone: M14 → M15
Assignee | ||
Comment 17•25 years ago
|
||
Okay, sounds like a dup then.
Comment 18•25 years ago
|
||
I believe that the dup of this bug is bug #19608 -- "flashing gray boxes around
select form controls".
*** This bug has been marked as a duplicate of 19608 ***
Status: ASSIGNED → RESOLVED
Closed: 25 years ago → 25 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•