Skip to content

Doesn't work with inner scrolling areas. #13

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

Open
Aaron-Pool opened this issue Jul 19, 2016 · 9 comments
Open

Doesn't work with inner scrolling areas. #13

Aaron-Pool opened this issue Jul 19, 2016 · 9 comments
Assignees

Comments

@Aaron-Pool
Copy link

Unless I'm totally misreading the docs, there doesn't seem to be a way to make elements stick within a scrollable area that isn't the main area. That's probably a pretty important feature to a large group of people (including me).

@harm-less harm-less self-assigned this Jul 24, 2016
@harm-less
Copy link
Owner

harm-less commented Jul 24, 2016

Hey Aaron,
I'm fully aware of this and recently needed the feature myself too. I just haven't had time to do so unfortunately.
I'm currently working on a complete refactoring of the sticky engine which would surely include this feature. But I don't want to make any promises as I'm not quite sure about the scope of the project and when exactly I can work on it.
So I'll keep it in mind.

@harm-less harm-less added this to the Sticky engine refactoring milestone Jul 24, 2016
@sandeepsukhija
Copy link

@harm-less any update on this?

@Disorrder
Copy link

need this feature too

@paulflo150
Copy link

@harm-less any thoughts on when this may make it into a release? Thanks!

@developandbehappy
Copy link

developandbehappy commented Dec 16, 2019

Oh, i need it in this moment :(

@harm-less
Copy link
Owner

Hey @zidein07,
I'm not planning on actively developing on this repo. It's ok if you or someone else wants to make a PR to support this feature.

@developandbehappy
Copy link

developandbehappy commented Dec 16, 2019

Hi @harm-less .
We use your directive in a biggest bank in russia ;)
Unfortunately we use angularjs yet :) May be i'll make a PR.
Thank you for your answer!

@harm-less
Copy link
Owner

That's amazing, that's great to hear and I'm glad you trust my code! :D
It's too bad big often means legacy, but it's understandable to some degree.

A PR will be nice and of course you can always fork it and walk your own path.

@phazei
Copy link
Contributor

phazei commented Jan 16, 2020

Spent a day making an attempt at this, has a lot of technical hurdles. If it's inside a scrolling div, position fixed can't be used because it will show even if it's outside the overflow area, so it would need to be switched to using transform translate position absolute instead. Also need to change the scroll event listener since it needs to trigger on any scrolling, not just on window.
This is incomplete:
#46

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

No branches or pull requests

7 participants