Summary: | [Qt] Support text-shadow blur | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Jonathon Jongsma (jonner) <jonathon> | ||||||
Component: | Text | Assignee: | Ariya Hidayat <ariya.hidayat> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Enhancement | CC: | ademar, ariya.hidayat, benjamin, bero, hausmann, j, vestbo | ||||||
Priority: | P3 | Keywords: | Qt, QtTriaged | ||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
Bug Depends on: | |||||||||
Bug Blocks: | 34479 | ||||||||
Attachments: |
|
Description
Jonathon Jongsma (jonner)
2008-06-23 09:14:58 PDT
hm, I tried to port the old implementation and performance seemed incredibly bad. I may have done something wrong, but my first impression is that it will be difficult to add blur support with decent performance. I may come back to this later, but I'm moving on to other things for the moment. If anybody else is interested in adding blur, feel free to grab this bug. I can imagine that the performance is related to the FIXME's in the old code, which allocates a QImage with the size of the entire viewport. Add and assign to myself. Created attachment 65169 [details]
Screenshot of the result of applying the patch
I have a patch ready, the attached screenshot shows the result of the blur shadow.
Created attachment 65170 [details]
Patch
Comment on attachment 65170 [details]
Patch
Isn't there some way to factor some of this into online methods?
> Isn't there some way to factor some of this into online methods?
It is not trivial since one works with QPainter::drawText while the other uses QTextLayout/QTextLine. It is unfortunate that the boilerplate for the prolog and epilog are similar, but I can't see a clean way to refactor them, without falling back to C-ism macro magic.
Committed r65876: <http://trac.webkit.org/changeset/65876> There must be a very good reason to include any more bugs as blockers for the qtwebkit-2.1 release at this point. Since I don't see one, I'm removing it from the blocking list. :-) Feel free to add it as a *candidate* for inclusion (bug #44677) and please state the motivation/reason/requirement for it. |