Bug 49131

Summary:

[Qt] CSS3: text-shadow: Shadow is not displayed, instead shadow text is seen

Product:

WebKit

Reporter:

Nancy Piedra <nancy.piedra>

Component:

CSS

Assignee:

qi <qi.2.zhang>

Status:

RESOLVED DUPLICATE

Severity:

Normal

CC:

ademar, ariya.hidayat, cshu, Ramesh.Bapanapalli, suresh.voruganti

Priority:

Keywords:

Version:

528+ (Nightly build)

Hardware:

S60 Hardware

OS:

S60 3rd edition

URL:

http://iop1.nokia-boston.com/CSS/index.html

Attachments:

Description	Flags
Failure test cases camparison between QtTestBrowser and Chrome	none

Nancy Piedra

Reported 2010-11-06 13:38:39 PDT

1. Load http://iop1.nokia-boston.com/CSS/index.html 2. Select CSS3_Text link. 3. Select the following link and observe its behavior: Link 1: 3.CSS3-Text: 'text-shadow' Property Link 2: 4.CSS3-Text: 'text-shadow' Property Link 3: 7.CSS3-Text: 'text-shadow' Property Link 4: 8.CSS3-Text: 'text-shadow' Property Link 5: 13.CSS3-Text: 'text-shadow' Property ACTUAL RESULTS: Link 1 and 2: Blue color shadow text is seen. Link 3: Black color shadow text is seen. Link 4: Multiple shadow text is seen. Link 5: Red shadow is not seen. Other info: Issue is observed on QtTestBrowser. Issue is not observed on Google Chrome browser. EXPECTED RESULTS: Link 1 and 2: Blue color shadow should be seen. Link 3: Grey color should be seen. Link 4: Multiple color shadow should be seen Link 5: Red shadow should be seen.

Attachments
Failure test cases camparison between QtTestBrowser and Chrome (250.69 KB, application/vnd.openxmlformats-officedocument.spreadsheetml.sheet) 2010-11-08 10:23 PST, qi	no flags	Details
View All Add attachment proposed patch, testcase, etc.

Comment 1 2010-11-08 10:23:32 PST

Created attachment 73249 [details] Failure test cases camparison between QtTestBrowser and Chrome Actually, we just have one issue: Looks like currently QtTestBrowser doesn't support "blur" in shadow. "[Blur-radius]: A length value indicating the boundary of the blurring for the current text-shadow effect." Need to investigate why?

Comment 2 2010-11-09 08:00:10 PST

Those test cases works fine on qt4.7+webkit trunk(linux). I investigated and found qtwebkit missed some patch like r65876 which for text blur, I know we have some patchs about shadow, r65876 is one of them.

Suresh Voruganti

Comment 3 2010-12-13 14:11:38 PST

Fix required for Qtwebkit 2.1/2.2

Ariya Hidayat

Comment 4 2010-12-21 08:32:12 PST

Everything works well with trunk. See the master bug https://bugs.webkit.org/show_bug.cgi?id=34479 for all the shadow handling in Qt. *** This bug has been marked as a duplicate of bug 34479 ***

Ademar Reis

Comment 5 2010-12-21 11:21:33 PST

(In reply to comment #3) > Fix required for Qtwebkit 2.1/2.2 This bug is blocking the 2.2 release but is marked as closed/duplicate of bug 34479. Bug 34479 on the other hand depends on 20 other smaller bugs. I'm removing the dependency, as it should be added on bug 34479 or to a small set of patches if we really decide to cherry-pick/backport this feature (or part of it).

Chang Shu

Comment 6 2010-12-21 12:07:25 PST

(In reply to comment #5) > (In reply to comment #3) > > Fix required for Qtwebkit 2.1/2.2 > This bug is blocking the 2.2 release but is marked as closed/duplicate of bug 34479. > Bug 34479 on the other hand depends on 20 other smaller bugs. > I'm removing the dependency, as it should be added on bug 34479 or to a small set of patches if we really decide to cherry-pick/backport this feature (or part of it). It seems really hard to cherry-pick the patch of bug 34479 as it's dependent on too many other patches. Is it possible we add this feature to a later release that's off the trunk?

Note You need to log in before you can comment on or make changes to this bug.