Enable Style Customization

[SharpSeeEr]

Issue Number

#415

Overview of PR

I just recently discovered this library, and it is the perfect fit for my project. I did find it strange that the only way to custom style the component was using an itemRenderer, groupRenderer, and passing specific styles. Since it is a component library meant to be used in any project styling should be the simplest thing to override and customize, IMO.

So I have moved all styles that can be safely overridden from jsx to scss. Overriding the default styling is now as simple as overriding the css classes.

I also removed the use of composeEvents() in items.js, as it caused creation of new functions to pass as props to itemRenderer on every render. I know, I know, multiple changes in one PR! Horrible. But it seemed to give a decent performance increase. I have absolutely zero hard facts to back that claim up.

I'm using the timeline with several thousand groups with anywhere from 3 to 20 items per group, so performance is very important to me, and performance is a little slow right now.

[Ilaiwi]

@SharpSeeEr I am glad you found what you need from the library. Thank you for this. I will look into the PR and review it but the compose event is there for a reason... It is needed for you to be able to pass event handlers to the item that we already use like mouseup, mousedown... you can provide memoize function for the compose and it will solve your problem.
For the styles I haven't taken a good look yet but thank you for finding the issue related...

[SharpSeeEr]

I did see the purpose of composeEvent(), and I updated the sections of code that called it to still check for and call the passed in event handlers for mouseup, mousedown, etc.

I am a veteran developer, but a junior react developer - feedback is always welcome.

I also read the contributing guidelines today (one day too late) and see that pull requests should go to master. I would be more than happy to close this PR and reopen a new one to master if needed.

[Ilaiwi]

@SharpSeeEr develop branch is okay no worries.
also no need to run the linting rules checker.
So event handlers can come in 2 ways:

• through item.props
• propsGetter

You handled the first case but not the second. I would suggest just memorizing the function composeEvent for each handler. I am fine with adding memoize-one as a dependency...
This is still not an in-depth code review I will get to that soon I promise 😇

[SharpSeeEr]

Thanks! I'll look into the memoization and update the pull request... soonish. I appreciate this early feedback just as much!

[berdyshev]

any updates on that? I think it's nice to have