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

feat: add YaruScrollViewUndershoot #789

Merged
merged 5 commits into from
Oct 23, 2023

Conversation

Jupi007
Copy link
Member

@Jupi007 Jupi007 commented Oct 20, 2023

Add a new YaruScrollViewUndershoot widget which displays an undershoot at the start and/or end of a scroll view, when the scroll offset is not at the very beginning or end. It works in both scroll axis, and you can also disable the start or end undershoot (ex: if the scroll view is close to the window border it is useless to add an undershoot).
It is also providing a builder named constructor which handle the ScrollController creation and lifecycle, so we don't need to create a statefull widget to use this.

Capture.video.du.2023-10-20.18-11-11.webm
Dark Light
Capture d’écran du 2023-10-20 18-12-06 Capture d’écran du 2023-10-20 18-11-57

I tried to mimic Libadwaita look:

Dark Light
Capture d’écran du 2023-10-20 18-12-35 Capture d’écran du 2023-10-20 18-12-28

@Jupi007 Jupi007 enabled auto-merge (squash) October 20, 2023 17:35
Copy link
Member

@Feichtmeier Feichtmeier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice! very nice idea, cant see any mistake in the code
if you want let @d-loose double check for code

@Jupi007 Jupi007 disabled auto-merge October 20, 2023 17:36
@Jupi007 Jupi007 requested a review from d-loose October 20, 2023 17:37
@Jupi007
Copy link
Member Author

Jupi007 commented Oct 23, 2023

@d-loose Can you quickly check this PR, please?

Copy link
Member

@d-loose d-loose left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@Jupi007 Jupi007 merged commit 8849681 into ubuntu:main Oct 23, 2023
7 checks passed
@Jupi007 Jupi007 deleted the YaruScrollViewUndershoot branch October 23, 2023 10:44
@Jupi007
Copy link
Member Author

Jupi007 commented Oct 23, 2023

Thanks @d-loose

@github-actions github-actions bot mentioned this pull request Oct 23, 2023
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.

None yet

3 participants