andyyou / react-coverflow

A Coverflow effect component of React version made for the web.
http://andyyou.github.io/react-coverflow/
MIT License
347 stars 142 forks source link

Css Customization & Support #57

Open Shailth opened 5 years ago

Shailth commented 5 years ago

Hi Folks,

I would say tremendous effort to bring up such a fine and precise work. For sure a re-usable and easy to implement component. Would request the team to provide help on the following enhancements: 1) The parameter/prop list supplied to CoverFlow to hold custom updates to css like background color/fonts/etc. 2) Have navigation buttons on the sides rather than bottom(other related customization options if possible.)

Thanks a ton, Shalabh

asalem1 commented 5 years ago

Hey @Shailth, those sound like great feature additions. I'd be happy to incorporate them into the project going forward. I've been a little behind with some of the issues but will keep it in mind going forward. Thanks for your input!

Shailth commented 5 years ago

Hi Asalem, i have done some work in this direction, would like to contribute if you allow. Also, i have some ideas further to enrich CoverFlow, if you please to discuss, at your convenience.

screen shot 2019-01-24 at 12 34 57 pm
asalem1 commented 5 years ago

@Shailth that sounds great! I'm definitely open to checking out PRs and working together to implement some of the changes you're looking to make

nojaf commented 5 years ago

Hello, any update on this? I would like to customize the background of the container and the colours of the alt text. Pretty clueless about how to proceed here.

asalem1 commented 5 years ago

Hi @nojaf , I still haven't heard back from @Shailth about implementing this feature. If you'd like to send a PR and contribute to the project you're more than welcome to make additions

nojaf commented 5 years ago

Hi @Asalem1 , I currently have a version where I skip the entire Radium thing. Modifying the scss file directly better fitted my workflow. Would that be an option as well?

asalem1 commented 5 years ago

@nojaf , definitely. I mentioned in #53 that I'm looking to remove the project's dependency on Radium so that would be great!