photopea / Typr.js

Typr.js - process fonts in Javascript
MIT License
914 stars 73 forks source link

Total overhaul PR #34

Closed wuoarh closed 4 years ago

wuoarh commented 4 years ago

Hi there!

I just wanted to submit a big bulk of changes you may not agree with at all.

As mentioned briefly in the readme, I

I didn't manage to apply those changes in a convenient/easily reviewable way, as I made them in a different repo and just moved them over to my fresh fork.

Please excuse this PR, but I wanted to contribute back what I found most necessary myself. I hope, those changes and the reason for doing so can be understood and appreciated.

I do this, because I like typr very much. It's a great library and it's my choice in regards of functionality and performance. But I think, it needs to evolve.

I'm also concerned about the large (?) number of forks. I'm afraid of missing out on updates and features. I hope, this PR can be accepted some time, and hope to benefit from other PRs as well.

I get if this changeset is too unmanagable to give it a proper look. But, please, give it a try.

With kind regards, wuoarh

photopea commented 4 years ago

Hi, I am afraid I can not accept this pull request.

I write JS in a certain way, and I find using "var" nicer, than switching between var / const / let. Also, I don't like to use JS modules, as they are bound to specific files, and you can not merge them into one file just by concatenating multiple files. I always write JS, that works in browsers "as is".

But feel free to create your own fork.

wuoarh commented 4 years ago

Hi!

Thank you for having a look at this huge mess! I completely understand that.

Sad to hear that our use cases don't seem to fit together that well.

I had some versions of my response and decided to go for the shooort one... I could discuss this further for quite some time 😄

Maybe some day it's a better fit :) Thanks for your time again.

Best, Jan

laptou commented 3 years ago

Hello @wuoarh, I am interested in this fork, but I can't find it. Can you tell me where to look please?

wuoarh commented 3 years ago

Hey! you can still check that fork out at https://github.com/wuoarh/Typr.js , but:

Still, I'm glad to get feedback and thank you for the interest! I still consider it a step forward for a great lib :)