Skip to content
This repository has been archived by the owner on Jul 10, 2022. It is now read-only.

scrollTo() method #20

Closed
wants to merge 95 commits into from
Closed

scrollTo() method #20

wants to merge 95 commits into from

Conversation

roszpun
Copy link

@roszpun roszpun commented Aug 4, 2017

I had to remove old repo. Here is new pull request with fixed code.

@DominikSerafin
Copy link
Owner

Thanks! I'll merge it after some tests and documenting that feature on site

@DominikSerafin
Copy link
Owner

Hi again,

After some planning I've decided to introduce this feature in Vuebar 2.1 release. So when I'll finalize Vuebar 2.0 release then I'll merge this PR.

https://github.com/DominikSerafin/vuebar/projects/1

@mlg-ipleic
Copy link

When is next release?

@DominikSerafin
Copy link
Owner

@mlg-ipleic

#32 (comment)

@JohnRSim
Copy link

JohnRSim commented Jan 4, 2018

ERROR in static/js/vendor.23cf009c84b9d066de57.js from UglifyJs
Unexpected token: name (calculate) [./node_modules/vuebar/vuebar.js:348,0][static/js/vendor.23cf009c84b9d066de57.js:12416,16]

I got this error when running the vuejs build on the fixes above by roszpun.
but works fine in dev mode..

@JohnRSim
Copy link

JohnRSim commented Jan 8, 2018

ok I see the issue - just need to change the let to var.
or const as var does not change - then build and it works

@ieshan
Copy link

ieshan commented Jul 5, 2018

Any plan to implement native scrolling feature scrollIntoView()?

@DominikSerafin
Copy link
Owner

#84

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

Successfully merging this pull request may close these issues.

6 participants