5748 - In Editor, a link is inserted even though no URL was specified

Status: VERIFIED FIXED

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

Description

[Chris Petersen]

Version: Apprunner
Build: April 29th build. (1999042908)
Expected Results: If no url is specified in the Link to field, the link should
not be inserted.
What I got: The linked text is inserted into the document.
Steps to Reproduce:
1) Launch Apprunner.
2) Select Editor from the Task menu.
3) Place cursor in document and click on  Link from the toolbar.
4) Type in some text for the link source. (Text that is to displayed in the
document)
5) Leave the Link to field empty. Click the OK button.
6) Notice that the linked text is inserted.Since there isn't a url source
specified, the linked text shouldn't be inserted.

Comment 1

[Greg Kostello]

Assign to Akkana

Comment 2

[Akkana Peck]

Part of the link dialog rework.

Comment 3

[Charles Manske]

Based on the "crawl, walk, run" philosophy, we think it is better to get more
dialogs working rather than getting each one perfect.
Its OK to file bugs on missing behavior, of course, I'm just explaining the
implementation process.

Comment 4

[Charles Manske]

It would be best to fix this after the "onchange" handler works for text
fields, so postponing for a better milestone.

Comment 5

[Charles Manske]

Thinking about this, I'm not sure its a bug. In the case of a new link, if
there is no URL, nothing happens (we don't insert the "A" tag), so no harm done.
In the case of an existing link, allowing the user to delete the URL
seems a logic way to remove the link. Otherwise they have to know about the
"Remove Link(s)" item in the Format menu.
Any opinions? (I'll bring up the issue in the editor and UI news groups)

Comment 6

[Charles Manske]

Checked in 7/22
We now allow an empty link editfield as a means to remove an existing link:
Select some text, click on the link button, enter the link URL (if you leave
this empty, no link will be created), click OK.
Click on the link button again, you should see the link URL you typed before.
If you clear that text and click OK, the link should be removed from the
document.

Comment 7

[sujay]

verified in 7/26 build.