Closed Bug 26158 Opened 25 years ago Closed 25 years ago

Remove link needs to include insertion

Categories

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

x86
All
defect

Tracking

()

VERIFIED FIXED

People

(Reporter: rubydoo123, Assigned: mozeditor)

Details

Need to expand Remove link functionality to include insertion point instead of only working when the link is highlighted.
Target Milestone: M14
I've been working on a fix for this. But I've discovered that some of my existing rules code is hostile to the idea of leaving selection in between links - it tends to move the selection over into a nearby text node, which usually will be back in the link. Turning off this code casues other bugs. So I have to develop a more robust solution...
I have a fix for this now, if anyone cares to get it in m14.
Status: NEW → ASSIGNED
Whiteboard: have fix, need checkin permission
fixed
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
still not working...I have the link insertion point inside the link and at the end of the link.....then I do Format | Remove Link....and doing this doesn't remove the link...it does however work for when the link is highlighted... used 2/23 build.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
ok talked to Joe....the insertion point does get reset...but the UI(selection) doesn't get reset....this is a UI issue...will consider opening a new bug..
Status: REOPENED → RESOLVED
Closed: 25 years ago25 years ago
Resolution: --- → FIXED
verified in 2/23 build.
Status: RESOLVED → VERIFIED
Steps to reproduce problem: 1. Use tag mode for extra visibility. 2. Put the insersion point just before the end of the link. 3. Manually delete until the end of the link. 4. Type. Characters are still linked. 5. Select Remove link(s). 6. Type. Characters are unlinked, but an extra empty link is created.
Whiteboard: have fix, need checkin permission
You need to log in before you can comment on or make changes to this bug.