Closed
Bug 1355734
Opened 8 years ago
Closed 8 years ago
Consider removing -moz-anchor-decoration magic keyword
Categories
(Core :: CSS Parsing and Computation, enhancement)
Tracking
()
RESOLVED
FIXED
mozilla55
Tracking | Status | |
---|---|---|
firefox55 | --- | fixed |
People
(Reporter: xidorn, Assigned: xidorn)
References
Details
(Keywords: dev-doc-complete, Whiteboard: [stylo])
Attachments
(1 file)
We have a magic keyword -moz-anchor-decoration for text-decoration-line that, it would reset text-decoration-line to none if user sets the preference to not underline links.
This was added in bug 57757, but it is not clear why it is still needed, since it seems user pref stylesheet is enabled everywhere.
Updated•8 years ago
|
Whiteboard: [stylo]
Comment hidden (mozreview-request) |
Comment 2•8 years ago
|
||
mozreview-review |
Comment on attachment 8857829 [details]
Bug 1355734 - Remove text-decoration-line: -moz-anchor-decoration.
https://reviewboard.mozilla.org/r/129848/#review132418
::: commit-message-5e37b:1
(Diff revision 1)
> +Bug 1355734 - Remove text-decoration-line: -moz-anchor-deciration. r?masayuki,heycam
-moz-anchor-decoration
Attachment #8857829 -
Flags: review?(cam) → review+
Comment 3•8 years ago
|
||
mozreview-review |
Comment on attachment 8857829 [details]
Bug 1355734 - Remove text-decoration-line: -moz-anchor-decoration.
https://reviewboard.mozilla.org/r/129848/#review132456
This isn't enough to remove the feature entirely.
You need to remove nsPresContext::mUnderlineLinks and kPresContext_UnderlineLinks. Additionally, you need to remove the pref from all.js and setting UI in color.xul.
I wonder, did you talk about this to Thunderbird and SeaMonkey people? At least, you should notify them of this bug.
Attachment #8857829 -
Flags: review?(masayuki) → review-
Comment 4•8 years ago
|
||
CCing :jaws who manages preference UI.
And I'd be happy if you'd add example of the user stylesheet into the commit message.
Assignee | ||
Comment 5•8 years ago
|
||
I'm not removing the underline link feature. I'm only removing the magic keyword, which, I believe, is no longer necessary for that feature.
Comment 6•8 years ago
|
||
And the reason it's not necessary is that we take it into account when building the pref style sheet:
http://searchfox.org/mozilla-central/rev/944f87c575e8a0bcefc1ed8efff10b34cf7a5169/layout/style/nsLayoutStylesheetCache.cpp#924-929
Comment 7•8 years ago
|
||
mozreview-review |
Comment on attachment 8857829 [details]
Bug 1355734 - Remove text-decoration-line: -moz-anchor-decoration.
https://reviewboard.mozilla.org/r/129848/#review132460
Ah, I see.
Attachment #8857829 -
Flags: review- → review+
Comment hidden (mozreview-request) |
Comment 9•8 years ago
|
||
We're sorry, Autoland could not rebase your commits for you automatically. Please manually rebase your commits and try again.
hg error in cmd: hg rebase -s 889991bbb65c -d 2df7d0984161: rebasing 389599:889991bbb65c "Bug 1355734 - Remove text-decoration-line: -moz-anchor-decoration. r=heycam,masayuki" (tip)
merging layout/style/test/stylo-failures.md
warning: conflicts while merging layout/style/test/stylo-failures.md! (edit, then use 'hg resolve --mark')
unresolved conflicts (see hg resolve, then hg rebase --continue)
Assignee | ||
Updated•8 years ago
|
Assignee: nobody → xidorn+moz
Comment hidden (mozreview-request) |
Comment 11•8 years ago
|
||
Pushed by xquan@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/52ba40a2c133
Remove text-decoration-line: -moz-anchor-decoration. r=heycam,masayuki
Comment 12•8 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox55:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
Comment 13•7 years ago
|
||
Keywords: dev-doc-needed
Comment 14•7 years ago
|
||
(In reply to Kohei Yoshino [:kohei] from comment #13)
> https://developer.mozilla.org/en-US/docs/Web/CSS/text-decoration-line
I've deleted it from the reference page, and also added a note to the Fx55 rel notes:
https://developer.mozilla.org/en-US/Firefox/Releases/55#Removals_from_the_web_platform
Keywords: dev-doc-needed → dev-doc-complete
You need to log in
before you can comment on or make changes to this bug.
Description
•