Skip to content

FEATURE - Toolbar scrollable #84

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
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

GIGAMOLE
Copy link

@GIGAMOLE GIGAMOLE commented May 8, 2023

By default, CollapsingToolbar is not scrollable. In order to enable it, set toolbarScrollable = true in CollapsingToolbarScaffold.

Preview:

toolbar-scrollable

@GIGAMOLE
Copy link
Author

GIGAMOLE commented May 8, 2023

By the way, who wants to use this feature, I created a separated remote dependency, while this PR is under review:

Add it in your root build.gradle at the end of repositories:

all projects {
	repositories {
		...
		maven { url 'https://jitpack.io' }
	}
}

Step 2. Add the dependency

dependencies {
        implementation 'com.github.GIGAMOLE:ComposeCollapsingToolbar:latest-version'
}

Or you can simply download it from there:

https://github.com/GIGAMOLE/ComposeCollapsingToolbar/releases

@GIGAMOLE GIGAMOLE force-pushed the feature/toolbar-scrollable branch from 36fe670 to eca590f Compare May 8, 2023 18:47
@GIGAMOLE
Copy link
Author

GIGAMOLE commented May 8, 2023

Fixed the issue with forced verticalScroll() container clipping:

ezgif com-video-to-gif

@GIGAMOLE GIGAMOLE force-pushed the feature/toolbar-scrollable branch from eca590f to 2fb821a Compare May 8, 2023 20:39
@onebone onebone added feature New feature or request api Improvement of existing library contracts that may break backwards compatibility labels May 9, 2023
…ollable

# Conflicts:
#	README.md
#	lib/src/main/java/me/onebone/toolbar/CollapsingToolbarScaffold.kt
#	lib/src/main/java/me/onebone/toolbar/ToolbarWithFabScaffold.kt
@GIGAMOLE
Copy link
Author

TODO:

Currently, there is no toolbar scroll limitation for each scroll type. After the merge of the Snapping Feature: #83, it will be possible to track whether the offset or toolbar is scrolling to handle properly the toolbar scroll for each scroll strategy.

@Veeksi
Copy link

Veeksi commented Mar 1, 2024

I also need this change. Hopefully this gets merged asap.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api Improvement of existing library contracts that may break backwards compatibility feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants