Closed
Bug 107143
Opened 23 years ago
Closed 23 years ago
flashing when changing pages with xlib toolkit
Categories
(Core :: XUL, enhancement, P2)
Tracking
()
RESOLVED
FIXED
mozilla0.9.7
People
(Reporter: gonufer, Assigned: roland.mainz)
Details
Attachments
(1 file)
2.23 KB,
patch
|
roland.mainz
:
review+
|
Details | Diff | Splinter Review |
Every time a "tab" is changed or a new page is fetched and drawn there's
a flash as the entire page is cleared to white then the new contents drawn.
This doesn't happen with gtk.
Roland says:
Maybe it's the issue that I am not setting a |None|-Pixmap as
background (widget/src/gtk/ code does that to hide that it makes ugly
position stunts _after_ mapping the window) - causing that the window
background is filled with the bg color (white) first.
and asked me to file a bug.
Comment 1•23 years ago
|
||
does xlib double-buffer?
Assignee | ||
Comment 2•23 years ago
|
||
Uhm... what do you mean with "double buffer" ?
Xlib stuff is just like GTK+ - but without using the GDK/GTK+ wrappers. This
stuff is just pure bloat.
Assignee | ||
Comment 3•23 years ago
|
||
Taking...
Assignee: hyatt → Roland.Mainz
Status: UNCONFIRMED → NEW
Ever confirmed: true
Assignee | ||
Comment 4•23 years ago
|
||
Accepting bug, setting milestone etc. etc.
Patch follows...
Severity: normal → enhancement
Status: NEW → ASSIGNED
OS: SunOS → Linux
Priority: -- → P2
Hardware: Sun → All
Target Milestone: --- → mozilla0.9.7
Assignee | ||
Comment 5•23 years ago
|
||
Assignee | ||
Comment 6•23 years ago
|
||
I just added a "workaround" via setting the background pixmap of the windows to
|None| whcih disables the fill with the background pixel.
However, the real solution would be to avoid that windows get resized/moved
after they've been mapped. The issue hides AFAIK somewhere in the XP widget code
- but I am not much willing to touch this as the review/superreview/moa/approval
etc. etc. takes too much time.
The current solution just does the same as the widget/src/gtk/ code does: Hide
the issue... ;-(
Requesting r= (patched files not part of build, no sr= required) ...
Assignee | ||
Updated•23 years ago
|
Attachment #56924 -
Flags: review+
Assignee | ||
Comment 8•23 years ago
|
||
Patch checked-in...
gonufer@yahoo.com, wanna VERIFY please ? :-)
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•