-
Notifications
You must be signed in to change notification settings - Fork 22.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Align managed storage snippet with favorite color example #20254
Align managed storage snippet with favorite color example #20254
Conversation
Preview URLsFlawsNone! 🎉 External URLsURL: No new external URLs (this comment was updated 2022-09-07 21:57:30.886659) |
} | ||
} | ||
``` | ||
|
||
Given this manifest, the "favorite-color-examples\@mozilla.org" extension could access the data using code like this: | ||
Given this manifest, the ["favourite-colour-examples\@mozilla.org"](https://github.com/mdn/webextensions-examples/tree/master/favourite-colour) extension accesses the data using code like this: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"could access" is still accurate. The code snippet below does not appear in that extension.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good point, reverted
} | ||
} | ||
``` | ||
|
||
Given this manifest, the "favorite-color-examples\@mozilla.org" extension could access the data using code like this: | ||
Given this manifest, the ["favourite-colour-examples\@mozilla.org"](https://github.com/mdn/webextensions-examples/tree/master/favourite-colour) extension accesses the data using code like this: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is the \
before @
needed? If not, let's remove it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I changed this just to represent the extension name
* Align managed storage snippet with favorite color example * Addressed feedback * Removed space
Summary
The favourite-colour example web extension uses British English e.g. favourite and colour. The instructions for using the example include details on how to create the manage storage file but also point the reader to storage.managed where the example of the manage storage file is presented in American English e.g. favorite and color. This may be causing issues if the reader uses the storage.managed file version to create the file.
Motivation
To avoid potential issues with the example extension not working if the user copeis the file content from storage.managed.
Supporting details
This issue appears to have occurred in Favourite Color bug manifest #498
Metadata
This PR…