-
Notifications
You must be signed in to change notification settings - Fork 91
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
ISSUE:#41 ADDED CAROUSEL SLIDER TO SHOW OUR APPS SCREENSHOTS #66
base: website
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I still don't understand what happened here. What was happening and also why so much duplication? Why did you edit the library files?
So what i did is as it was using owl carousel so first i change the nav attribute to true in script.js to turn on the carousel feature. and then i have added css related to carousel in owl.carousel.min.css, owl.theme.default.min.css and responsive.css. And can you please elaborate about library files issue so that i can figure what wrong i have done. |
Every time i tried the whole file get duplicated, i am also not getting how is it happening, eventhough i have never deleted anything, though i have just added one line of code. |
Oh wow, interesting, let me check. |
@krushn-a One last thing, on the |
Yeah, sure.. |
Is there any way to solve this problem. |
I am not sure, it is very interesting here, anyway, don't worry, let me fix it and merge it tomorrow. Is that alright? 🤔 |
Yeah, sure |
hello @praveenscience can I get any updates regarding my PR? |
Hi @praveenscience can I get any updated regarding my PR. |
Related Issue(s)
ISSUE:#41 Carousel Slider was not present in the "Show our Apps ScreenShots"
Proposed Changes
Carousel Slider is applied to all devices
Type of Change
Your Idea for Level (GSSoC)
Screenshots (if applicable)
Additional Notes