Hacker News new | ask | show | jobs
by synnik 5296 days ago
Some suggestions:

1) If you are going to suggest such a massive scope of changes, offer to help instead of just dumping a list out there.

2) Check to see if the poster on HN is actually the author of the article. Doesn't look like it in this case, so you may be talking into a vacuum.

3) I love that timeline widget. But it is one of the most un-intuitive and complicated things I have ever seen. I recommend re-coding its functions into a simpler library vs. adding even more complexity to timeline.

4) Dependencies can get evil. Your suggestions add quite a lot of complexity into footprint of plugin. I fear that the end result might look good to a coder, but be a bit of a nightmare to actually use in a production environment.

1 comments

1) If you look at my github account, you can see that I've contributed quite a lot to open source over a long period of time. I've got a long history of 'helping'. Looking at your github account... oh wait...

2) Good point. Can't argue that.

3) Maybe. Maybe not. It would need to be investigated further.

4) I made those suggestions based on code I've got in production. Other than Timeline, I suggested two libraries, both of which would remove a lot of code duplication in the Gantt plugin.