Bug 8407

Summary:

REGRESSION (NativeTextField): Leading and trailing spaces trimmed from text field value attribute

Product:

WebKit

Reporter:

mitz

Component:

HTML Editing

Assignee:

Adele Peterson <adele>

Status:

VERIFIED FIXED

Severity:

Normal

CC:

adele, darin, hyatt

Priority:

Keywords:

Regression

Version:

420+

Hardware:

Mac

OS:

OS X 10.4

URL:

data:text/html,There%20should%20be%20spaces%20before%20"foo"%20and%20after%20"bar"%20<INPUT%20value="%20foo%20bar%20">

Attachments:

Description	Flags
Possible fix for the text field case	none
patch to set white-space:pre	darin: review+

mitz

Reported 2006-04-15 03:02:53 PDT

<input value=" foo bar "> doesn't show the leading and trailing spaces in the text field.

Attachments
Possible fix for the text field case (818 bytes, patch) 2006-04-15 04:12 PDT, mitz	no flags	Details Formatted Diff Diff
patch to set white-space:pre (4.25 KB, patch) 2006-04-18 00:01 PDT, Adele Peterson	darin: review+	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

mitz

Comment 1 2006-04-15 03:55:29 PDT

The contenteditable version of this bug is <div contenteditable> lorem ipsum </div> where the outer spaces are initially invisible, but when you add a space at the beginning/end you get two spaces to show.

mitz

Comment 2 2006-04-15 04:12:34 PDT

Created attachment 7726 [details] Possible fix for the text field case

Darin Adler

Comment 3 2006-04-15 19:18:04 PDT

I think the best way to fix this is to use the appropriate whitespace mode inside the text fields. Assuming there is an appropriate whitespace mode.

Adele Peterson

Comment 4 2006-04-16 15:37:25 PDT

mitz and i talked about using whitespace:pre for the inner div, instead of just whitespace:nowrap. i will investigate.

Adele Peterson

Comment 5 2006-04-18 00:01:10 PDT

Created attachment 7794 [details] patch to set white-space:pre

Adele Peterson

Comment 6 2006-04-18 00:01:44 PDT

Comment on attachment 7794 [details] patch to set white-space:pre I'll need to re-generate pixel results for this test on a PPC machine before checking in.

Darin Adler

Comment 7 2006-04-18 08:52:41 PDT

Comment on attachment 7794 [details] patch to set white-space:pre r=me

Note You need to log in before you can comment on or make changes to this bug.