[Webkit-unassigned] [Bug 126907] [GTK] Hardcoded text color in input fields
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Wed May 30 23:24:29 PDT 2018
https://bugs.webkit.org/show_bug.cgi?id=126907
--- Comment #63 from Carlos Garcia Campos <cgarcia at igalia.com> ---
(In reply to Carlos Eduardo Ramalho from comment #62)
> (In reply to Carlos Garcia Campos from comment #57)
>
> Thanks for the review! Forgot to respond to the suggestions and there was
> just one point I didn't follow:
>
> > > Source/WebCore/rendering/RenderThemeGtk.cpp:982
> > > + if (element)
> > > + style.setColor(inputColor(element, Entry));
> >
> > I think we should move this after the next if. Again, I think this could be
> > done with current styleColor().
>
> Moving it after the if would result in only calling setColor in textInputs
> with spinButtons.
Oh, you are right.
--
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20180531/17dbdf90/attachment.html>
More information about the webkit-unassigned
mailing list