Closed
Bug 11491
Opened 25 years ago
Closed 25 years ago
Background color not covering all backround region
Categories
(Core :: Layout, defect, P3)
Core
Layout
Tracking
()
VERIFIED
FIXED
M16
People
(Reporter: salwan.searty, Assigned: attinasi)
References
()
Details
Attachments
(3 files)
In Viewer: when automatically changing the background color from a function in
JavaScript, only a part of the background region changes color and not the
whole window (please see provided URL as an example.)
http://www.chalcedony.com/javascript/scripts/chap03/script15.html
V -- Since I'm out this week, can you please take a quick look at this? It's
likely a bug in the view system, but since JS is involved I wanted to make it
was well formed.
Comment 2•25 years ago
|
||
Updated•25 years ago
|
Whiteboard: [TESTCASE] proper area needs to get invalidated
Comment 3•25 years ago
|
||
This looks like a paint/invalidate problem. If you force a localized repaint
(drag a window over the affected area) the correct colors show up as the
localized area repaints. Tested on 19990809 build on Win98.
Updated•25 years ago
|
Assignee: vidur → troy
Component: Viewer App → Layout
OS: Linux → All
Hardware: PC → All
Comment 4•25 years ago
|
||
Moving this to layout. This looks like repaint problems relating to the BODY
background fixup rule (to make it the whole canvas). It's only repainting
within the border edge of the BODY element (note carried out margin of H1).
EXPECTED: Entire canvas should rotate through colors.
OBSERVED: Only the header (out to near the side edges) is rotating through
colors, except when a repaint is forced by dragging a window over it or
something.
TESTED ON:
* Linux viewer 1999-09-03-11-M11
* (Win98 1999-08-09 as mentioned above)
Marking All/All.
Peter, I guess we're not invalidating the HTML window as well when the
background color is changed?
Comment 6•25 years ago
|
||
Moving non-beta 1 items to M15
Comment 7•25 years ago
|
||
Reassigning peterl's bugs to myself.
Comment 8•25 years ago
|
||
Accepting peterl's bugs that have a Target Milestone
Comment 10•25 years ago
|
||
Pushing my M15 bugs to M16
Updated•25 years ago
|
QA Contact: petersen → chrisd
Comment 11•25 years ago
|
||
Bulk moving [testcase] code to new testcase keyword. Sorry for the spam!
Keywords: testcase
Comment 12•25 years ago
|
||
Another good candidate for the bug contest this week...
Could someone verify whether it's not a dup of bug 1886 and bug 24684? Thanks.
Whiteboard: [TESTCASE] proper area needs to get invalidated → need help [TESTCASE] proper area needs to get invalidated
Comment 13•25 years ago
|
||
Replacing "need help" in the Status Whiteboard with the 'helpwanted' keyword.
Keywords: helpwanted
Whiteboard: need help [TESTCASE] proper area needs to get invalidated → [TESTCASE] proper area needs to get invalidated
Comment 14•25 years ago
|
||
Comment 15•25 years ago
|
||
This is a major ugliness for Composer -- When we set the background color on a
page, which sets BGCOLOR on the BODY tag, only the frames behind the lines of
selected text are colored. Resizing the window triggers the proper repaint.
Attached is a test file that might help.
Blocks: 21410
Updated•25 years ago
|
QA Contact: chrisd → petersen
Comment 16•25 years ago
|
||
adding myself to Cc: list..
Comment 17•25 years ago
|
||
reassign to attinasi. Please look at this per rickg's request.
Assignee: pierre → attinasi
Status: ASSIGNED → NEW
Comment 18•25 years ago
|
||
*** Bug 28494 has been marked as a duplicate of this bug. ***
Assignee | ||
Updated•25 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Comment 19•25 years ago
|
||
*** Bug 1886 has been marked as a duplicate of this bug. ***
Assignee | ||
Comment 20•25 years ago
|
||
I have a fix for this: essentially I just update all of the views if the BODY is
restyled. Not the most elegant solution, but all I could think of. I'm
requesting input from troy on a potentially better approach.
Keywords: helpwanted,
testcase
Whiteboard: [TESTCASE] proper area needs to get invalidated
Comment 21•25 years ago
|
||
*** Bug 30429 has been marked as a duplicate of this bug. ***
Comment 22•25 years ago
|
||
*** Bug 33251 has been marked as a duplicate of this bug. ***
Comment 23•25 years ago
|
||
Comment 24•25 years ago
|
||
*** Bug 36527 has been marked as a duplicate of this bug. ***
Comment 25•25 years ago
|
||
Hmm, that last dup is a restricted bug... weird.
A similar problem exists for background images. Bug 34560.
Assignee | ||
Comment 26•25 years ago
|
||
*** Bug 34560 has been marked as a duplicate of this bug. ***
Assignee | ||
Comment 27•25 years ago
|
||
Fix is finally ready: in CSSFrameConstructor::AttributeChanged we check it the
attribute being changed is the background. If it is, we find the canvas frame
and call its (new) AttributeChanged method. That method then checks if the
attribute being changed is the bgcolor or the background and invalidates the
frame if it is. This way we get the correct region invaildated, and it happens
whenever the bgcolor or image is changed, through the BODY tag or through
script. I'm getting a checkin ready now...
Comment 28•25 years ago
|
||
When you mark this bug fixed, could check whether it has any effect on bug 33387?
Please put the result under 33387. Thanks.
Assignee | ||
Comment 29•25 years ago
|
||
Finally, fixed.
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Comment 30•25 years ago
|
||
You need to log in
before you can comment on or make changes to this bug.
Description
•