Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(slider clickable) : Added slider as clickable #7

Merged
merged 2 commits into from
Mar 17, 2018

Conversation

ssatz
Copy link
Contributor

@ssatz ssatz commented Feb 7, 2018

@dockleryxk : Thanks for the wonderful package.
I have added slider as clickable. Please review it and let me know if you are okay.
I have changed the <div> to <a> tag for SEO also am not sure about the property initial

@dockleryxk
Copy link
Owner

dockleryxk commented Feb 8, 2018

@ssatz Thank you for this great contribution! At first glance this looks good to me, but I will check thoroughly tonight.

Also, the 'initial' property is not important to these changes.

@dockleryxk
Copy link
Owner

@ssatz update: I had to work late so I did not get to look at this yet. I'll have plenty of time this weekend though.

@ssatz
Copy link
Contributor Author

ssatz commented Feb 10, 2018

@dockleryxk 👍 Also need to add router module to support routerlink.

@dockleryxk
Copy link
Owner

@ssatz I have not forgotten about this! When I tried to merge a different PR it took up too much time to merge this. I have reverted that and will get to this asap.

@ssatz
Copy link
Contributor Author

ssatz commented Feb 22, 2018

@dockleryxk : 👍 pls check #11 which will be easy for you to merge.

@dockleryxk dockleryxk merged commit 1cf5301 into dockleryxk:master Mar 17, 2018
@dockleryxk
Copy link
Owner

@ssatz This is live as well as #11 in v1.15. Be aware that I removed the href from ISlide and only use it from IImage

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants