[Webkit-unassigned] [Bug 117405] [EFL]Background Style of element not visible because of default theme style

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=117405





--- Comment #34 from Santosh Mahto <santosh.ma at samsung.com>  2013-09-03 08:59:52 PST ---
(In reply to comment #31)
> (In reply to comment #25)
> > (In reply to comment #24)
> > > (In reply to comment #22)
> > > > (In reply to comment #19)
> > > > > Created an attachment (id=209959)
 --> (https://bugs.webkit.org/attachment.cgi?id=209959&action=review) [details] [details] [details] [details] [details]
> > > > > Background display of select element
> > > > 
> > > > That looks better and is indeed an improvement, but isn't there a way to get rid of the colored rectangle behind the widget?
> > > I wanted to stick to original(without background styled display), in older case
> > > The size of combo widget is smaller than select rect. if we want to fill the complete rect with combo widget we should do for normal case(without bg) also.
> > > Incase of styled we can fit the rect but will be biasing with repsect to nonstyled case.
> > > 
> > > So I am not sure about fitting widget to element rect.
> > > Will ask Ruyan about this
> > 
> > It's because of current images which combo.edc use. image itself contains unnecessary empty space.
> > 
> > FYI, We have plan to improve image and edc files including combo.
> 
> Isn't it possible to fix this issue here, or merge this patch with the rest of the improvements you guys are planning on making?
It looks better and safe to make separate bugs for "updating Images" and another enhancement because:
It will be easy to work/optimize on small patches and would ensure we don't break current one.
updating images itself will require update of all Layout test expectation. So it will become big patch.
Bug Title also will become ambiguous.
I will try to land other enhancement on  control element asap.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.