IE: The sampleposteddata pages are not wrapping the "Value" collumn

[fredck]

Steps to Reproduce

1. Load the following HTML in sample01.html:

<p>Lorem ipsum dolor sit amet, consectetuer adipiscing elit. Vivamus id ipsum
blandit quam sagittis varius. Sed ipsum metus, lobortis sodales, congue quis,
interdum quis, risus. Duis aliquet, ante sed fermentum pulvinar, sem nibh
vulputate pede, sed accumsan odio elit sed libero.</p>

2. Switch to WYSIWYG.
3. Submit the form.

The text will be shown all in one line in IE. In FF, Safari and Opera it will wrap properly.

[martinkou]

Using IE6 on the client side and IIS5.1/WinXP on the server side, sampleposteddata.asp doesn't even seem to work (i.e. no output). Strangely, using Firefox on the client side instead works.

I'm not sure if I've got some settings messed up.

[w.olchawa]

I've done some tests and I have exactly opposit results. FF doesn't wrap the text and IE does. Tested using FCKeditor SVN

[w.olchawa]

Patch proposal

[alfonsoml]

It's interesting to note that all the three files output a different HTML. I would prefer to change all of them to this kind of structure:

<table id="sampleOutput">
	<thead>
		<tr>
			<th>Field Name</th>
			<th>Value</th>
		</tr>
	</thead>
	<tbody>

	<% For Each sForm in Request.Form %>
	<tr>
		<th><%=sForm%></th>
		<td><%=Server.HTMLEncode( Request.Form(sForm) )%></td>
	</tr>

	<% Next %>
	</tbody>
</table>

and put all the styles in the sample.css file.

Of course, now in the html folder the file is the php version, not the asp.

[w.olchawa]

Patch proposal

[alfonsoml]

the rules in CSS for td has an invalid value.

And while I'm testing this, I can't see the output in the php page, I'm gonna attach a patch that fixes the problem, and my previous suggestion.

[alfonsoml]

New patch

[fredck]

There is a lot of confusion here... this ticket has been opened when we were using an ASP page to show the posted data for the HTML samples ( see it here). Now that we are using the PHP page, this ticket is not anymore valid, because I thought all server side integrations were using the same system. But they are not. Actually, the HTML correctly wraps today with the HTML samples.

The original ASP page has one important difference, that I expect to have in all implementations. It uses a <pre> to properly display the data. It means that white spaces and line breaks are correctly displayed.

So, the fix for this ticket is much broader. It must still propose a <pre> or anything similar, but also properly wrap the text. Actually, the we need the exact same output on all integration files.

Also, the patch must include the fix for all our server side integration files, not only ASP and PHP.

[fredck]

The solution at " Formatting quoted code in blog posts" may be handy.

[alfonsoml]

Revised patch

[alfonsoml]

I've modified the css for the pre element and added it to all the samples, as well as cleaning up the inline styles and attributes that were being used.

I just hope that all the samples keep on working, I'm unable to test them myself.

[fredck]

No lucky... it is not yet wrapping in IE. It looks good with FF and Safari.

Also, is there any chance of having the Field Name vertically aligned at the top? It may disappear for long data.

[alfonsoml]

Revised patch

[alfonsoml]

It seems that the problem was that IE was automatically expanding the width of the table, so I've forced it to keep the widths.

The alignments have been changed in the css file.

I've noted in the what's new that this patch also fixes #2239

[alfonsoml]

fixed with [2039]