Added event types and their descriptions #402
Merged
Conversation
This file contains 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
I was looking for a complete list of form types and couldn't find any anywhere. I went through the source code and extracted these with adding some description to them, either using MDN's description or some other website explanation.
eps1lon
reviewed
Apr 13, 2021
There was a problem hiding this comment.
I would rather link https://developer.mozilla.org/en-US/docs/Web/API/Event#introduction. This cheatsheet is more about TypeScript and now you're trying to explain DOM API.
Is there anything missing from https://developer.mozilla.org/en-US/docs/Web/API/Event#introduction that we may want to upstream instead?
|
Well, I used these descriptions so that developers are not confused with which event handler belongs to which event type and just wrote a general description of what they are. Moreover, by these descriptions I wrote, users easily can see the differences between UIEvent, MouseEvent,... and how they are connected. And attention, https://developer.mozilla.org/en-US/docs/Web/API/Event#introduction not all events from your link are implemented in TS |
That's my point. Now we have to maintain that list and we don't have anything to add regarding the description. I don't think we can maintain a separate MDN just for TypeScript. |
swyxio
reviewed
Apr 14, 2021
swyxio
reviewed
Apr 14, 2021
swyxio
approved these changes
Apr 14, 2021
stepanenko
added a commit
to stepanenko/react-ts-best-practices
that referenced
this pull request
May 3, 2021
* Updated README on 2021-04-02T14:21:23.928Z * Fix missing parenthesis in troubleshooting types code example (typescript-cheatsheets#399) Co-authored-by: swyx <shawnthe1@gmail.com> * format * minor improvements to "adding non-standard attributes" (typescript-cheatsheets#400) * Added event types and their descriptions (typescript-cheatsheets#402) * Added form types and their descriptions I was looking for a complete list of form types and couldn't find any anywhere. I went through the source code and extracted these with adding some description to them, either using MDN's description or some other website explanation. * Apply suggestions from code review Co-authored-by: swyx <shawnthe1@gmail.com> * fix: Remove incorrect usage of "widening" (typescript-cheatsheets#405) Using an intersection type here is actually narrowing the type, but the intent here seems to be to introduce type assertions, not talk about the difference between wider and narrower types. * add type assertion for useState thanks @priscilaandreani closes typescript-cheatsheets#403 * format * Fixed typo (typescript-cheatsheets#406) Fixed an issue where a paragraph described three different options as two. * Add rollpkg as an option for creating libraries (typescript-cheatsheets#407) * useLocalStorage Type detection bug (typescript-cheatsheets#409) in my vs code it detects that the type of soredValue is (T | (value: T | ((val: T) => T)) => void) but it must be just (T) so it throws error with this change it solved. * Updated README on 2021-04-25T20:13:23.562Z Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Denis LE <ledenis@users.noreply.github.com> Co-authored-by: swyx <shawnthe1@gmail.com> Co-authored-by: Sebastian Silbermann <silbermann.sebastian@gmail.com> Co-authored-by: Lars <32460418+advename@users.noreply.github.com> Co-authored-by: Gerrit Birkeland <gerrit@gerritbirkeland.com> Co-authored-by: Christopher Kumm <40023417+ChrisCrossCrash@users.noreply.github.com> Co-authored-by: Rafael Pedicini <rafael@rafgraph.dev> Co-authored-by: Mandy YP <63463742+mmdHasan-yazdanPanah@users.noreply.github.com>
stepanenko
added a commit
to stepanenko/react-ts-best-practices
that referenced
this pull request
May 9, 2021
* Updated README on 2021-04-02T14:21:23.928Z * Fix missing parenthesis in troubleshooting types code example (typescript-cheatsheets#399) Co-authored-by: swyx <shawnthe1@gmail.com> * format * minor improvements to "adding non-standard attributes" (typescript-cheatsheets#400) * Added event types and their descriptions (typescript-cheatsheets#402) * Added form types and their descriptions I was looking for a complete list of form types and couldn't find any anywhere. I went through the source code and extracted these with adding some description to them, either using MDN's description or some other website explanation. * Apply suggestions from code review Co-authored-by: swyx <shawnthe1@gmail.com> * fix: Remove incorrect usage of "widening" (typescript-cheatsheets#405) Using an intersection type here is actually narrowing the type, but the intent here seems to be to introduce type assertions, not talk about the difference between wider and narrower types. * add type assertion for useState thanks @priscilaandreani closes typescript-cheatsheets#403 * format * Fixed typo (typescript-cheatsheets#406) Fixed an issue where a paragraph described three different options as two. * Add rollpkg as an option for creating libraries (typescript-cheatsheets#407) * useLocalStorage Type detection bug (typescript-cheatsheets#409) in my vs code it detects that the type of soredValue is (T | (value: T | ((val: T) => T)) => void) but it must be just (T) so it throws error with this change it solved. * Updated README on 2021-04-25T20:13:23.562Z * Clarify useRef hook usages (typescript-cheatsheets#412) Co-authored-by: swyx <shawnthe1@gmail.com> * Update index.md * Update from-flow.md * docs: throw when ref.current is null (typescript-cheatsheets#413) * Update types.md Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Denis LE <ledenis@users.noreply.github.com> Co-authored-by: swyx <shawnthe1@gmail.com> Co-authored-by: Sebastian Silbermann <silbermann.sebastian@gmail.com> Co-authored-by: Lars <32460418+advename@users.noreply.github.com> Co-authored-by: Gerrit Birkeland <gerrit@gerritbirkeland.com> Co-authored-by: Christopher Kumm <40023417+ChrisCrossCrash@users.noreply.github.com> Co-authored-by: Rafael Pedicini <rafael@rafgraph.dev> Co-authored-by: Mandy YP <63463742+mmdHasan-yazdanPanah@users.noreply.github.com> Co-authored-by: Thien Do <hello@thien.do> Co-authored-by: Christina Grannas <christina.grannas@gmail.com>
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.
I was looking for a complete list of form types and couldn't find any anywhere.
I went through the source code and extracted these with adding some description to them, either using MDN's description or some other website explanation.