Bug 119786 - [EFL] decoration of search type is different from other platforms
Summary: [EFL] decoration of search type is different from other platforms
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit EFL (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Ryuan Choi
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-08-14 00:25 PDT by Ryuan Choi
Modified: 2013-08-14 22:25 PDT (History)
5 users (show)

See Also:


Attachments
Patch (744.20 KB, patch)
2013-08-14 01:12 PDT, Ryuan Choi
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Ryuan Choi 2013-08-14 00:25:23 PDT
Search field has three types of decoration style, -webkit-search-results-button, -webkit-search-results-decoration and -webkit-search-decoration

Other platforms except Efl port draw glass icon for search type in first two cases,
but only Efl ports draw icon in last case.

It looks better for Efl port to draw icon in first two cases like other ports.
Comment 1 Ryuan Choi 2013-08-14 01:12:32 PDT
Created attachment 208705 [details]
Patch
Comment 2 Gyuyoung Kim 2013-08-14 01:26:39 PDT
Comment on attachment 208705 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=208705&action=review

I made that the magnifying glass icon was always shown. I don't object to follow behavior of other ports. r=me.

> Source/WebCore/ChangeLog:7
> +        Other platforms except EFL port draw glass icon for

glass -> a magnifying glass ?

> Source/WebCore/ChangeLog:9
> +        but Efl port draw it for -webkit-search-decoration.

draws ?
Comment 3 Gyuyoung Kim 2013-08-14 01:49:00 PDT
Comment on attachment 208705 [details]
Patch

I checked chrome also has same behavior as this patch's. Thanks.
Comment 4 Ryuan Choi 2013-08-14 22:13:09 PDT
Committed r154089: <http://trac.webkit.org/changeset/154089>
Comment 5 Ryuan Choi 2013-08-14 22:25:26 PDT
Comment on attachment 208705 [details]
Patch

landed after fixed changelog.

Thanks.