Closed
Bug 174558
Opened 22 years ago
Closed 22 years ago
Delete Table command doesn't work
Categories
(SeaMonkey :: Composer, defect)
SeaMonkey
Composer
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla1.2final
People
(Reporter: bugzilla2, Assigned: cmanske)
References
Details
(Keywords: regression)
Attachments
(1 file)
(deleted),
patch
|
Brade
:
review+
kinmoz
:
superreview+
asa
:
approval+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Windows; U; Win 9x 4.90; en-US; rv:1.2b) Gecko/20021015
Build Identifier: Mozilla/5.0 (Windows; U; Win 9x 4.90; en-US; rv:1.2b) Gecko/20021015
The DELETE TABLE command from the menu or the right-click menu stopped working
sometime between the 20021005 and 20021012 builds.
You can delete a table by deleting the last row, but the delete table command
does nothing.
Reproducible: Always
Steps to Reproduce:
1. Open a blank composer document
2. Insert any table
3. With the caret in the table, try to delete the tanle with Table/DeleteTable
or right-click and select Table Delete/Table
Actual Results:
The table is not deleted
Expected Results:
The table should be deleted
Assignee | ||
Comment 2•22 years ago
|
||
Oops! A trivial spelling mistake in ComposerCommands.js:
GetCurrentEditor().delectTable();
should be
GetCurrentEditor().deleteTable();
I'm already making a lot more changes to ComposerCommands.js, so I'll include
the fix there.
Status: UNCONFIRMED → ASSIGNED
Depends on: 169029
Ever confirmed: true
Target Milestone: --- → mozilla1.2beta
Comment 3•22 years ago
|
||
can this get fixed for 1.2final?
Keywords: nsbeta1,
regression
OS: Windows ME → All
Hardware: PC → All
Whiteboard: editorbase
Target Milestone: mozilla1.2beta → mozilla1.2final
Assignee | ||
Comment 4•22 years ago
|
||
Yes! I will try to get this in ASAP.
I fully expected to be checking in all other changes to ComposerCommands.js,
but unfortunately, that won't be happening :(
Assignee | ||
Comment 5•22 years ago
|
||
Updated•22 years ago
|
Attachment #104686 -
Flags: review+
Comment 6•22 years ago
|
||
Comment on attachment 104686 [details] [diff] [review]
fix v1
r=brade
Attachment #104686 -
Flags: superreview+
Assignee | ||
Updated•22 years ago
|
Whiteboard: editorbase → editorbase [FIX IN HAND] need a=
Assignee | ||
Updated•22 years ago
|
Whiteboard: editorbase [FIX IN HAND] need a= → [FIX IN HAND] need a=
Comment 8•22 years ago
|
||
Comment on attachment 104686 [details] [diff] [review]
fix v1
recent regression, firly visible, low-risk, my kinda fix. a=asa for checkin to
1.2 (on behalf of drivers)
Attachment #104686 -
Flags: approval+
Assignee | ||
Comment 9•22 years ago
|
||
Checked into 1.2final trunk
Status: ASSIGNED → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Whiteboard: [FIX IN HAND] need a=
Comment 10•22 years ago
|
||
*** Bug 178065 has been marked as a duplicate of this bug. ***
Updated•20 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•