Bug 7075 - textarea with align=right should not cause contained text to be right-justified
Summary: textarea with align=right should not cause contained text to be right-justified
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 420+
Hardware: Mac OS X 10.4
: P2 Normal
Assignee: Nobody
URL:
Keywords: HasReduction
Depends on:
Blocks:
 
Reported: 2006-02-04 16:31 PST by David Kilzer (:ddkilzer)
Modified: 2007-01-02 09:58 PST (History)
2 users (show)

See Also:


Attachments
Test case (77 bytes, text/html)
2006-02-04 16:34 PST, David Kilzer (:ddkilzer)
no flags Details
Test case v2 (77 bytes, text/html)
2006-02-28 14:42 PST, David Kilzer (:ddkilzer)
no flags Details
patch (34.50 KB, patch)
2007-01-02 07:30 PST, Sam Weinig
mitz: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description David Kilzer (:ddkilzer) 2006-02-04 16:31:29 PST
A <textarea align="right"></textarea> element causes text inside of it to be right-justified.

Instead, the contained text should be left-justified, and the align="right" should be ignored.  (The align="right" should not render the whole element on the right side of the page, either.)

Firefox 1.5.0.1 and MSIE 6 both behave this way, e.g., effectively ignoring this attribute.
Comment 1 David Kilzer (:ddkilzer) 2006-02-04 16:34:36 PST
Created attachment 6254 [details]
Test case

Note that this is not a regression as Safari v2.0.3 (417.8) on 10.4.4 behaves the same way.
Comment 2 David Kilzer (:ddkilzer) 2006-02-28 14:42:08 PST
Created attachment 6781 [details]
Test case v2

This is a better test case.  It sets the "cols" parameter large enough so you can tell which way the text was justified (right or left)!
Comment 3 Sam Weinig 2007-01-02 07:30:53 PST
Created attachment 12161 [details]
patch

Patch.  Don't map 'align' attribute for textarea elements.
Comment 4 mitz 2007-01-02 09:47:23 PST
Comment on attachment 12161 [details]
patch

r=me
Comment 5 Sam Weinig 2007-01-02 09:58:04 PST
Landed in r18527.