Skip to content

Remove Notification's vibrate feature#241

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

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

Conversation

@marcoscaceres
Copy link
Copy Markdown
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.

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


💥 Error: 422 Unprocessable Entity 💥

PR Preview failed to build. (Last tried on May 9, 2026, 12:44 PM UTC).

More

PR Preview relies on a number of web services to run. There seems to be an issue with the following one:

🚨 Spec Generator - Spec Generator is the web service used to build bikeshed/ReSpec specs

🔗 Related URL

Error output:

[
    {
        "lineNum": null,
        "messageType": "warning",
        "text": "Skipped generating some MDN panels, because the following IDs weren't present in the document. Use `Ignore MDN Failure` if this is expected.\n  #dom-notification-vibrate"
    },
    {
        "lineNum": null,
        "messageType": "failure",
        "text": "Did not generate, due to errors exceeding the allowed error level."
    }
]

This seems to be an issue with the Spec Generator service. PR Preview doesn't manage this service and so has no control over it. If you've identified an issue with it, you can report the issue to the maintainers of Spec Generator directly. Please be courteous. Thank you!

If you don't have enough information above to solve the error by yourself or if the issue doesn't seem related to Spec Generator, you can file an issue with PR Preview.

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.
Comment thread 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
Copy Markdown
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
Copy Markdown
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.

Comment thread notifications.bs Outdated
@marcoscaceres
Copy link
Copy Markdown
Member Author

@saschanaz, is this ok on with you?

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

@saschanaz
Copy link
Copy Markdown
Member

Moz is happy with the removal.

Comment thread notifications.bs Outdated
Comment thread notifications.bs Outdated
- Remove `Ignore MDN Failure` suppressor for #dom-notification-vibrate;
  the MDN data should be fixed upstream instead
- Restore "or vibrations" to the silent preference description; the
  silent flag suppresses vibrations independently of the vibrate feature
The algorithm had a single step, so inline it at the call site and
remove the "Alerting the end user" section.
Copy link
Copy Markdown
Member

@saschanaz saschanaz left a comment

Choose a reason for hiding this comment

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

Thanks!

@marcoscaceres
Copy link
Copy Markdown
Member Author

Can this be merged? @foolip ping?

@foolip
Copy link
Copy Markdown
Member

foolip commented May 8, 2026

I haven't worked on this feature in Chromium, looping in @beverloo who originally shipped this:
https://groups.google.com/a/chromium.org/g/blink-dev/c/0mBO8Q5FhTo/m/2OagzgagBgAJ

I can't find use counters for this so I don't know anything about feasibility of removal.

For now, please assume Google would object to removal. I understand that won't change implementer interest, but we want everything we ship to be spec'd somewhere, so some action is needed here.

@foolip
Copy link
Copy Markdown
Member

foolip commented May 8, 2026

cc @Dp-Goog @scheib

@marcoscaceres
Copy link
Copy Markdown
Member Author

happy to move it to compat spec if that helps.

@annevk
Copy link
Copy Markdown
Member

annevk commented May 8, 2026

I'm not sure how that helps?

@Dp-Goog
Copy link
Copy Markdown

Dp-Goog commented May 8, 2026

I'm going to defer to @beverloo here as someone who helped ship the original implementation, and as someone with more knowledge here to make the final decision about supporting this feature.

Like @foolip has said above, I can't seem to find a lot of metrics about its usage, but allowing web developers to show notifications and support vibrations patterns through the API seems like a reasonable behavior to be supported.

@beverloo
Copy link
Copy Markdown
Member

beverloo commented May 8, 2026

Android deprecated support for per-notification vibration patterns with Android 8, so availability will be rare at best now. No concerns at all with this removal.

@foolip
Copy link
Copy Markdown
Member

foolip commented May 8, 2026

Thanks @beverloo! Nothing further from me either, if it's gone from Android I'm sure we can eventually remove it from Chromium.

@marcoscaceres
Copy link
Copy Markdown
Member Author

marcoscaceres commented May 8, 2026

Filed MDN issue mdn/mdn#825 cc @hamishwillee for guidance

@marcoscaceres
Copy link
Copy Markdown
Member Author

Could someone please file a Chrome bug for me?

@Dp-Goog
Copy link
Copy Markdown

Dp-Goog commented May 8, 2026

@marcoscaceres created crbug.com/511304741 for you.

@annevk
Copy link
Copy Markdown
Member

annevk commented May 9, 2026

@whatwg/documentation could you all help with updating MDN so it no longer references this document for the vibrate feature? That way we can land this PR and remove it completely.

@annevk annevk added do not merge yet Pull request must not be merged per rationale in comment impacts documentation Used by documentation communities, such as MDN, to track changes that impact documentation labels May 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do not merge yet Pull request must not be merged per rationale in comment impacts documentation Used by documentation communities, such as MDN, to track changes that impact documentation

Development

Successfully merging this pull request may close these issues.

6 participants