_moz_dirty should be renamed to -moz-dirty

VERIFIED INVALID

Status

()

Core
Editor
P2
normal
VERIFIED INVALID
17 years ago
16 years ago

People

(Reporter: Akkana Peck, Assigned: Akkana Peck)

Tracking

Trunk
x86
Linux
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [nsbeta3+][p:2])

Attachments

(1 attachment)

(Assignee)

Description

17 years ago
It looks like we're settling on hyphens rather than underscores for special moz
attributes -- see bug 3935.  In some cases we obey this, in other cases, like
_moz_dirty, we use underscores.  We should be consistent and switch to hyphens.
(Assignee)

Comment 1

17 years ago
Accepting.  Beth, this is easy cleanup work, would be nice to get it in before
release but probably not critical since it's not easily user visible.
Status: NEW → ASSIGNED

Comment 2

17 years ago
m19
Keywords: correctness
Target Milestone: --- → M19

Comment 3

17 years ago
this is a must fix, by not doing so will result in invlaid html on output, 
Akkana says this is a low risk fix. This was reviewed by Bijal and beppe, 
setting to p2, nsbeta3+
Priority: P3 → P2
Whiteboard: [nsbeta3+][p:2]
(Assignee)

Comment 4

17 years ago
Created attachment 14961 [details] [diff] [review]
a fix
(Assignee)

Comment 5

17 years ago
Kin pointed out that the -moz- style is only standard in CSS.  Outside of CSS,
it's more standard to use underscores.  We used lxr to look at all the
occurrences of both _moz_ and -moz-, and agreed that we're probably better off
not changing this after all.  Marking invalid.
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → INVALID

Comment 6

17 years ago
verified in 9/20 build.
Status: RESOLVED → VERIFIED
(Assignee)

Comment 7

17 years ago
Sorry for the spam!  But apparently all these closed bugs need to have their
target milestones changed since M19 and M20 are going away.  Since they're
allready closed, I'm choosing M18.
Target Milestone: M19 → M18
You need to log in before you can comment on or make changes to this bug.