4 ms·
It has been a fun journey implementing hooks in React. Let me know your thoughts on the code :)
by r0rshrk 7y ago
It has been a fun journey implementing hooks in React. Let me know your thoughts on the code :)
- modarts 7y ago“Journey” is a bit of a stretch here, no? :) This is a few lines of pretty trivial code
- TomMarius 7y agoMaybe he meant more hooks, in general. For me that is true, hooks really are a very inspirational pattern
- wolco 7y agoGetting the tooling up and running took awhile.
- buildawesome 7y agoMust've been pretty hard to type `npx create-react-app my-app`
- dang 7y agoPlease don't be a jerk on HN, regardless of how correct you are or feel you are. https://news.ycombinator.com/newsguidelines.html https://news.ycombinator.com/newsguidelines.html
- buildawesome 7y agoSorry Dang and Wolco.
- deleted 7y ago[deleted]
- dang 7y agoCan you please review the Show HN guidelines? I appreciate that you're not exactly flaming the OP here, but still, the idea is to respond more charitably to people's work. If you know more than others, find a constructive way to express that, such as by teaching them—and by extension, the rest of us—something related to what they've done so far. https://news.ycombinator.com/showhn.html https://news.ycombinator.com/showhn.html https://news.ycombinator.com/newsguidelines.html https://news.ycombinator.com/newsguidelines.html
- modarts 7y agoThis is a great point, and on a second reading my comment did come off as a little insensitive -- will leave it up to give others context and make this a teachable moment for all of us to improve the level of discourse :)
- ljoshua 7y agoHey congrats! People will complain about the package being to small or trivial, but just keep on creating. We all start somewhere, and hooks are a fun thing to experiment with. It's true that most folks probably would do their own implementation of this and the GPL license would be a non-starter for most projects, but if it suits your needs and you've learned something in the process, more power to you. :)
- funemployed 7y agobrowser compatibility/other edge case handling would be one decent reason to utilize a package for otherwise straightforward implementations. The MDN docs have a helpful walkthrough for feature detecting localStorage that would make a useful addition to your library. https://developer.mozilla.org/en-US/docs/Web/API/Web_Storage_API/Using_the_Web_Storage_API https://developer.mozilla.org/en-US/docs/Web/API/Web_Storage...
- r0rshrk 7y agoOoh, thanks. Must have a look