ISSUE:#41 ADDED CAROUSEL SLIDER TO SHOW OUR APPS SCREENSHOTS#66
ISSUE:#41 ADDED CAROUSEL SLIDER TO SHOW OUR APPS SCREENSHOTS#66krushn-a wants to merge 2 commits intoCatsInTech:websitefrom
Conversation
praveenscience
left a comment
praveenscience
left a comment
There was a problem hiding this comment.
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.. |
|
In my local system its already indented and aligned but when i tried to push to github, in github its somehow gets unindented. |
|
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. |
2 participants
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