Visited links @ don't change color




19 years ago
11 years ago


(Reporter: scott, Assigned: radha)




Firefox Tracking Flags

(Not tracked)


(Whiteboard: nsbeta1+, URL)



19 years ago
If you do a search at, click on one of the results, then click the back
button, the followed link does not change color.

Nor does it do so in Commmunicator 4.72.  Works fine in IE5.  This is probably a
dupe but I couldn't find anything.

Comment 1

19 years ago
After typing "Pretty Park" into the Quick Search box at
the links turned to the visited colour after visiting a page and going back,
using the 2000-02-24-08-M14 nightly binary.

Code snippet from that page:
<b>gt;gt; <a href="rate/item.xp?CID=11675&PDID=16009">Dorney Park & Wildwater 
Kingdom</a></b> <span class="small">in <a 
href="rate/list_items.xp?CID=11675&PCID=11524">U.S. Amusement Parks</a></span>
----------, what build of Mozilla were you using? M13 or a later
binary (the build ID appears at the right-hand end of the status bar at the
bottom of the browser window)

A similar bug about the same issue on a cnet page was marked WONTFIX due to
4.x not doing any better with bad nesting of tags, but that does not seem
to be the case here.

Comment 2

19 years ago
I was using a post-M13 nightly.  Still happening as of build 2000022716.  I'm on
NT4 (SP6) with a 3Dfx Velocity 100 AGP video card.

Visited links on other sites changing color just fine.
i'm seeing a link color change only where css is not defined - example, where it
says "Click here to see how you can use the new and improved to help
you buy with precision.", 'here' link changes colors.

Comment 4

19 years ago
Confirmed on my 98 machine too (Build 2000030516)
Happens on too.

Bug 30607 is also very similar to this bug.
Component: Browser-General → Style System
Ever confirmed: true

Comment 5

19 years ago
I dont see visited link change color on any site. example
buildid 2000052313
linux 686 machine with gnome.
I see them working on 200052208 win98 on the menu for example.

This smells like a dupe of 30607, unless this should be turned to a linux bug?
Assigning bug to owners of style system, not the browser-general people.

Can anyone on win98 reproduce this?
Assignee: cbegle → pierre
QA Contact: asadotzler → chrisd

Comment 7

19 years ago
This does not look like a dup of 30607, which is a bug that the history is not 
matching with

Anyway, this is working on a fresh build for the Deja site - I tested a bunch of 
links and they are fine (Linux and WinNT).

Last Resolved: 19 years ago
Resolution: --- → WORKSFORME

Comment 8

19 years ago
Rechecked. OK.
Bug was gone when I removed ~/.mozilla before starting mozilla.
some old config stuff had messed up in time I guess.
30607 is about 'Visited links in targets/anchors don't change colors'...anyways 

Marking verified on win98. Reporter - deleting profile often does get rid of 
bugs. You should always do that before reporting.


Comment 10

19 years ago
*** Bug 40837 has been marked as a duplicate of this bug. ***

Comment 11

19 years ago
No wonder no one responded to my new comments... Bugzilla ate them :)
Ok I'll try this again:

This is still broken, but I have more info for you.  It only happens (links
don't change color) if you do a Power Search.  If you do a normal search, they
change.  Here's the exact steps:

Go to <>.  Click on "Search discussions", which takes you to
<>.  Click on "Power Search", which takes you to <>.  Search on anything and get the results
page.  Click the back button.  Link hasn't changed color.

Works fine in Communicator 4.73 and IE5.  Still not working as of build
2000060120.  I think this should be reopened (I'd do it myself but I'm not sure
if that's kosher).

PS: I always delete my profile info, along with the entire Mozilla directory
structure and moz*.dat in the Windows directory, before I install a new build.

Comment 12

19 years ago
It works for me. After doing a search, I click Back twice to come back to and the "Power Search" link is shown as visited.

Comment 13

19 years ago

Yes, THAT link works.  That's not what I'm talking about though.  I'm talking
about the links for the individual results for the search.  To reiterate:

Go to the Power Search page.  Type in "test" (or anything) and click on the
[Search] button.  You'll get a bunch of search results.  Click on any result.
Once the selected usenet post is up, click the [Back] button ONCE to return to
the list of search result.  The search result you followed will not have changed
to the followed-link color.

Comment 14

19 years ago
Correct. Reopening.
Resolution: WORKSFORME → ---

Comment 15

19 years ago
Reassigning to Chris Waterson. To reproduce:
- go to
- enter "test" and click Search
- click on the Subject of any article
- when the page is loaded, click the Back button
==> the link you clicked on to display the article is not shown as visited
Assignee: pierre → waterson

Comment 16

19 years ago
I think that the code in the anchor element is computing the correct URL, but it
looks like somebody in the front-end is URL escaping it. If you mouse over the
link, you'll see that the sqare brackets in the URL are maintained; however,
once you click on the URL, it escapes them to %5B. Somebody shouldn't be
escaping, here.
Target Milestone: --- → M17
Are square brackets valid characters in URLs? makes me think not
(appendix A).  Should it be instead that the escaping should be done at a lower

Comment 18

19 years ago
Currently we only allow them as part of the param, query and ref component when
the netwerk/base/src/nsURLHelper escaping code is used. With the old escaping
code in xpcom/io/nsEscape we escape it every time. [ and ] are also part of the
new ipv6 adresses but they are stripped out during parsing and put back in when
calling GetSpec or something similar. 

Comment 19

19 years ago
See my previous comment. Layout is computing the correct URL, but the front-end 
is munging it because it's "over escaping". This is either an embedding issue 
or an XPApps issue; I'll try XPApps first...
Assignee: waterson → don
Component: Style System → XP Apps
QA Contact: chrisd → sairuh
Target Milestone: M17 → ---

Comment 20

19 years ago
*** Bug 45250 has been marked as a duplicate of this bug. ***
radha, would this be yours?
Assignee: don → radha
this is an embedding issue. 
Assignee: radha → valeski
Component: XP Apps → Embedding: Docshell
over to the default qa contact...
QA Contact: sairuh → adamlock

Comment 24

19 years ago
Works for me

Comment 25

19 years ago

Other people are seeing the behavior I'm describing, and I'm still seeing it as
of 2000102304.  Can you double-check your steps in recreating the problem?  Make
sure you're doing an ADVANCED search, and note my other comments since the
initial posting of the bug.  Thanks.

Comment 26

19 years ago
Apologies, I see the problem now.

The RFC says that square brackets should be escaped:

2.4. Escape Sequences

Data must be escaped if it does not have a representation using an
unreserved character; this includes data that does not correspond to
a printable character of the US-ASCII coded character set, or that
corresponds to any US-ASCII character that is disallowed, as
explained below.

So any character which is not unreserved (alphanum | "-" | "_" | "." | "!" | "~" 
| "*" | "'" | "(" | ")") or reserved (";" | "/" | "?" | ":" | "@" | "&" | "=" | 
"+" | "$" | ",") must be escaped. 

Therefore, it is legal that Mozilla escapes the URL in the address field. 
Probably it should escape the link too but that might break javascript (see bug 
56529). The visited link history should also do unescaped string comparison so 
link colouring and caching is correct.


19 years ago
Keywords: embed


19 years ago
Assignee: valeski → radha
Component: Embedding: Docshell → History
OS: Windows NT → All
QA Contact: adamlock → claudius
Hardware: PC → All
Whiteboard: The Link's Characters include [] and other stuff that must be escaped

Comment 27

19 years ago
Adam: Ok, so I guess you're saying this should go to history.

Comment 28

19 years ago
Based on adam's and timless's comments, I'd say this is a dup of bug 
12493, "layout does not canonicalize URIs for global history lookup".  Waterson 
(or others), any objections?
nav triage team: how widespread/common is this. does it only happen in some edge 
cases? it has to be common to be a beta stopper. lowerign priority. will raise 
only if common. thanks. 
Keywords: nsbeta1
Priority: P3 → P5

Comment 30

18 years ago
This is fairly widespread... I'm surprised that no one complains but me :)

I see it also at Yahoo, AltaVista, Google, Excite, Lycos...  I'm not sure if
it's because of the same reasons as Deja (I'll leave that to those more adept
than I), but using search engines is VERY difficult when you can't keep track of
which results you've visited and which you haven't.

Comment 31

18 years ago
shows this too
click a link on the page and go back or restart mozilla -> not marked as visited
buildid: 2000121908 M18 0.6

Comment 32

18 years ago
nav triage team:

Marking nsbeta1+

And why is this stuff previously in the status whiteboard?

"The Link's Characters include [] and other stuff that must be escaped"
Whiteboard: The Link's Characters include [] and other stuff that must be escaped → nsbeta1+

Comment 33

18 years ago

*** This bug has been marked as a duplicate of 12493 ***
Last Resolved: 19 years ago18 years ago
Resolution: --- → DUPLICATE

Comment 34

18 years ago
verified dup.


11 years ago
Component: History: Session → Document Navigation
QA Contact: claudius → docshell
You need to log in before you can comment on or make changes to this bug.