176473 - Invalid markup generated when creating two adjacent links

Status: NEW

(SeaMonkey :: Composer, defect)

Description

[Daniel Glazman (:glazou) (not active in Mozilla any more)]

Reproduced with trunk build 20021023 and NS7.0

1. launch Composer
2. type   aaabbbbcccc
3. select all the "c" characters
4. create a link, target "c"
5. then select all the "b" characters
6. create a link, target "b"

Expected result:
   aaa<a href="b">bbbb</a><a href="c">cccc</a>

Actual result:
   aaa<a href="c"><a href="b">bbbb</a>cccc</a>

Hint: the HTML4 content model for A excludes A (section 12.2 of spec)

  <!ELEMENT A - - (%inline;)* -(A) -- anchor -->

Comment 1

[Daniel Glazman (:glazou) (not active in Mozilla any more)]

nominating EB

Comment 2

[Daniel Glazman (:glazou) (not active in Mozilla any more)]

Attachment: Joe, is this simple patch the correct way to fix this bug ?

Comment 3

[Daniel Glazman (:glazou) (not active in Mozilla any more)]

Attachment: better, cleaner ; Joe, same question ?

Comment 4

[Kevin McCluskey (gone)]

EDITORBASE+. nsbeta1+

Comment 5

[Joe Francis]

I've discussed this with Daniel.  I'm worried this fix will disable merging of
like styles in some cases.  

I think the core issue is that we are putting in the <a> tags before we have the
href.  Offhand I can't think of any good reason for that.  So I suggest we look
into changing that, which should have the effect of fixing this bug.

Comment 6

[saari (gone)]

EDITORBASE+ topembed+ normalization

Comment 7

[Samir Gehani]

adt: nsbeta1+/adt2

Comment 8

[Kevin McCluskey (gone)]

Daniel can you take a look. 

Comment 9

[Andrei Purice]

Hey Daniel,
Does this issue still reproduce for you or can we close it?

Comment 10

[BugBot [:suhaib / :marco/ :calixte]]

The bug assignee didn't login in Bugzilla in the last months and this bug has severity 'major'.
:hsinyi, could you have a look please?
For more information, please visit auto_nag documentation.

Comment 11

[BugBot [:suhaib / :marco/ :calixte]]

Redirect a needinfo that is pending on an inactive user to the triage owner.
:hsinyi, since the bug has high severity and recent activity, could you have a look please?

For more information, please visit auto_nag documentation.

Comment 12

[Hsin-Yi Tsai (she/her) [:hsinyi]]

Moving to another component as the STR in comment 0 mentioned "launch composer"