IE 6&7: Error when editing cell properties in table

[Jakub Ś]

Happens in IE6 & IE7

• Load ​rev 6478 sample
• Press "New Page"
• Switch to source
• Paste the following code:

  <p>
  	&nbsp;</p>
  <table border="1" cellpadding="0" cellspacing="0">
  	<tbody>
  		<tr>
  			<td style="width: 197px">
  				<p>
  					Ewrew</p>
  			</td>
  			<td style="width: 197px">
  				<p>
  					435</p>
  			</td>
  			<td style="width: 197px">
  				<p>
  					43543543</p>
  			</td>
  		</tr>
  		<tr>
  			<td style="width: 197px">
  				<p>
  					Werw</p>
  			</td>
  			<td style="width: 197px">
  				<p>
  					435</p>
  			</td>
  			<td style="width: 197px">
  				<p>
  					click here</p>
  			</td>
  		</tr>
  		<tr>
  			<td style="width: 197px">
  				<p>
  					435</p>
  			</td>
  			<td style="width: 197px">
  				<p>
  					435435</p>
  			</td>
  			<td style="width: 197px">
  				<p>
  					ertretretre</p>
  			</td>
  		</tr>
  	</tbody>
  </table>
  

• Switch to wysiwyg
• Click with RBM on cell with value "click here" (DON'T CLICK WITH LBM) and choose cell -> cell properties...
• Result: the error is displayed:

'null' is null or not an object

 function getSelectedCells( selection )
 {
      // Walker will try to split text nodes, which will make the current selection
      // invalid. So save bookmarks before doing anything.
     var bookmarks = selection.createBookmarks(); // <-------

     tabletools/plugin.js, line 14

[Wiktor Walc]

On my computer it happens straight after the dialog loads, without having to press the "Ok" button.

[Jakub Ś]

This has first occured in revision [6478]

[Sa'ar Zac Elias]

I'm unable to reproduce the bug for some reason after trying a few times.

[Garry Yao]

I'm unable to reproduce with either cached dialog or fresh dialog load, looks like a randomly occurring, can we defer it?

[Garry Yao]

I've figured the key to reproduce this bug is to have a paragraph inside the table cell, it's a really weird bug of IE6/7.

[Garry Yao]

Fixed by [6882] on trunk.