Closed
Bug 1341925
Opened 7 years ago
Closed 7 years ago
Restrict :-moz-table-border-nonzero pseudo-class to UA stylesheet
Categories
(Core :: CSS Parsing and Computation, defect)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
FIXED
mozilla54
Tracking | Status | |
---|---|---|
firefox54 | --- | fixed |
People
(Reporter: xidorn, Assigned: xidorn)
References
(Blocks 1 open bug)
Details
(Keywords: dev-doc-complete, site-compat)
Attachments
(1 file)
Searching our codebase, it is something that is only used in our UA stylesheet html.css. No chrome usage and addon usage at all. Searching the web also returns no result indicating anyone is using it on their page. Apparently this isn't something useful for authors. So I purpose that we restrict this pseudo-class to UA stylesheet.
Comment 2•7 years ago
|
||
You probably mean nonzero, not non-zero. Search results: https://www.google.com/search?q=%22moz-table-border-nonzero%22
Summary: Restrict :-moz-table-border-non-zero pseudo-class to UA stylesheet → Restrict :-moz-table-border-nonzero pseudo-class to UA stylesheet
Assignee | ||
Comment 3•7 years ago
|
||
(I have no idea how I could misspell it... Yes, it is :-moz-table-border-nonzero, good catch.)
Assignee | ||
Comment 4•7 years ago
|
||
If we want to continue testing the behavior of this pseudo-class like what we currently do in test_selectors.html, we would need to use nsStyleSheetService to inject UA sheet, which needs some work.
Comment hidden (mozreview-request) |
Comment 6•7 years ago
|
||
mozreview-review |
Comment on attachment 8841865 [details] Bug 1341925 - Restrict :-moz-table-border-nonzero to UA sheet. https://reviewboard.mozilla.org/r/115954/#review117600 r=me with one request: ::: layout/style/nsCSSPseudoClassList.h:124 (Diff revision 1) > CSS_PSEUDO_CLASS(mozWindowInactive, ":-moz-window-inactive", 0, "") > > // Matches any table elements that have a nonzero border attribute, > // according to HTML integer attribute parsing rules. > -CSS_PSEUDO_CLASS(mozTableBorderNonzero, ":-moz-table-border-nonzero", 0, "") > +CSS_PSEUDO_CLASS(mozTableBorderNonzero, ":-moz-table-border-nonzero", > + CSS_PSEUDO_CLASS_ENABLED_IN_UA_SHEETS, "") I want to be sure we have tests that ensure we aren't fully-disabling this pseudoclass (and that we don't accidentally do that in the future). For example: I suspect we could (or might already) test this using a "!=" reftest that compares <table> vs <table border>. Or something like that. So: before landing, please make sure our usages of this pseudoclass are tested -- e.g. you might do a Try run with this pseudo-class *fully disabled* in UA sheets (maybe by using CSS_PSEUDO_CLASS_ENABLED_IN_CHROME instead of IN_UA_SHEETS in the line that you're modifying). And hopefully some tests will fail in that Try run -- but if none do, please add test(s) that would.
Attachment #8841865 -
Flags: review?(dholbert) → review+
Assignee | ||
Comment 7•7 years ago
|
||
Looks like there are already lots of tests check this: https://treeherder.mozilla.org/#/jobs?repo=try&revision=098c9fea7362e359cdb65dec745136f26ba964c9&selectedJob=80687580
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → xidorn+moz
Pushed by xquan@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/d038d48f6be1 Restrict :-moz-table-border-nonzero to UA sheet. r=dholbert
Comment 9•7 years ago
|
||
Posted the site compatibility doc: https://www.fxsitecompat.com/en-CA/docs/2017/moz-table-border-nonzero-pseudo-class-has-been-removed/
Comment 10•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/d038d48f6be1
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox54:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla54
Comment 11•7 years ago
|
||
We never documented this on MDN; I've added a note to the Fx 54 release notes: https://developer.mozilla.org/en-US/Firefox/Releases/54#CSS
Keywords: dev-doc-needed → dev-doc-complete
You need to log in
before you can comment on or make changes to this bug.
Description
•