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

fixed the carousel button scrolling up issue #94

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

fixed the carousel button scrolling up issue #94

wants to merge 3 commits into from

Conversation

shr218
Copy link

@shr218 shr218 commented Oct 15, 2021

Title

Fixed the carousel button scrolling up issue.

Description

I added var $target = $(target) and removed function to provide smooth scrolling.

Linked Issue

Closes #89

@chick-bot
Copy link

chick-bot bot commented Oct 15, 2021

Hi, @shr218! Thank you for your Pull Request 🥳🚀 A Maintainer will review your PR Shortly. Till then, hold tight!

@shr218
Copy link
Author

shr218 commented Oct 15, 2021

Unable to see the comment.Pls help

js/main.js Outdated Show resolved Hide resolved
@shr218
Copy link
Author

shr218 commented Oct 16, 2021

Can anyone review my PR .Thx!

@arnxv0
Copy link
Collaborator

arnxv0 commented Oct 16, 2021

!bounty 75

1 similar comment
@techverve
Copy link
Member

!bounty 75

@chick-bot
Copy link

chick-bot bot commented Oct 16, 2021

Congrats @shr218 🥳🥳🚀🚀, you got 75 Bounty Points! Check out the Leaderboard to see your new score 😁

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

Successfully merging this pull request may close these issues.

Page scrolls to the top when clicking carousel buttons
3 participants