197623 - prefetch links should not be shown in Site Navigation Bar

Status: RESOLVED FIXED

(SeaMonkey :: UI Design, defect)

Description

[Brant Gurganus]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.4a) Gecko/20030307
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.4a) Gecko/20030307

If a Webmaster believes certain links will be followed in a Web page, he or she
may add link elements with prefetch relations.  This, a decidedly internal
mechanism from the consensus of bug 193563, should not appear in the Site
Navigation Bar.

For details of the previous discussion of prefetch links in the toolbar, see bug
193563.
For details on prefetching itself, see
<http://www.mozilla.org/projects/netlib/Link_Prefetching_FAQ.html>.

Reproducible: Always

Steps to Reproduce:
1. Create a page with the following code in either HTML or XML syntax:
  <link rel="prefetch" href="page2.html" title="Prefetched Content">
2. Open the page in Mozilla Navigator.
3. Make sure the Site Navigation Bar is visible by clicking Show/Hide > Site
Navigation Bar > Show Always in the View menu.
4. If the toolbar is not visisble, expand any unexpanded grippies until you find it.
5. Click the More button in the Site Navigation Bar.
Actual Results:  
In the menu expanded by the More button, there is a prefetch folder listing
"Prefetch Content."

Expected Results:  
There should not be a prefetch folder.

Comment 1

[Brant Gurganus]

I am adding active CCs from other bug.

Comment 2

[Brant Gurganus]

I haven't been able to modify my patch from the original bug so that <link
rel="prefetch next"> appears as Next, but not as prefetch.  Does anybody have
any ideas on how this could be done.

Comment 3

[Brant Gurganus]

cc'ing some others from bug 173674 to see if they can help on this one

Comment 4

[Steve Chapel]

Attachment: testcase

Comment 5

[Steve Chapel]

Attachment: testcase with rel="next prefetch"

This is the problematic case with the original patch -- if rel="next prefetch"
is specified, the Next button should be enabled but there should not be a More
> prefetch menu.

Comment 6

[Steve Chapel]

Attachment: fix

Comment 7

[Johnny Stenback (:jst)]

Comment on attachment 119815 [details] [diff] [review]
fix

r/sr=jst

Comment 8

[Asa Dotzler [:asa]]

Comment on attachment 119815 [details] [diff] [review]
fix

a=asa (on behalf of drivers) for checkin to 1.4

Comment 9

[:deanis74]

Comment on attachment 119815 [details] [diff] [review]
fix

r=me; jst's r= now becomes sr=.

Comment 10

[Steve Chapel]

timeless just checked this in