91288 - Aligning blank table cell in show all tags mode splits table

Status: VERIFIED FIXED

(Core :: DOM: Editor, defect, P3)

Description

[[e-mail]]

Using Build ID: 2001071508

Steps to reproduce problem:
1. Open Editor.
2. Show All Tags (problem does not occur in normal or preview mode).
3. Insert the default 2x2 table.
4. Select the bottom left cell.
5. Click "Align Right".

Actual results: table is split into two by right-aligned paragraph.

Comment 1

[sujay]

confirming.....I see the problem...works fine in normal mode.

Comment 2

[Dwayne]

Confirming in build 2001-07-14-08, Win2k.

Comment 3

[Dwayne]

*** Bug 91310 has been marked as a duplicate of this bug. ***

Comment 4

[kinmoz]

I'll take this one.

Comment 5

[kinmoz]

FYI, you can reproduce this bug in normal mode by using Control-Left-Click with 
the mouse to select the cell, and then hitting one of the alignment buttons on 
the toolbar.

Note that this is similar to bug 91310, but it takes a slightly different code 
path.

Comment 6

[kinmoz]

I just attatched a patch to bug 91310 that also fixes this bug.

Note the patch only addresses the splitting issue. I noticed that when in 
all-tags mode, after you do the alignment, you can't place your caret in the 
alignment DIV or the table cell. That is a different bug.

Comment 7

[kinmoz]

Fix checked into TRUNK:

   mozilla/editor/base/nsHTMLEditRules.cpp  revision 1.187

Comment 8

[kinmoz]

Really marking fixed this time. :-)

Comment 9

[James Baxter]

Verified on 8-30 Build