Skip to content

Remove Notification's vibrate feature#241

Open
marcoscaceres wants to merge 3 commits intowhatwg:mainfrom
marcoscaceres:remove_vibration
Open

Remove Notification's vibrate feature#241
marcoscaceres wants to merge 3 commits intowhatwg:mainfrom
marcoscaceres:remove_vibration

Conversation

@marcoscaceres
Copy link
Member

@marcoscaceres marcoscaceres commented Mar 13, 2026

Remove vibration support from notifications

This PR removes vibration support from notifications.

The feature does not have interoperable browser implementations.

Current implementation status appears to be:

As a result, the feature is effectively a single-engine extension.

The generic Vibration API (Navigator.vibrate) remains available for user agents that support vibration.

If there is implementer interest in maintaining this feature, that information would be helpful in determining whether it should remain in the specification.

  • At least two implementers are interested (and none opposed):
    • WebKit: feature not implemented
  • Tests are written and can be reviewed and commented upon at:
  • Implementation bugs are filed:
  • MDN issue is filed: …
  • The top of this comment includes a clear commit message to use.

(See https://whatwg.org/working-mode#changes for more details.)(https://github.com/whatwg/meta/blob/main/COMMITTING.md) to use.

Vibration via notifications does not appear to have interoperable browser support.

Current implementation status appears to be:

* Chromium-based browsers: implemented for Android.
* Firefox: [not implemented](https://bugzilla.mozilla.org/show_bug.cgi?id=1273641) after 9 years of development.
* WebKit: not implemented.

As a result, the feature is not interoperable across browser engines.

If there is implementer interest in maintaining this feature,
that information would be helpful in determining whether it should remain
in the specification.
notifications.bs Outdated
Translation: ja https://triple-underscore.github.io/notifications-ja.html
Translation: zh-Hans https://htmlspecs.com/notifications/
Translation: ko https://ko.htmlspecs.com/notifications/
Ignore MDN Failure: #dom-notification-vibrate
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should fix this on the MDN side, ideally before landing.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

agree... was just making the bot happy.

@marcoscaceres
Copy link
Member Author

@saschanaz, is this ok on with you?

@foolip, you ok with this change from the Chrome side?

@saschanaz
Copy link
Member

Moz is happy with the removal.

Comment on lines +84 to +85
initially null. When true, indicates that no sounds should be made. When null, indicates that
producing sounds should be left to platform conventions.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be kept intact, because notifications can still make vibrations regardless of Vibration API?

<a for=notification>vibration pattern</a>, if any.
</ol>
</div>
<p>The <dfn>alert steps</dfn> for alerting the end user about a given <a for=/>notification</a> are
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not necessarily something should be done in this PR, but this can probably just be inlined.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants