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

Sticky: Sticky content that is larger than the viewport does not automatically scroll independently from the context #2372

Open
onoufriosm opened this issue Dec 11, 2017 · 8 comments

Comments

@onoufriosm
Copy link

Steps

• Go to: https://react.semantic-ui.com/modules/sticky#sticky-example-oversized
• Scroll

Expected Result

Left column scrolls independently from the context

Actual Result

Left column scrolls only when the end of the context (centered column) is reached

Version

0.77.1

Testcase

https://react.semantic-ui.com/modules/sticky#sticky-example-oversized


Works as expected on https://semantic-ui.com/modules/sticky.html

@levithomason
Copy link
Member

Thanks for the complete report. It seems Sticky is assuming the content should stick to the top of the viewport and not accounting for oversized content.

Also, the description in our doc example should be updated. It is missing crucial info about this feature:

Sticky content that is larger than the viewport.

Should read:

Sticky content that is larger than the viewport will automatically scroll independently from the context, shifting scroll direction at any time will also immediately adjust the scroll position of the fixed content.

@cyrielo
Copy link

cyrielo commented Mar 6, 2018

did this issue get fixed?

@asifsomy
Copy link

asifsomy commented Apr 3, 2018

Is this bug fixed? Long Stuck Content behaves differently in react version as compared to jquery version. I think jquery one is more correct.

@brianespinosa
Copy link
Member

@cyrielo @asifsomy This report is tagged as help wanted and looking at the history above, nobody has linked any pull requests here. That means it is still an open issue and either of you are more than welcome to submit a pull request to solve it.

@stale
Copy link

stale bot commented Jul 3, 2018

There has been no activity in this thread for 90 days. While we care about every issue and we’d love to see this fixed, the core team’s time is limited so we have to focus our attention on the issues that are most pressing. Therefore, we will likely not be able to get to this one.

However, PRs for this issue will of course be accepted and welcome!

If there is no more activity in the next 90 days, this issue will be closed automatically for housekeeping. To prevent this, simply leave a reply here. Thanks!

@stale stale bot added the stale label Jul 3, 2018
@stale
Copy link

stale bot commented Dec 30, 2018

This issue will be closed due to lack of activity for 12 months. If you’d like this to be reopened, just leave a comment; we do monitor them!

@stale stale bot closed this as completed Dec 30, 2018
@layershifter layershifter reopened this Dec 30, 2018
@stale stale bot removed the stale label Dec 30, 2018
@stale
Copy link

stale bot commented Jun 28, 2019

There has been no activity in this thread for 180 days. While we care about every issue and we’d love to see this fixed, the core team’s time is limited so we have to focus our attention on the issues that are most pressing. Therefore, we will likely not be able to get to this one.

However, PRs for this issue will of course be accepted and welcome!

If there is no more activity in the next 180 days, this issue will be closed automatically for housekeeping. To prevent this, simply leave a reply here. Thanks!

@stale stale bot added the stale label Jun 28, 2019
@jjylha
Copy link

jjylha commented Nov 3, 2020

Please fix. 🙏 Oversized SUI-R Sticky is pretty useless with this bug :/

@layershifter layershifter removed the stale label Nov 4, 2020
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