Skip to content

added scrollTo#831

Merged
ArrayZoneYour merged 3 commits into
leecade:masterfrom
alattanzio:master
Jun 26, 2019
Merged

added scrollTo#831
ArrayZoneYour merged 3 commits into
leecade:masterfrom
alattanzio:master

Conversation

@alattanzio

Copy link
Copy Markdown
Contributor

Is it a bugfix ?

  • No ?

Is it a new feature ?

  • Yes
  • scrollTo specific slider

Describe what you've done:

scroll to specific slider

How to test it ?

call scrollTo using a button, and swiper reference, using index as parameter (eg, using a toolbar on top)

@ue

ue commented Jul 26, 2018

Copy link
Copy Markdown

Merge It !

@danikane

Copy link
Copy Markdown

Please merge. 😃

@acollazomayer

Copy link
Copy Markdown

Merge it!!

@danikane

Copy link
Copy Markdown

this.refs.swiper.scrollBy(index - this.state.index) works around this for me btw.
A scrollTo index would still be better though. :)

@thiswayq

thiswayq commented Dec 7, 2018

Copy link
Copy Markdown

👍 Can we have this? the above solution works but scrollTo would be very nice.

@ManigandanRaamanathan

Copy link
Copy Markdown

@leecade please merge this. ScrollTo is very important compared to ScrollBy

@alattanzio

alattanzio commented Mar 11, 2019

Copy link
Copy Markdown
Contributor Author

I don't have write access. sorry. Ask @leecade 💯

@tobiascornille

Copy link
Copy Markdown
Contributor

Shouldn't this also be added to the typescript definition like #931?

@edoantonioco

Copy link
Copy Markdown

how is this meant to be used? I dont see any scrollTo property in latest nightly version

@sintylapse

Copy link
Copy Markdown

Doesn't seems to be working

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.