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)
Toolkit
Find Toolbar
Tracking
()
NEW
People
(Reporter: daniel, Unassigned)
References
()
Details
Attachments
(1 file)
|
4.09 KB,
patch
|
Details | Diff | Splinter Review |
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.
Updated•19 years ago
|
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
Updated•19 years ago
|
Summary: FAYT for links matches two of more links at once → FAYT for links matches two or more links at once
Comment 1•19 years ago
|
||
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
Updated•19 years ago
|
Status: UNCONFIRMED → NEW
Ever confirmed: true
Comment 2•19 years ago
|
||
Assignee: pkasting → mano
Status: NEW → ASSIGNED
Updated•19 years ago
|
Priority: -- → P3
Target Milestone: --- → Firefox 3 alpha2
| Assignee | ||
Updated•17 years ago
|
Product: Firefox → Toolkit
Updated•16 years ago
|
Target Milestone: mozilla1.9alpha2 → ---
Updated•7 years ago
|
Assignee: asaf → nobody
Status: ASSIGNED → NEW
Updated•3 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•