Skip to content
This repository has been archived by the owner on Mar 14, 2024. It is now read-only.

Browser action docs use incorrect property name in sample manifest #871

Merged
merged 2 commits into from
Jun 10, 2021

Conversation

dotproto
Copy link
Contributor

The first code sample in the browser action API docs indicates that the manifest key for a browser action is "action". This PR fixes that reference to say "browser_action".

This PR also contains a minor clarification on page action.

@chrome-devrel-review-bot
Copy link
Collaborator

Hello! This is an automated review by our custom reviewbot. It updates automatically when code or GitHub comments in this pull request are created or updated.

Requested changes

If there are any common problems with the content files you created or modified, they will be listed here.

site/en/docs/extensions/reference/browserAction/index.md

  • Please change your usage of the following words:
    • Do not use master. Affected lines: 145

@netlify
Copy link

netlify bot commented Jun 10, 2021

✔️ Deploy Preview for developer-chrome-com ready!

🔨 Explore the source changes: 456c27f

🔍 Inspect the deploy log: https://app.netlify.com/sites/developer-chrome-com/deploys/60c29df436fe0b000805df40

😎 Browse the preview: https://deploy-preview-871--developer-chrome-com.netlify.app

@pullapprove pullapprove bot requested a review from awfuchs June 10, 2021 21:30
@google-cla google-cla bot added the cla: yes label Jun 10, 2021
@dotproto dotproto changed the title Fix incorrect property name in manifest Browser ation docs use incorrect property name in manifest sample Jun 10, 2021
@dotproto dotproto changed the title Browser ation docs use incorrect property name in manifest sample Browser ation docs use incorrect property name in sample manifest Jun 10, 2021
@dotproto dotproto added the extensions Issues related to extensions documentation. label Jun 10, 2021
Copy link
Contributor

@rdcronin rdcronin left a comment

Choose a reason for hiding this comment

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

Change LGTM.

Commit message says

...
This PR also contains a minor clarification on page action.

But I don't see the change for page action?

Copy link
Contributor

@rdcronin rdcronin left a comment

Choose a reason for hiding this comment

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

slgtm.

Also note: typo in the commit title ("Browser ation" -> "Browser action")

@dotproto dotproto changed the title Browser ation docs use incorrect property name in sample manifest Browser action docs use incorrect property name in sample manifest Jun 10, 2021
@dotproto
Copy link
Contributor Author

Words are hard. Thanks, @rdcronin.

@dotproto dotproto merged commit 6a7525a into main Jun 10, 2021
@dotproto dotproto deleted the browseraction-fix branch June 10, 2021 23:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla: yes extensions Issues related to extensions documentation.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants