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

Move script tag in MV3 inline example#2696

Merged
dotproto merged 1 commit intomainfrom
mv3-move-script-tag
Apr 26, 2022
Merged

Move script tag in MV3 inline example#2696
dotproto merged 1 commit intomainfrom
mv3-move-script-tag

Conversation

@jeffposnick
Copy link
Contributor

Fixes #2684

@netlify
Copy link

netlify bot commented Apr 25, 2022

Deploy Preview for developer-chrome-com ready!

Name Link
🔨 Latest commit be177fd
🔍 Latest deploy log https://app.netlify.com/sites/developer-chrome-com/deploys/6266b6590bf361000871091a
😎 Deploy Preview https://deploy-preview-2696--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.

@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/mv3/getstarted/index.md

  • This file passed all of our automated Markdown audits.

AmySteam added a commit to GoogleChrome/chrome-extensions-samples that referenced this pull request Apr 25, 2022
This PR moves the script tag inside of the body tag. Related GoogleChrome/developer.chrome.com#2696
Copy link
Collaborator

@AmySteam AmySteam left a comment

Choose a reason for hiding this comment

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

@jeffposnick Looks good to me 👍.

@dotproto I also made a PR to reflect this change on the tutorial code sample repo here

@Les-Seigneurs-des-Guerriers-de-Lile

This comment was marked as off-topic.

@AmySteam AmySteam added the extensions Issues related to extensions documentation. label Apr 25, 2022
Copy link
Contributor

@dotproto dotproto left a comment

Choose a reason for hiding this comment

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

Thanks for fixing this, @jeffposnick!

@dotproto dotproto merged commit ef36764 into main Apr 26, 2022
@dotproto dotproto deleted the mv3-move-script-tag branch April 26, 2022 18:22
dotproto pushed a commit to GoogleChrome/chrome-extensions-samples that referenced this pull request Apr 26, 2022
This PR moves the script tag inside of the body tag. Related GoogleChrome/developer.chrome.com#2696
Hattrick212 pushed a commit to Hattrick212/Chrome-extension that referenced this pull request Aug 4, 2022
This PR moves the script tag inside of the body tag. Related GoogleChrome/developer.chrome.com#2696
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

extensions Issues related to extensions documentation.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Stray script tag for options.html code snippet

5 participants