Summary: | [Qt] Issue with page display using QtWebKit 2.1 | ||
---|---|---|---|
Product: | WebKit | Reporter: | nokiabugz |
Component: | Layout and Rendering | Assignee: | Jay Tucker <jay.tucker> |
Status: | RESOLVED FIXED | ||
Severity: | Critical | CC: | ademar, benjamin, ext-prasad.adimulam, hausmann, jturcotte, markus, ostap73, suresh.voruganti, timothy.lu |
Priority: | P2 | Keywords: | PlatformOnly, Qt |
Version: | 528+ (Nightly build) | ||
Hardware: | S60 Hardware | ||
OS: | S60 3rd edition |
Description
nokiabugz
2010-07-27 06:40:31 PDT
This is critical issue that is blocking Webkit 2.1 testing in Browser. Which Qt version are you using? I think this issue is solved by getting latest Qt package. This is not reproducible any more with latest Qt version. so resolving the bug. I think still issue is seen interminently. Does any one know which error from Qt has fixed this issue? (In reply to comment #5) I don't know how Qt handles platform color palettes on Symbian but this feels like bug #22123 to me. Did you try changing the phone's theme to see if it changes the behavior? Re opening the issue as it is blocking the testing. (In reply to comment #6) > (In reply to comment #5) > > I don't know how Qt handles platform color palettes on Symbian Yes, it's theme issue. There is workaround for this in QGraphicsWebView based classes: QPalette pal = palette(); pal.setColor(QPalette::ButtonText, Qt::black); pal.setColor(QPalette::Base, Qt::white); setPalette(pal); page()->setPalette(pal); This also fixes the problem of white radio/checkbox buttons in case of phone themes with dark background. > but this feels like bug #22123 to me. Not exactly. > Did you try changing the phone's theme to see if it changes the behavior? Previously I've tried to figure out, should QGraphicsWebView by default pick up theme colors. Those colors can conflict with web colors used by most web pages. I've got different opinions about this. It's a workaround for the nokia browser but no solution. After a lot of discussion here in Oslo we saw that this needs a deeper fix in QtWebKit. More will follow.. Assigning the error to Jay as he is working on the issue together with others in the code camp Worked together with Tor Arne on a patch that provides a principled solution to this issue. Need to finish patch in next day or two. Removing the blocks 39121 as the fix is not required for Qtwebkit 2.1. Original issue reported by browser team is not reproducible, so removing the 39121 (In reply to comment #12) > Need to finish patch in next day or two. Any update? Should this task be closed? (In reply to comment #14) > (In reply to comment #12) > > Need to finish patch in next day or two. > > Any update? Should this task be closed? I'm removing the block for bug 44677, as 2.1 appears to not be affected anymore (see comment #13) and nobody appears to be working on it anyway. Jay: Please close this bug or update its status. Per the comments above, this issue has been resolved. |