developer.chrome.com icon indicating copy to clipboard operation
developer.chrome.com copied to clipboard

Theme Documentation: Correct manifest documentation link and source links

Open stevennyman opened this issue 2 years ago • 3 comments

Linking manifest.json to the tabs API page doesn't seem to make sense, so this PR changes the link to point to the Manifest page. However, the Manifest page doesn't mention the theme key so maybe somewhere else would be a better place to link?

Additionally, this PR adds a manifest_version to the sample theme manifest.

This PR also updates the cs.chromium.org links to source.chromium.org links.

This PR updates both the manifest v2 and manifest v3 theme pages.

stevennyman avatar Jul 14 '22 20:07 stevennyman

Deploy Preview for developer-chrome-com ready!

Name Link
Latest commit 738f1680767b0219249dfb793e69d34de715d817
Latest deploy log https://app.netlify.com/sites/developer-chrome-com/deploys/62d079aaa8806d00089eb014
Deploy Preview https://deploy-preview-3231--developer-chrome-com.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

netlify[bot] avatar Jul 14 '22 20:07 netlify[bot]

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/mv2/themes/index.md

  • This file passed all of our automated Markdown audits.

site/en/docs/extensions/mv3/themes/index.md

  • This file passed all of our automated Markdown audits.

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/mv2/themes/index.md

  • This file passed all of our automated Markdown audits.

site/en/docs/extensions/mv3/themes/index.md

  • This file passed all of our automated Markdown audits.