93598 - Implement Complete CSS1 Anchor Pseudo-Classes For Simple XLink Links in XML Documents

Status: VERIFIED WORKSFORME

(Core :: CSS Parsing and Computation, defect, P1)

Description

[Greg K.]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (Macintosh; U; PPC; en-US; rv:0.9.3) Gecko/20010802
BuildID:    2001080214

The three CSS1 Anchor Pseudo-Classes should apply to links created using XLink
in XML documents. At present, only :link and :active appear to work. :visited
should be implemented.

The complete pseudo-classes are ":link", ":visited", and ":active", per
[http://www.w3.org/TR/REC-CSS1#anchor-pseudo-classes].

Reproducible: Always
Steps to Reproduce:
1. Access the attached testcase
2. Observe that the link is green, as :link is defined
3. Click the link; notice the yellow color, which is :active.
4. Follow the link, and return to the testcase. Notice that the link, though
visited, does not display in the :visited style, but rather the :link style.

Actual Results:  The ":visited" CSS1 Anchor Pseudo-Class style is not displayed.

Expected Results:  The ":visited" CSS1 Anchor Pseudo-Class style should be
displayed.

Comment 1

[Greg K.]

Attachment: Testcase of CSS1 Anchor Pseudo-Classes as applied to an XLink element in an XML document

Comment 2

[Heikki Toivonen (remove -bugzilla when emailing directly)]

Haven't checked this, but if the implementation of a property expects us to
store some style value in the element itself, then it will be automatically
broken for XLinks. Bug 76669 might have a fix for this as well.

Comment 3

[Kevin McCluskey (gone)]

Bulk moving from Moz1.1 to future-P1. I will pull from this list when scheduling
work post Mozilla1.0.

Comment 4

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

See also bug 93602.

Comment 5

[Madhur Bhatia]

cc'ing myself

Comment 6

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Assigning pierre's remaining Style System-related bugs to myself.

Comment 7

[Noah Sager]

This affects more than just XML documents. Basic HTML documents that use the
pseudo-classes from CSS are also having the same issues. The pseudo-classes
involving links are very important.

This issue is also present in the Mac OS X build. The current build of this
posting is 20020803.

Comment 8

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Both the :active and :visited rules in the testcase work correctly for me in
Linux Mozilla 1.0 and in a current Linux trunk build.  I recall that they didn't
when I tested bug 93602, so I'm marking as WORKSFORME.

(I don't know why they wouldn't have worked -- this should have worked for ages.
 The code is slightly slower for XLinks (although only in that it retests the
link state when we do style reresolution or event state changes), and there's a
bug on improving the performance.)

Comment 9

[Greg K.]

Verified fixed using Mac/2002080208/9.2.2. (Note that :active works if you make
it the third of the rules, following :visited.)

Comment 10

[Anne (:annevk)]

v