[webkit-reviews] Bug #3360: align="right" on a text input aligns the input field to the right, not the text inside

Darin Adler darin at apple.com
Thu Jun 9 06:05:10 PDT 2005


On Jun 9, 2005, at 3:34 AM, Mark Rowe wrote:

> On 9/06/2005, at 22:09 , Niels Leenheer wrote:
>
>> I attached a patch to solve bug #3360:
>> http://bugzilla.opendarwin.org/show_bug.cgi?id=3360
>>
>> I did not use the cvs-extract tool to generate the patch, because  
>> I am currently
>> working on a Windows machine. This is also the reason why was not  
>> able to
>> actually test if this patch solves the bug.
>
> I've tested this patch.  The patch causes the test cases at <http:// 
> bugzilla.opendarwin.org/attachment.cgi?id=2155>  and <http:// 
> www.nousoft.org/webkit-2.html> to render correctly, and does not  
> affect the layout tests.  It could probably do with a layout test  
> to ensure it's behaviour in the future.

That's great guys! Thanks!

What's the purpose of the m_render check? It doesn't make sense to me  
to ignore the ALIGN attribute if it's changed at a time the element  
isn't attached to the render tree. I think we want a patch without it.

Once we deal with that, and add a layout test, I'll land the change.

     -- Darin




More information about the webkit-reviews mailing list