BIDI: Alignment set to text in Table cell is not shown in the Tool bar when we press Enter to start a new Paragraph

[Satya Minnekanti]

To reproduce the defect:

1. Open Ajax sample.

2. Insert a Table, go in to a Table cell, and click on an Alignment option(for eg: Align Right) in the Tool bar.

3. Now Type the text in the cell and see that it is Right Aligned and Align Right icon in the Tool bar is highlighted.

4. Press Enter.

Expected Result:

A New Paragraph starts and it should have Right Alignment as well and Align Right Icon in the Tool bar should be highlighted.

Actual Result:

A New Paragraph is started and it has Right Alignment but Align Right Icon in the Tool bar is not showing as highlighted and when we keep the cursor in the previous paragraph Align Right icon is not even highlighted for that Paragraph as well.

same behavior happens for all the Alignment options.

[Tobiasz Cudnik]

I can't reproduce it, Alignment icon always represents correct state. Here's the content i've used:

<table>
	<tbody>
		<tr>
			<td style="text-align: right;">
				<p>
					foo</p>
				<p>
					bar</p>
			</td>
		</tr>
	</tbody>
</table>

On what browser you're experiencing this issue ?

[Satya Minnekanti]

Replying to tobiasz.cudnik:

I can't reproduce it, Alignment icon always represents correct state. Here's the content i've used:

<table>
	<tbody>
		<tr>
			<td style="text-align: right;">
				<p>
					foo</p>
				<p>
					bar</p>
			</td>
		</tr>
	</tbody>
</table>

On what browser you're experiencing this issue ?

I could reproduce it on all the browsers FF3.5,IE6 & 7..even though <td style="text-align: right;"> is shown in HTML source for the Table cell, the corresponding icon(Align Right)is not shown as highlighted in the Tool bar when we keep the cursor in the paragraph.

[Satya Minnekanti]

Replying to tobiasz.cudnik:

I can't reproduce it, Alignment icon always represents correct state. Here's the content i've used:

<table>
	<tbody>
		<tr>
			<td style="text-align: right;">
				<p>
					foo</p>
				<p>
					bar</p>
			</td>
		</tr>
	</tbody>
</table>

On what browser you're experiencing this issue ?

It might be useful to mention that this issue occurs when config.useComputedState = false;

[Frederico Caldeira Knabben]

Replying to satya:

It might be useful to mention that this issue occurs when config.useComputedState = false;

Yes, that changes everything.

[Sa'ar Zac Elias]

On L40 you should use getAttribute() and hot hasAttribute(). Also, please add || '' after it to avoid having a null value.

[Tobiasz Cudnik]

Fixed with [5856].