Confirmed users
112
edits
(→Intent to implement: mention standards-positions in suggested additions section) |
(Restructure this page significantly to put the more relevant information higher up and have somewhat less duplication (also remove "intent to implement and ship" as that can just be "intent to ship")) |
||
Line 1: | Line 1: | ||
= | =Adding features= | ||
One way Mozilla aims to advance the state of the web platform is with new features. Always ask: ''Is this good for the web?'' | |||
If you aim to expose a new feature to the web or change an existing feature, please follow the steps below. Note that trivial changes might not need to worry about this. | |||
# Email [https://lists.mozilla.org/listinfo/dev-platform dev-platform] declaring your [[#Intent to implement|intent to implement]] or [[#Intent to ship|intent to ship]]. If you are implementing a feature that is working its way through a standards body process such as the W3C's, it's best to email as soon as possible (i.e., before [http://www.w3.org/2005/10/Process-20051014/tr#q74 W3C CR status] if possible). | |||
# Email [https://lists.mozilla.org/listinfo/dev-platform dev-platform] declaring your [[#Intent to | |||
# Implement as normal. Code review rounds will take place, etc. Module owners or their designated peers will provide a super-review for all interface changes. | # Implement as normal. Code review rounds will take place, etc. Module owners or their designated peers will provide a super-review for all interface changes. | ||
# [https://blog.mozilla.org/security/2018/01/15/secure-contexts-everywhere/ Restrict the feature to secure contexts]. | # [https://blog.mozilla.org/security/2018/01/15/secure-contexts-everywhere/ Restrict the feature to secure contexts]. | ||
Line 46: | Line 11: | ||
# Land potentially disruptive features behind a preference which is off by default. | # Land potentially disruptive features behind a preference which is off by default. | ||
== | ==Intent to implement== | ||
= | |||
'''To''': <tt>dev-platform@lists.mozilla.org</tt><br /> | '''To''': <tt>dev-platform@lists.mozilla.org</tt><br /> | ||
'''Subject''': Intent to implement: <your feature goes here> | '''Subject''': Intent to implement: <your feature goes here> | ||
''Summary'': [https://en.wikipedia.org/wiki/Elevator_pitch elevator pitch] for the new functionality including benefits to web developers<br /> | ''Summary'': [https://en.wikipedia.org/wiki/Elevator_pitch elevator pitch] for the new functionality including benefits to web developers.<br /> | ||
''Bug'': link to | ''Bug'': link to Bugzilla (tracking) bug.<br /> | ||
'' | ''Standard'': link to standard or link to public discussions with other browser vendors.<br /> | ||
''Platform coverage'': where will this be available? | ''Platform coverage'': where will this be available? Android, Desktop, only exposed to privileged apps (certified app-only functionality does not require an email), etc.<br /> | ||
''Preference'': if applicable, how can interested parties test this before it ships pref'd on by default?<br /> | |||
''Preference | |||
''DevTools bug'': link to a [https://bugzilla.mozilla.org/enter_bug.cgi?product=Firefox&component=Developer%20Tools Firefox :: Developer Tools bug] coordinating work with the DevTools team to build tools for this feature.<br /> | ''DevTools bug'': link to a [https://bugzilla.mozilla.org/enter_bug.cgi?product=Firefox&component=Developer%20Tools Firefox :: Developer Tools bug] coordinating work with the DevTools team to build tools for this feature.<br /> | ||
'' | ''Other browsers'': address with shipped (since version X, behind what flags if any), intent to implement emailed (mailing list URL), or considering (citation).<br /> | ||
''web-platform-tests'': Please link to the test suite. If any part of the feature is not tested by web-platform-tests, please include links to one or more of: | ''web-platform-tests'': Please link to the test suite. If any part of the feature is not tested by web-platform-tests, please include links to one or more of: | ||
* A web-platform-tests issue explaining why a certain thing cannot be tested ([https://github.com/w3c/web-platform-tests/issues/3867 example]). | * A web-platform-tests issue explaining why a certain thing cannot be tested ([https://github.com/w3c/web-platform-tests/issues/3867 example]). | ||
* A spec issue for some change that would make it possible to test ([https://github.com/whatwg/fullscreen/issues/70 example]). | * A spec issue for some change that would make it possible to test ([https://github.com/whatwg/fullscreen/issues/70 example]). | ||
* A Bugzilla bug for the creating web-platform-tests. | * A Bugzilla bug for the creating web-platform-tests. | ||
'' | ''Secure contexts'': Provide justification if this feature is not restricted to secure contexts. Otherwise, use "Yes". | ||
''Is this feature enabled by default in sandboxed iframes?'' If not, is there a proposed sandbox flag to enable it? If allowed, does it preserve the current invariants in terms of what sandboxed iframes can do?<br/> | |||
===Suggested additions=== | |||
The above is the minimum required that should be in an "Intent to implement" email. If you've covered those, you're good, and brevity is a virtue. | The above is the minimum required that should be in an "Intent to implement" email. If you've covered those, you're good, and brevity is a virtue. | ||
Line 97: | Line 42: | ||
==Intent to ship== | ==Intent to ship== | ||
'''To''': <tt>dev-platform@lists.mozilla.org</tt><br /> | '''To''': <tt>dev-platform@lists.mozilla.org</tt><br /> | ||
'''Subject''': Intent to ship: <your feature goes here> | '''Subject''': Intent to ship: <your feature goes here> | ||
Line 104: | Line 50: | ||
''Bug to turn on by default'': link to main relevant bug (https://bugzilla.mozilla.org/show_bug.cgi?id=) ['''note''': this bug should ideally have the ''dev-doc-needed'' keyword set]<br /> | ''Bug to turn on by default'': link to main relevant bug (https://bugzilla.mozilla.org/show_bug.cgi?id=) ['''note''': this bug should ideally have the ''dev-doc-needed'' keyword set]<br /> | ||
This feature was previously discussed in this " | This feature was previously discussed in this "Intent to implement" thread: <https://groups.google.com/forum/#!forum/mozilla.dev.platform>. '''If anything changed since that thread please include that information in this email'''. | ||
It's acceptable to merge the "intent to implement" into the "intent to ship" email as long as all the relevant requirements are met. | |||
=Removing features= | |||
Occasionally there is a need to remove features. These could be features that are proprietary to Mozilla, were standardized but never got adopted by all browsers, or features that are so bad that all browsers want to get rid of them together. | |||
In order to make this possible while impacting users as little as possible the following steps are to be taken: | |||
* Gather telemetry on the feature. | |||
* Consult dev-platform with an [[#Intent to unship|intent to unship]] that includes the telemetry data. | |||
* Indicate in the developer console whenever the feature is used that it's deprecated and might be removed. It's best to avoid doing this until there's agreement that the feature can be removed (which requires telemetry) as otherwise developers are needlessly spammed in the console. | |||
* Remove the feature when the telemetry data and dev-platform agreement indicate it can be. | |||
==Intent to unship== | ==Intent to unship== | ||
Line 125: | Line 78: | ||
* [https://www.chromium.org/blink Chrome/Blink change process] | * [https://www.chromium.org/blink Chrome/Blink change process] | ||
** [https://www.chromium.org/blink/launching-features Chrome/Blink process for Launching Features] in particular | ** [https://www.chromium.org/blink/launching-features Chrome/Blink process for Launching Features] in particular | ||
=FAQ= | |||
==When is a feature ready to ship?== | |||
Features which Mozilla makes available by default to the web on the release channel should be "ready". This can mean that they are de jure standards (e.g., a W3C Recommendation) or de facto standards. Indications that a feature is ready for shipping to the web include: | |||
# other browser engines ship compatible implementations in their releases or behind a preference with clear signals it will graduate to being on by default | |||
# other browser engines state their ''intention'' to ship a compatible implementation | |||
# a large number of web developers indicate their satisfaction with the feature design | |||
# there exists a specification that is no longer at risk of significant changes, is on track to become a standard with a relevant standards body, and is acceptable to a number of applicable parties and other browser engines | |||
==How do we know what web developers want?== | |||
* Mozilla's [[Engagement/Developer_Engagement|Developer Engagement]] coordinates a [https://openwebapps.uservoice.com/forums/258478-open-web-apps Open Web Apps UserVoice] forum | |||
* [http://chromestatus.com chromestatus.com] has a column for new features entitled "Documented or perceived web developer views" | |||
==How do we know what other browser engines think?== | |||
* some of them participate in dev-platform discussions | |||
* watch for positive signals on the specification's discussion forum (likely a GitHub repository) | |||
* watch for "intent to implement" emails on mailing lists such as [https://groups.google.com/a/chromium.org/forum/#!forum/blink-dev blink-dev] | |||
* lack of feedback will not delay our implementation as it may simply indicate lack of interest at that time from another browser engine | |||
==How do we let other browser engines know what we think?== | |||
* By documenting our position on [standards-positions https://github.com/mozilla/standards-positions]. | |||
* Participating in public discussions of new features. | |||
* Comment on "intent to implement" threads on [https://groups.google.com/a/chromium.org/forum/#!forum/blink-dev blink-dev]. | |||
==What about prefixes?== | |||
In the past, Mozilla has shipped experimental features with a "moz" prefix to indicate their lack of standardization (e.g., <code>mozRequestAnimationFrame()</code>). Unfortunately, this approach turned out to be harmful to the web as experimental features ended up being used in some websites before they were ready. In many cases, this meant that we were unable to innovate on certain features because to change them would break content on the web. Browsers have in some cases also been [https://compat.spec.whatwg.org/ forced to implement each other's prefixed features]. Therefore, to allow us to continue innovating without negatively affecting content on the web, '''Mozilla will no longer ship new "moz"-prefixed features''' (see [https://groups.google.com/forum/#!topic/mozilla.dev.platform/34JfwyEh5e4 Henri Sivonen's proposal]). | |||
We will instead prototype features behind preferences which can be toggled through <code>about:config</code>. Once we feel there is an acceptable level of consensus in the web community about the stability of an feature and we feel it is ready, we will make it generally available to the web platform (more details below on this process). We feel this strikes the right balance between allowing developers to experiment with new features, while at the same time protecting the web from being exposed to new functionality prematurely. | |||
==Who decides?== | |||
If the dev-platform thread results in a conflict, the respective [[Modules|module owner]] is responsible for resolving that conflict and making a decision on how to proceed. |