Closed Bug 53539 Opened 24 years ago Closed 24 years ago

Using OK when editing existing anchor gives error if name isn't changed

Categories

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

defect

Tracking

()

VERIFIED FIXED
mozilla0.9

People

(Reporter: Brade, Assigned: cmanske)

Details

(Whiteboard: [nsbeta3-])

Attachments

(1 file)

I can't edit an existing anchor. Steps to reproduce: * Create a New Composer Window * insert some text * insert an anchor named "foo" * (click OK) * double-click the anchor icon If I click OK, I get an error about "foo" already existing in the document. If I click Cancel, I crash sometimes (sorry; no stack crawl was available). Expectations: I shouldn't see an error dialog about the anchor already existing if that's the anchor I'm editing OR the OK button shouldn't be enabled. I shouldn't crash when I click the Cancel button.
I'll look into it as soon as i finish the build.
Target Milestone: --- → M19
on win98, this works fine using comm build 2000092108, will test on mac when install is complete
hmm - let me rephrase that last statement -- I'll test on mac when I get a build that actually launches
Whiteboard: [waiting for a good mac build]
Seems to work ok on NT today.
actually I can dup the error dialog, but I do not get the crash, the process should be smart enough to know it is editing itself and not present the dialog -- but that can wait till post-rtm
Whiteboard: [waiting for a good mac build] → [nsbeta3-]
Target Milestone: M19 → Future
handing to cmanske
Assignee: beppe → cmanske
Keywords: crash
Status: NEW → ASSIGNED
Changed summary to reflect problem. You can edit an existing anchor if you change the textfield contents.
Summary: can't edit existing anchor → Using OK when editing existing anchor gives error if name isn't changed
nominating this bug to be fixed before mozilla 1.0. We should either disable the OK button if the anchor name hasn't changed or not require the user to change the anchor before they click OK (unless it *is* a duplicate of a different anchor in the document).
Target Milestone: Future → mozilla0.9
We now remember the original "name" and hanlde that case correctly.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
verified in 1/17 build.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: