55921 - Clicking to the right of a link doesn't clear selection

Status: VERIFIED FIXED

(Core :: DOM: UI Events & Focus Handling, defect, P3)

Description

[Jesse Ruderman]

Clicking to the right of a link at the end of a line doesn't clear the 
selection, and double-clicking in the same place doesn't select the nearest 
word.  2000100708 Win98.

Comment 1

[Jesse Ruderman]

Attachment: testcase

Comment 2

[joki (gone)]

This one seems like its most likely a logic error where selection isn't doing
anything if the nearest element is a link.  Or I could be wrong.  Could you take
a look Tony?

Comment 3

[anthonyd]

Joki,

still investigating this, but in editor this works as its supposed to.  single 
clicking to the right of the link will 'snap' the caret to the right of the 
link, double clicking will bring up the ;inks properties dialog (im not sure if 
the later part is correct).  in the browser, clicking does nothing.

anthonyd

Comment 4

[anthonyd]

got a fix, will attach diff, need reviews/approvals.  adding pink since the fix 
is in his code.

anthonyd

Comment 5

[anthonyd]

Attachment: patch for this bug

Comment 6

[Mike Pinkerton (not reading bugmail)]

uhhh, i didn't write any of this...

Comment 7

[anthonyd]

cc'ing saari, as this is actually joki's code and i need a code review for this.

anthonyd

Comment 8

[Jesse Ruderman]

I wonder if this is also why I can't select quips on bugzilla's query result 
pages.

Comment 9

[Hixie (not reading bugmail)]

Reassigning QA Contact for all open and unverified bugs previously under Lorca's
care to Gerardo as per phone conversation this morning.

Comment 10

[saari (gone)]

Um, looks okay to me, but I don't know the code well enough to give you an r=.
Have to ping Tom for that.

Comment 11

[anthonyd]

Attachment: new patch with kins changes

Comment 12

[rubydoo123]

moving to moz0.8 and asked Tom if he could review the patch

Comment 13

[joki (gone)]

Looks good to me.  r:joki

Comment 14

[anthonyd]

is that an sr= from joki, or an r=?  whichever, i still need the other, 
requesting r= from sfraser.

anthonyd

Comment 15

[Simon Fraser [no longer active]]

Looks sensible. r/sr=sfraser

Comment 16

[anthonyd]

fix was checked in, forgot to close bug :-P

Comment 17

[Madhur Bhatia]

QA contact updated

Comment 18

[Madhur Bhatia]

verfied fixed on build 2001-08-06-09trunk

Comment 19

[luis rincon]

verified on build 2001-08-06-trunk