Conversation
`generateKey` returns a Promise. The variable names (`key` and `keyPair`) seem to indicate that these are actual values, but unawaited they are promises. Adding an await makes that more clear.
* Fix developers.google.com link * Update files/en-us/web/api/console/dir/index.md Co-authored-by: wbamberg <[email protected]> Co-authored-by: wbamberg <[email protected]>
* PWA Add to home icon is now install menu * typo Co-authored-by: wbamberg <[email protected]>
* WebAPI: Clean syntax from property pages, part 11.1 * WebAPI: Clean syntax from property pages, part 11.2
* Fix grammar issue * Update files/en-us/web/api/geolocation/clearwatch/index.md Co-authored-by: Jean-Yves Perrier <[email protected]> Co-authored-by: Jean-Yves Perrier <[email protected]>
Arrow functions does not have their own 'this' context. Replacing 'this' with 'volumeSlider' to access value property or we can replace the arrow function with 'function ()' and use 'this' to access value.
* FF98 Remove HTTP Large-Allocation * Apply suggestions from code review Co-authored-by: Ruth John <[email protected]> Co-authored-by: Ruth John <[email protected]>
Correct `timeZoneName` and mention what default value of `timeZone` is.
Bumps [@mdn/yari](https://github.com/mdn/yari) from 0.9.69 to 0.9.71. - [Release notes](https://github.com/mdn/yari/releases) - [Changelog](https://github.com/mdn/yari/blob/main/docs/npm-releases.md) - [Commits](mdn/yari@v0.9.69...v0.9.71) --- updated-dependencies: - dependency-name: "@mdn/yari" dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Fix private methods link Add `/` to the private methods link to make it work. * Update files/en-us/web/javascript/closures/index.md Co-authored-by: wbamberg <[email protected]> Co-authored-by: wbamberg <[email protected]>
* Update index.md adapting the module title Because the other modules in this course also always have "CSS" in front of them. * Change just the title not the slug * Change text of links accordingly * Change text of link accordingly
* Rewrites Site definition * Make site definition friendlier * typography Co-authored-by: Daniel D. Beck <[email protected]> * typography Co-authored-by: Daniel D. Beck <[email protected]> * typography Co-authored-by: Daniel D. Beck <[email protected]> * typography Co-authored-by: Daniel D. Beck <[email protected]> Co-authored-by: Will Bamberg <[email protected]> Co-authored-by: Daniel D. Beck <[email protected]>
* Fix developers.google.com link * fix
Co-authored-by: Eric Bailey <[email protected]>
…inks (mdn#14617) * Corrected grammar, punctuation and spelling errors * Fixed broken links
* Replace var with let * Update files/en-us/web/api/document/createelementns/index.md Co-authored-by: Jean-Yves Perrier <[email protected]> Co-authored-by: Jean-Yves Perrier <[email protected]>
* Corrected grammar * Corrected grammar, punctuation and vocabulary * Corrected punctuation and grammar
* Update index.md * Update index.md * Update index.md * Update index.md * Update index.md * Update index.md * Update index.md * Update index.md * Update index.md * Update index.md * Update index.md
* Update index.md * Update index.md * Update index.md
Author
|
What the hell happened here? Closing this. |
Author
|
Argh not in the repo. Sorry Dipika! |
dipikabh
pushed a commit
that referenced
this pull request
Nov 22, 2023
* Add `cancel` event https://html.spec.whatwg.org/multipage/indices.html#event-cancel: Fired at dialog elements when they are canceled by the user (e.g., by pressing the Escape key), or at input elements in the File state when the user does not change their selection * 📝 Make linter happy Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> * 📝 Add an example * 📝 highlight * 🐛 typo * 📝 Move cancel_event to HTMLElement * 🔥 Removed original HTMLDialogElement/cancel_event * 📝 Update all links * 🐛 Sort redirects * chore: Merge suggestion #1 Co-authored-by: Estelle Weyl <[email protected]> * chore: Merge suggestion #2 Co-authored-by: Estelle Weyl <[email protected]> * chore: Merge suggestion #3 Co-authored-by: Estelle Weyl <[email protected]> * chore: Merge suggestion #4 Co-authored-by: Estelle Weyl <[email protected]> * chore: Merge suggestion (5) Co-authored-by: Estelle Weyl <[email protected]> * chore: Merge suggestion (6) Co-authored-by: Estelle Weyl <[email protected]> * Update files/en-us/web/html/element/input/file/index.md Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> --------- Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Estelle Weyl <[email protected]> Co-authored-by: Estelle Weyl <[email protected]>
dipikabh
pushed a commit
that referenced
this pull request
Jan 4, 2024
* chore: replace create-react-app with vite * chore: .js --> .jsx * chore: add dev script section * chore: update App component walkthrough * fix: use correct h1 link * feat: rewrite more * chore: apply suggestions from review Co-authored-by: Chris Mills <[email protected]> * chore: fix npm script syntax * chore: rewrite node install instructions * chore: delete extraneous section * chore: delete image from replacement code * chore: add syntax explainer * chore: clean up final section * chore: rewrite jsx section * chore: nitpick notes * chore: delete React 17 note * chore(nit): hard drive -> machine * chore: apply suggestions from code review Co-authored-by: Chris Mills <[email protected]> * remove superfluous quote * chore: replace create-react-app image with vite * chore: index.jsx --> main.jsx * chore: clean up prop-comment section * chore: delete note on multi-word attributes * chore: use code syntax in heading * chore: change npm script note * chore: update note again * chore: fix image alt * chore: apply suggestions from code review Co-authored-by: Chris Mills <[email protected]> * chore: replace rollup mention with generic bundler reference * chore: document assets directory * chore: document lockfile * chore: disambiguate app function and component * chore: add some * Small App() related tweaks --------- Co-authored-by: Chris Mills <[email protected]>
dipikabh
pushed a commit
that referenced
this pull request
Jan 17, 2024
* chore: replace create-react-app with vite * chore: .js --> .jsx * chore: add dev script section * chore: update App component walkthrough * fix: use correct h1 link * feat: rewrite more * chore: apply suggestions from review Co-authored-by: Chris Mills <[email protected]> * chore: fix npm script syntax * chore: rewrite node install instructions * chore: delete extraneous section * chore: delete image from replacement code * chore: add syntax explainer * chore: clean up final section * chore: rewrite jsx section * chore: nitpick notes * chore: delete React 17 note * chore(nit): hard drive -> machine * chore: apply suggestions from code review Co-authored-by: Chris Mills <[email protected]> * remove superfluous quote * chore: replace create-react-app image with vite * chore: index.jsx --> main.jsx * chore: clean up prop-comment section * chore: delete note on multi-word attributes * chore: use code syntax in heading * chore: change npm script note * chore: update note again * chore: fix image alt * chore: apply suggestions from code review Co-authored-by: Chris Mills <[email protected]> * chore: replace rollup mention with generic bundler reference * chore: document assets directory * chore: document lockfile * chore: disambiguate app function and component * chore: add some * Small App() related tweaks --------- Co-authored-by: Chris Mills <[email protected]>
dipikabh
pushed a commit
that referenced
this pull request
Nov 4, 2024
fix: grammar in mixed_content md (#1)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Removing the last few occurrences of
{{draft}}\o/The last steps will be to mark it as deprecated in the meta-docs and update the macro in yari to raise a flaw.