Closed
Bug 1099032
(enable-writing-mode-dev)
Opened 10 years ago
Closed 10 years ago
enable CSS writing-mode support in developer channels
Categories
(Core :: Layout, defect)
Core
Layout
Tracking
()
RESOLVED
FIXED
mozilla39
People
(Reporter: jfkthame, Assigned: jfkthame)
References
(Blocks 1 open bug)
Details
(Keywords: dev-doc-complete)
Attachments
(1 file, 2 obsolete files)
(deleted),
patch
|
smontagu
:
review+
|
Details | Diff | Splinter Review |
This is intended to be the bug where we'll toggle the layout.css.vertical-text.enabled pref to true, so that vertical writing mode can actually be used.
This will not be the end of work on vertical writing mode, as the initial release will not support vertical mode for all elements or CSS constructs; e.g. tables, grid, form controls, etc., etc., may be horizontal-only at first.
Add "depends on" references here for the vertical writing-mode bugs that are essential to fix before we can usefully enable the feature. Additional aspects of vertical support that we want to implement eventually, but that need not block the initial release, should be marked as blocking (directly or indirectly) the main writing-mode bug instead of this one.
Assignee | ||
Comment 1•10 years ago
|
||
Here's the patch to enable writing-mode; we're not ready to consider landing this yet, though.
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → jfkthame
Status: NEW → ASSIGNED
Assignee | ||
Updated•10 years ago
|
Updated•10 years ago
|
Alias: enable-writing-mode
Updated•10 years ago
|
Keywords: dev-doc-needed
Assignee | ||
Comment 2•10 years ago
|
||
Note that the patch in bug 1082017 will need to land together with this bug, otherwise we'll get mochitest failures. (But it can't land earlier, because it depends on writing-mode being supported.)
Assignee | ||
Comment 3•10 years ago
|
||
(In reply to Jonathan Kew (:jfkthame) from comment #2)
> Note that the patch in bug 1082017 will need to land together with this bug,
> otherwise we'll get mochitest failures. (But it can't land earlier, because
> it depends on writing-mode being supported.)
This is no longer an issue, as that patch was modified so as to land safely, independently of the pref change here.
Assignee | ||
Comment 4•10 years ago
|
||
Time to consider flipping the pref on non-release channels (nightly/dev), so that we can begin to get wider testing of what's been implemented so far? This will also mean that the writing-mode tests start running on mozilla-central, which IMO is highly desirable.
Attachment #8566478 -
Flags: review?(smontagu)
Assignee | ||
Updated•10 years ago
|
Attachment #8522865 -
Attachment is obsolete: true
Assignee | ||
Comment 5•10 years ago
|
||
If we do bug 1134598 separately, then all that's left here is the pref change.
Attachment #8566488 -
Flags: review?(smontagu)
Assignee | ||
Updated•10 years ago
|
Attachment #8566478 -
Attachment is obsolete: true
Attachment #8566478 -
Flags: review?(smontagu)
Assignee | ||
Comment 6•10 years ago
|
||
As per discussion with jet/dbaron/heycam today, the proposal is to do this *after* next week's uplift.
Comment 7•10 years ago
|
||
... #ifndef RELEASE_BUILD, that is. With notification to firefox-dev and dev-gaia that it's enabled for nightly/aurora only, so they shouldn't depend on it yet.
Updated•10 years ago
|
Attachment #8566488 -
Flags: review?(smontagu) → review+
Assignee | ||
Updated•10 years ago
|
Alias: enable-writing-mode → enable-writing-mode-dev
Summary: enable CSS writing-mode support → enable CSS writing-mode support in developer channels
Assignee | ||
Updated•10 years ago
|
Blocks: enable-writing-mode-release
Tracking for Firefox 39.
tracking-firefox39:
--- → +
Assignee | ||
Comment 9•10 years ago
|
||
Assignee | ||
Comment 10•10 years ago
|
||
Assignee | ||
Comment 11•10 years ago
|
||
Comment 12•10 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
status-firefox39:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla39
Release Note Request (optional, but appreciated)
[Why is this notable]: Cool new feature on dev channels.
[Suggested wording]: Vertical text mode enabled by default on Developer Edition
[Links (documentation, blog post, etc)]: https://developer.mozilla.org/en-US/docs/Web/CSS/writing-mode
relnote-firefox:
--- → 39+
Jonathan, is this likely to stay enabled or be turned off when 39 moves to beta? Would exploratory testing help or does this need the automated tests for us to feel that it's ready for release? Thanks.
Flags: needinfo?(jfkthame)
Assignee | ||
Comment 15•10 years ago
|
||
For 39, and probably another cycle or two, it'll stay disabled on beta/release channels. (The pref is subject to #ifdef RELEASE_BUILD in all.js.)
Bug 1138384 will track enabling the feature on release. (It's not primarily a question of testing, but of feature completeness: there are some substantial parts of CSS layout that do not yet handle vertical mode, such as tables.)
Flags: needinfo?(jfkthame)
Comment 16•10 years ago
|
||
Updated:
https://developer.mozilla.org/en-US/docs/Web/CSS/writing-mode
and
https://developer.mozilla.org/en-US/Firefox/Releases/39#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
•