Titus

Results 869 comments of Titus

Sure, but to reiterate my previous comment, I’d rather not wait on Rollup, esbuild, and other bundlers to start supporting import maps though, I think that can take at least...

I’m not proposing a solution, I’m explaining the severity of the problem: workers in Next.js are always broken, workers in webpack 5 are mostly broken, as I understand it. Me...

Because the version that uses DOM APIs is 0kb compared to the non-dom version, which is 10kb minzipped https://github.com/wooorm/parse-entities/blob/main/lib/decode-entity.browser.js https://github.com/wooorm/parse-entities/blob/main/lib/decode-entity.js. (this code is used in micromark, remark, react-markdown, MDX, to...

what do you propose for env switching? If that’s done, is the `browser` field still needed? Something with `process.env` you mean right? > adding the `webworker` property is bad idea,...

Hi Savanah! Thanks for reporting this. PRs to solve this bug are welcome—for examples, see https://github.com/words/syllable/commit/bbcad6f860cfaac91d52387b05b540ff275cfc9d or https://github.com/words/syllable/commit/a30ec92b6070d68cce76ee0c5f15f4b514a53978!

Nice, hector! Could you also check some other words that contain `arge`? (`sed -n '/arge/p' < /usr/share/dict/words` works for me)

Preferably by modifying the source code! [Here’s an example](https://github.com/words/syllable/pull/33#issuecomment-507716483) of how I find words to try and see how my changes to this library work!

Oh, sorry for being unclear, what I meant with “Preferably by modifying the source code” is more like: yup, should go in `index.js` if possible!

Hi Abdullah! Thanks for reporting this. PRs to solve this bug are welcome—for examples, see bbcad6f or a30ec92!