Skip to content

Bug-2012727 isEnabled updates for action and browseAction#42997

Open
rebloor wants to merge 1 commit intomdn:mainfrom
rebloor:Bug-2012727-isEnabled-updates-for-action-and-browseAction
Open

Bug-2012727 isEnabled updates for action and browseAction#42997
rebloor wants to merge 1 commit intomdn:mainfrom
rebloor:Bug-2012727-isEnabled-updates-for-action-and-browseAction

Conversation

@rebloor
Copy link
Contributor

@rebloor rebloor commented Feb 2, 2026

Description

Address is the dev-doc-needed requirements of Bug 2012727 "action.isEnabled() throws instead of returning the global enabled state" and Bug 2013477 Support action.isEnabled(tabId) next to action.isEnabled(details)by:

  • Marking details as optional and addiong 'tabId' in:
  • Add a note to the action.isEnabled documentation that in Chrome, the parameter is not a details object but an (optional) tabId integer.

Related issues and pull requests

Related BCD changes in mdn/browser-compat-data#28964.

@rebloor rebloor self-assigned this Feb 2, 2026
@rebloor rebloor requested a review from a team as a code owner February 2, 2026 18:02
@rebloor rebloor added the Content:WebExt WebExtensions docs label Feb 2, 2026
@rebloor rebloor requested review from Rob--W and removed request for a team February 2, 2026 18:02
@github-actions github-actions bot added the size/s [PR only] 6-50 LoC changed label Feb 2, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Feb 2, 2026

Copy link
Member

@Rob--W Rob--W left a comment

Choose a reason for hiding this comment

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

Could you rephrase the documentation to also mention the isEnabled(tabId) signature? It was linked to the original bug and fixed a few days later: https://bugzilla.mozilla.org/show_bug.cgi?id=2013477

@rebloor rebloor requested a review from Rob--W February 18, 2026 18:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Content:WebExt WebExtensions docs size/s [PR only] 6-50 LoC changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments