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.

Sign up for GitHub

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

Jump to bottom

handle hash in url #22

Closed
MoOx opened this issue Oct 14, 2014 · 1 comment
Closed

handle hash in url #22

MoOx opened this issue Oct 14, 2014 · 1 comment
Labels
enhancement

Comments

@MoOx
Copy link
Owner

MoOx commented Oct 14, 2014

ref https://github.com/defunkt/jquery-pjax/blob/863b802a84af8a9c72def4984d4d38163541fe6d/jquery.pjax.js#L302-L318
@mikeebee mikeebee mentioned this issue Apr 22, 2015
Closed
darylteo added a commit to darylteo/pjax that referenced this issue Jan 31, 2016
@darylteo
[MoOx#22] - jump to hash
defa72e
darylteo added a commit to darylteo/pjax that referenced this issue Jan 31, 2016
@darylteo
[MoOx#22] - jump to hash
c54f183
darylteo added a commit to darylteo/pjax that referenced this issue Jan 31, 2016
@darylteo
[MoOx#22] - jump to hash
5601aca
darylteo added a commit to darylteo/pjax that referenced this issue Jan 31, 2016
@darylteo
[MoOx#22] - jump to hash
c137181
@darylteo
Copy link
Collaborator

Oops sorry >.< I'm so used to just pushing shit with issue numbers.

I'm not planning on merging this in. Its a really rudimentary implementation and there are no tests for it yet.

Find the patch here if you want it https://github.com/darylteo/pjax/tree/patch-22

Daryl

@BehindTheMath BehindTheMath mentioned this issue Jan 14, 2018
Open
BehindTheMath added a commit that referenced this issue Jan 22, 2018
@BehindTheMath
Scroll to element position when URL contains a hash
bc2432b 
When the URL contains a hash, try to find the corresponding
element, and if found, scroll to its position.

Based on darylteo/pjax@4893a2a

Fixes #22.
@BehindTheMath BehindTheMath mentioned this issue Jan 22, 2018
Merged
@ivan-nginx ivan-nginx mentioned this issue Oct 7, 2019
Closed
33 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@MoOx @darylteo @BehindTheMath