301395 - <a name> becomes focussed if you search to it with /foo

Status: NEW

(Toolkit :: Find Toolbar, defect, P2)

Description

[Hixie (not reading bugmail)]

STEPS TO REPRODUCE
  1. data:text/html,%3C!DOCTYPE%20HTML%3E%3Cp%3E%3Ca%3Etest%3C%2Fa%3E%3C%2Fp%3E
  2. Search for "test" using "/test".

ACTUAL RESULTS
  The word "test" has a focus outline around it!

EXPECTED RESULTS
  The word "test" should not get an outline, just like it doesn't if you use 
  Ctrl+F to search for it, or if you put the caret there (f7) or if you try 
  tabbing to it or anything.

Comment 1

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

taking.

Comment 2

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

Attachment: Patch rv1.0

Comment 3

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

This patch changes that the found link content is not actual link(e.g., <a name>), the founld link element is not saved in its member. So, for out of the class(i.e., for findBar.js), the link is not found by previous search.

Comment 4

[Mike Connor [:mconnor]]

Comment on attachment 201962 [details] [diff] [review]
Patch rv1.0

Mano knows this code way better, not sure if this still applies, but I'm not going to have cycles for it anytime soon.

Comment 5

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

I'm resetting bugs which are assigned to me but I'm not working on them and I don't have plan for fixing them in near future.

Comment 6

[Asaf Romano (gone)]

Comment on attachment 201962 [details] [diff] [review]
Patch rv1.0

Unfortunately this doesn't apply any longer.

Comment 7

[Michael Kohler [:mkohler]]

This is a mass change. Every comment has "assigned-to-new" in it.

I didn't look through the bugs, so I'm sorry if I change a bug which shouldn't be changed. But I guess these bugs are just bugs that were once assigned and people forgot to change the Status back when unassigning.

Comment 8

[Justin [:JW_SoftvisionQA]]

Per comments 5 & 6, would you say this is a wont fix?

Comment 9

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

(In reply to Justin [:JW_SoftvisionQA] from comment #8)
> Per comments 5 & 6, would you say this is a wont fix?

No, just *I* stepped down from the assignee of this bug.

Comment 10

[Mike de Boer [:mikedeboer]]

This is intended behavior for Quickfind... search for links is enabled by default when activating with '/'.

Comment 11

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

(In reply to Mike de Boer [:mikedeboer] from comment #10)
> This is intended behavior for Quickfind... search for links is enabled by
> default when activating with '/'.

No, <a name> doesn't create a link. A link is created only when <a> element has href and it has valid URL. Didn't you misjudge?

Comment 12

[Mike de Boer [:mikedeboer]]

Hmm, I think you're right.