type-hangul icon indicating copy to clipboard operation
type-hangul copied to clipboard

Typescript

Open Bas950 opened this issue 4 years ago • 5 comments

Tested in HTML and Jest. Still works. You might want to change the output directory back, this is just what I am used to / prefer.

closes https://github.com/SDuck4/type-hangul/issues/10

Bas950 avatar Aug 20 '21 20:08 Bas950

Something we should look into: image If non-korean letters are included in the text those letters take twice as long to display.

Bas950 avatar Aug 20 '21 20:08 Bas950

Thank you for contributing to this library. I didn't expect to receive other people's PR from foreign countries. This is a very interesting and exciting experience.

Anyway, I am not familiar with the latest TypeScript and Vue, so the review may take a long time. I'd appreciate it if you could wait for the review.

SDuck4 avatar Aug 21 '21 05:08 SDuck4

Thank you for contributing to this library. I didn't expect to receive other people's PR from foreign countries. This is a very interesting and exciting experience.

Hahaha no problem, glad I can help you out!

Anyway, I am not familiar with the latest TypeScript and Vue, so the review may take a long time. I'd appreciate it if you could wait for the review.

I haven't done any vue stuff yet, this PR focuses on the TypeScript part. Take you time with reviewing!

Bas950 avatar Aug 21 '21 08:08 Bas950

If non-korean letters are included in the text those letters take twice as long to display.

Fixed in https://github.com/Bas950/type-hangul/commit/e99dd60d3833f070f9902ba7fc04ddaa3a4bcb50 image There is still an extra timeout after it finished writing a Korean sentence (this is intentional I think).

Bas950 avatar Aug 21 '21 08:08 Bas950

Something I haven't checked yet; But maybe the main in package.json needs to be changed to the index.js file.

Bas950 avatar Aug 21 '21 08:08 Bas950