Closed Bug 470396 Opened 17 years ago Closed 17 years ago

CSS generated counters handled inconsistently

Categories

(Core :: CSS Parsing and Computation, defect)

1.9.0 Branch
x86
Linux
defect
Not set
normal

Tracking

()

RESOLVED INVALID

People

(Reporter: szycha, Unassigned)

Details

Attachments

(2 files)

User-Agent: Mozilla/5.0 (X11; U; Linux i686; pl; rv:1.9.0.3) Gecko/2008092416 Firefox/3.0.3 Build Identifier: Mozilla/5.0 (X11; U; Linux i686; pl; rv:1.9.0.3) Gecko/2008092416 Firefox/3.0.3 CSS generated counters require initialization by {counter-reset: counterName;} property (as widely discussed under bug 288946). Otherwise counter won't increment, displaying '1' along the document. However applying {counter-increment: counterName;} to tr (and maybe others) elements results in 'proper' (?) counter implementation. See attachments Reproducible: Always Steps to Reproduce: 1. Open attachment 1 [details] [diff] [review] 2. Open attachment 2 [details] [diff] [review] 3. Compare sources Actual Results: Firefox treats td and tr elements inconsistently, allowing incrementation of uninitialized (not yet reset) counters on tr elements, while not incrementing such counters Page 1 displays: 1. 1. 1. 1. Page 2 displays: 1. 2. 3. 4. Expected Results: Both Page 1 and Page 2 should display the same, shouldn't them? Please do not mark this bug as duplicate of 288946 too quickly.
Attached file example Page 1
Counter is not reset and is not incremented on td, td.someclass elements
Attached file example Page 2
Counter is not reset, however increments on tr elements
Version: unspecified → 3.0 Branch
Component: General → Style System (CSS)
Product: Firefox → Core
QA Contact: general → style-system
Version: 3.0 Branch → 1.9.1 Branch
Version: 1.9.1 Branch → 1.9.0 Branch
Problem persists as of Fx 3.0.5
The counter-increment on the TR elements establishes a scope on the TBODY, so this behavior seems correct to me. If you disagree, please describe which testcase you think should be handled differently and what you think the results should be.
Status: UNCONFIRMED → RESOLVED
Closed: 17 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: