bogas04 / SikhJS

πŸ‘³ πŸš€ Gurbani Progressive Web Application
https://bogas04.github.io/SikhJS
MIT License
26 stars 16 forks source link

Rewrite #104

Closed bogas04 closed 7 years ago

bogas04 commented 7 years ago

It's very hard to maintain a project which doesn't have any contributors or users (lol).

Plan for rewriting one module at a time, one build step at a time:

Phase 1

Phase 2

bogas04 commented 7 years ago

I would really love a friend who can motivate me to work on this πŸ™ˆ

bogas04 commented 7 years ago

rewrite does master reorg and dependency changes.

It uses:

And doesn't use:

We've lost the styles and polished look, but we have more flexibility in hand now !

Our bundle size is already down to 400-ish kbs from 900-ish kbs! Of course it isn't fair to compare as we don't yet have calendar in place, but we can now employ code splitting for more perf benefits!

I tried using preact over react, but seeing how React is gonna change a lot with 16 (Fiber), I think it'll be better to stick around with react for a while. For reference, it brought bundle size by around 100kbs. Also, react@next as of 15 March 2017 actually increases the bundle size by 10kb or so. But I guess it's a minor trade-off for general perf improvements.

Also, the production webpack build is made by simply using webpack -p and no special plugins as such. Using those might help us reduce bundle size further down.

perminder-klair commented 7 years ago

@bogas04 good job bro! I am happy to help you in this project, let me know which area you want me to have a look.

bogas04 commented 7 years ago

Thank you so much @perminder-klair !! You can take time to get used to the app, get it building on your system and then I can help you in finding issues that you can take up.

I'm really sorry for late reply, have been very busy lately.

perminder-klair commented 7 years ago

@bogas04 I got it all setup already, all looks good!

bogas04 commented 7 years ago

That's great @perminder-klair . You could take up some of these issues:

These are some easy first-pr issues. Really sorry for late replies, have been very busy lately. I can't thank you enough for sticking around and trying to help this projec.t

bogas04 commented 7 years ago

Added redux support for two actions

bogas04 commented 7 years ago

Closing as most of the work is done. Remaining parts can be covered by individual issues.