Closed Bug 97164 Opened 23 years ago Closed 23 years ago

Join selected cells is an active option in menus even though it is not possible to join the cells

Categories

(Core :: DOM: Editor, defect)

defect
Not set
normal

Tracking

()

VERIFIED WONTFIX

People

(Reporter: TucsonTester2, Assigned: cmanske)

Details

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (Windows; U; Win98; en-US; rv:0.9.3+) Gecko/20010821
Netscape6/6.1b1
BuildID:    20010821

While using the join selected cells menu, I tried joining cells that should not
be joinable.  Composer does not join the cells but it does show  in the right
click and file menu that "joining the selected" cells is an active option.

Reproducible: Always
Steps to Reproduce:
1.Open composer
2.Make a 5x5 100 pixel table
3.right click on the cell in the top left of the table and choose "join with
cell to right"
4.Ctrl+click the newly made top left cell and the cell below it
5.Choose table and then "join selected cells"

Actual Results:  The cells will not join after choosing this option.

Expected Results:  I expect that if the cells are not joinable that the menus
will have the option in the menus to "join selected cells" would be grayed out.
 This is true for the right click and table menu.
-->cmanske
Assignee: brade → cmanske
Status: UNCONFIRMED → NEW
Ever confirmed: true
It is very tricky to determine if an arbitrary set of selected cells is 
"joinable". The case you found is typical: using rowspan > 1 and colspan > 1 
greatly complicates the allowable join cases. When cells are actually joined,
this logic is invoked, and there may be cells that are not joined as part of this
process, which takes multiple passes through the table as cells are recursively
joined. Thus we do not try to determine exactly if all cells can be joined. It
would be way to expensive to do so.
Thus we will probably never fix this.
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → WONTFIX
verified.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.