Open Bug 338240 Opened 19 years ago Updated 3 years ago

FAYT for links matches two or more links at once

Categories

(Toolkit :: Find Toolbar, defect, P3)

defect

Tracking

()

People

(Reporter: daniel, Unassigned)

References

()

Details

Attachments

(1 file)

User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.7.12) Gecko/20050926 Firefox/1.0.7 Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.7.12) Gecko/20050926 Firefox/1.0.7 The find-as-you-type for links will match any text that starts in a link. This can produce the confusing situation of matching two links at once: then, which should be focused? Reproducible: Always Steps to Reproduce: 1. Visit a website with links and text interspersed (in other words, pretty much any site). These instructions will refer to the address above. 2. Hit ' to begin the find-links-as-you-type 3. Search for the text of the link and some of the text after the link. On the sample page provided, try searching for "link with an" or so. Actual Results: FAYT matches both the link and the text after the link. Expected Results: FAYT matches up to the end of the link, then gives a "Phrase not found" beep. Another possible way to see why this behaviour is confusing is to visit http://www.dragongoserver.net. Searching for, i.e., "ms fa" (note the space) will match two links, one reading "Forums" and one reading "FAQ", that are on two different lines.
OS: Linux → All
Hardware: PC → All
Summary: FAYT for links matches non-links → FAYT for links matches two of more links at once
Version: unspecified → Trunk
Summary: FAYT for links matches two of more links at once → FAYT for links matches two or more links at once
I'll take this. I think I know how to fix the problem, it's just a question of when I can get to it. (RangeStartsInsideLink() needs to change names and functions to check both ends of the range.)
Assignee: nobody → pkasting
Status: UNCONFIRMED → NEW
Ever confirmed: true
Attached patch draftSplinter Review
Assignee: pkasting → mano
Status: NEW → ASSIGNED
Priority: -- → P3
Target Milestone: --- → Firefox 3 alpha2
Product: Firefox → Toolkit
Target Milestone: mozilla1.9alpha2 → ---
Assignee: asaf → nobody
Status: ASSIGNED → NEW
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: