Skip to content

Firefox 122 added screen-wake-lock permission#26566

Merged
Elchi3 merged 4 commits intomdn:mainfrom
queengooborg:api/Permissions/permission_screen-wake-lock/firefox-corrections
Aug 15, 2025
Merged

Firefox 122 added screen-wake-lock permission#26566
Elchi3 merged 4 commits intomdn:mainfrom
queengooborg:api/Permissions/permission_screen-wake-lock/firefox-corrections

Conversation

@queengooborg
Copy link
Contributor

This PR updates and corrects version values for Firefox and Firefox Android for the permission_screen-wake-lock member of the Permissions API. The data comes from the mdn-bcd-collector project (v10.12.11).

Check out the collector's guide on how to review this PR.

Tests Used: https://mdn-bcd-collector.gooborg.com/tests/api/Permissions/permission_screen-wake-lock

This PR updates and corrects version values for Firefox and Firefox Android for the `permission_screen-wake-lock` member of the `Permissions` API. The data comes from the [mdn-bcd-collector](https://mdn-bcd-collector.gooborg.com) project (v10.12.11).

_Check out the [collector's guide on how to review this PR](https://github.com/openwebdocs/mdn-bcd-collector#reviewing-bcd-changes)._

Tests Used: https://mdn-bcd-collector.gooborg.com/tests/api/Permissions/permission_screen-wake-lock
@queengooborg queengooborg added the data:api Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API label Apr 20, 2025
@github-actions github-actions bot added the size:xs [PR only] 0-6 LoC changed label Apr 20, 2025
@caugner caugner changed the title Update Firefox data for api.Permissions.permission_screen-wake-lock Firefox 122 added screen-wake-lock permission Apr 22, 2025
Copy link
Contributor

@caugner caugner left a comment

Choose a reason for hiding this comment

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

Copy link
Contributor

@caugner caugner left a comment

Choose a reason for hiding this comment

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

Actually, #22583 suggests this was removed in Firefox 124.

Copy link
Contributor

@caugner caugner left a comment

Choose a reason for hiding this comment

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

I can confirm that the collector tests pass in Chrome Firefox 122 to 126.

Could we replace this support statement by two:

  1. Full support from 126
  2. Partial support from 122, removed in 126, with notes: "The permission is available, but has no effect."

@caugner caugner marked this pull request as draft May 13, 2025 17:15
@queengooborg queengooborg marked this pull request as ready for review July 23, 2025 05:45
@github-actions
Copy link
Contributor

github-actions bot commented Jul 23, 2025

Tip: Review these changes grouped by change (recommended for most PRs), or grouped by feature (for large PRs).

Copy link
Member

@Elchi3 Elchi3 left a comment

Choose a reason for hiding this comment

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

Do you want to use a range with partial implementation for Fx 122-126?

@caugner
Copy link
Contributor

caugner commented Aug 14, 2025

Do you want to use a range with partial implementation for Fx 122-126?

I do think a partial implementation range from Fx 122-126 would be more correct, but I don't feel strongly about this.

@queengooborg Wdyt?

@github-actions github-actions bot added size:s [PR only] 7-24 LoC changed and removed size:xs [PR only] 0-6 LoC changed labels Aug 14, 2025
@Elchi3 Elchi3 dismissed caugner’s stale review August 15, 2025 09:24

Comment addressed

@Elchi3 Elchi3 merged commit 5d5e180 into mdn:main Aug 15, 2025
6 checks passed
@queengooborg queengooborg deleted the api/Permissions/permission_screen-wake-lock/firefox-corrections branch August 15, 2025 10:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

data:api Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API size:s [PR only] 7-24 LoC changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants