[Webkit-unassigned] [Bug 27889] [Chromium] RTL autocomplete popup is not layout correctly.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Aug 10 11:21:29 PDT 2009


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





--- Comment #18 from Jay Campan <jcampan at google.com>  2009-08-10 11:21:25 PDT ---
Here are my comments on the patch:

WebCore/platform/chromium/PopupMenuChromium.cpp
I think it would be better to make the RTL detection behavior not dependent on
m_settings.restrictWidthOfListBox and instead being a new parameter of
m_settings.
The idea would be to keep PopupMenuChromium generic and let the client decides
which heuristic to use.

In paintRow() regarding the ellipsis code. That code seems non-specific to the
autofill. Would it make sense to have
GraphicsContext::drawBidiTextWithinWidth() draw the ellipsis (may be the method
could have an extra-parameter specifying whether or not an ellipsis is
desired)?
I am assuming the ellipsis ... is the same in all languages. If it is not, may
be the extra-param could be a string of the ellipsis to use? (empty string for
no ellipsis)

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



More information about the webkit-unassigned mailing list