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]

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]

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]

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;

[fredck]

Replying to satya:

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

Yes, that changes everything.

[Saare]

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].