Skip to content
This repository has been archived by the owner on Nov 27, 2022. It is now read-only.

feature: add Collapsible TabView #1096

Closed
wants to merge 5 commits into from

Conversation

PedroBern
Copy link

@PedroBern PedroBern commented Nov 14, 2020

Edit

I have published a package with this feature. If this PR ever gets merged, I probably will archive it. @satya164 please if you are going to review it, let me know first, so I can update this PR to the latest of the published package, in the meanwhile I'm developing there. Thanks for this awesome lib.

Motivation

Same motivation as #1079 and #411, but solving with a different approach. Just like @jehartzog, I started from this medium article by @JungHsuan and made some great improvements to make it reusable.

My goal was to build a CollapsibleTabView keeping the same API as TabView, in fact, if someone uses the new component proposed on this PR without a header, it will be the same as the TabView.

@satya164 please let me know your thoughts on this. I'm available to discuss and make changes.

Related: #901, #791, #735

@PedroBern PedroBern changed the title Add Collapsible TabView feature: add Collapsible TabView Nov 14, 2020
@pskryuchkov
Copy link

pskryuchkov commented Apr 7, 2021

@PedroBern Hi! Nice work. Currently collapsible tab view scrolls only when tab is touched, but not a header. Could you please tell how to make header and tab bar responsive for scrolling? It's critical for cases when header height is greater than screen height.

@PedroBern PedroBern closed this Apr 7, 2021
@PedroBern
Copy link
Author

@pskryuchkov use one of the following packages instead of this or:

There are instructions on the readme on how to scroll on the header, as you want.

@jehartzog jehartzog mentioned this pull request Apr 7, 2021
4 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants