Closed
Bug 1253919
Opened 9 years ago
Closed 8 years ago
enable support for logical keywords (inline-start, inline-end) for 'float' and 'clear' properties
Categories
(Core :: CSS Parsing and Computation, defect)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
FIXED
mozilla55
Tracking | Status | |
---|---|---|
firefox55 | --- | fixed |
People
(Reporter: jfkthame, Assigned: jfkthame)
References
(Blocks 1 open bug)
Details
(Keywords: dev-doc-complete)
Attachments
(1 file)
(deleted),
patch
|
dbaron
:
review+
|
Details | Diff | Splinter Review |
In bug 1122918, we implemented these keywords, but they're preffed off for release channels (except on B2G) because it wasn't clear how stable the spec is... in particular, should the values be 'inline-start' and 'inline-end' or simply 'start' and 'end'?
Attempts to ask the CSSWG for clarification led to extensive discussion, but I don't recall a clear resolution emerging. Both [1] and [2] currently have the 'inline-*' forms, matching what we landed in bug 1122918, but it's also still flagged as an open issue in [1].
[1] https://drafts.csswg.org/css-logical-props/#float-clear
[2] https://drafts.csswg.org/css-page-floats/#float-property
Updated•9 years ago
|
Keywords: dev-doc-needed
Assignee | ||
Comment 1•9 years ago
|
||
Elika (as one of the editors of logical-props), do you have any sense of where this stands?
Flags: needinfo?(fantasai.bugs)
Comment 2•8 years ago
|
||
Discussed a little at this week's CSS WG teleconference, but will try to revisit more seriously next week.
Updated•8 years ago
|
Updated•8 years ago
|
Blocks: css-logical-props
Comment 3•8 years ago
|
||
Since I've landed few patches to Firefox that use these keywords, and there doesn't seems to be any progress in the last half a year, is there any progress? If not, is it possible to implement this as a prefixed -moz- keywords even if restricted to the browser chrome?
Assignee | ||
Comment 4•8 years ago
|
||
The CSS WG just made a decision on this, noted at https://log.csswg.org/irc.w3.org/css/2017-04-20/#e800353.
The spec[1] will no doubt be updated soon to resolve the open issue, and as this decision matches what we implemented, I think we can now move forward with shipping these values across all channels.
[1] https://drafts.csswg.org/css-logical-props/#float-clear
Assignee | ||
Comment 5•8 years ago
|
||
This just makes the pref default to enabled everywhere; once we've done this, assuming nothing unexpectedly breaks, we can follow up fairly soon with a patch to remove the pref completely.
Attachment #8860346 -
Flags: review?(dbaron)
Assignee | ||
Updated•8 years ago
|
Assignee: nobody → jfkthame
Status: NEW → ASSIGNED
Assignee | ||
Comment 6•8 years ago
|
||
Filed bug 1358450 about removing the pref. By keeping this bug to the minimal pref-flipping patch, rather than the actual code removal, perhaps we can even consider uplifting it to ship sooner, so that front-end code will be free to start using these values (as per comment 3).
Updated•8 years ago
|
Flags: needinfo?(fantasai.bugs)
Attachment #8860346 -
Flags: review?(dbaron) → review+
Assignee | ||
Comment 7•8 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/024606c3a9503ace4c89f5aba6ddf01dd5db6ee9
Bug 1253919 - Enable support for the logical values of CSS 'float' and 'clear' properties for all channels. r=dbaron
Comment 8•8 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox55:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
Comment 9•8 years ago
|
||
I've updated the browser supporting information on the ref pages:
https://developer.mozilla.org/en-US/docs/Web/CSS/float
https://developer.mozilla.org/en-US/docs/Web/CSS/clear
Removed the entry from the experimental features page:
https://developer.mozilla.org/en-US/Firefox/Experimental_features
And added an entry to the Fx55 release notes:
https://developer.mozilla.org/en-US/Firefox/Releases/55#CSS
I think that covers it; let me know if you think anything else is needed. Thanks!
Keywords: dev-doc-needed → dev-doc-complete
You need to log in
before you can comment on or make changes to this bug.
Description
•