Skip to content

feat(Tab): add horizontally scrollable Tab.Items - #3336

Merged
arpitBhalla merged 7 commits into
react-native-elements:nextfrom
entrymaster:tab_scrolling
Feb 12, 2022
Merged

feat(Tab): add horizontally scrollable Tab.Items#3336
arpitBhalla merged 7 commits into
react-native-elements:nextfrom
entrymaster:tab_scrolling

Conversation

@entrymaster

@entrymaster entrymaster commented Feb 4, 2022

Copy link
Copy Markdown
Contributor

Feat:

  • added scrollable tab
  • Added few Style Prop

Fix:

  • Remove default theme background

…ong with scrollHandler( ) function which syncs the scrollview with active tab.
@arpitBhalla arpitBhalla changed the title Scrolling was added to tabs. Implemetation:ScrollView added to map al… feat(Tab): add horizontally scrollable Tab.Items Feb 4, 2022

@entrymaster entrymaster left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I have removed some unnecessary code.

Comment thread packages/base/src/Tab/Tab.tsx Outdated
Comment thread packages/base/src/Tab/Tab.tsx Outdated
@arpitBhalla

arpitBhalla commented Feb 12, 2022

Copy link
Copy Markdown
Member

@entrymaster I made few more changes, so you can skip review change request, Thank you for your contribution 🎉

This was linked to issues Feb 12, 2022
@codecov

codecov Bot commented Feb 12, 2022

Copy link
Copy Markdown

Codecov Report

Merging #3336 (3239147) into next (ca88220) will decrease coverage by 0.90%.
The diff coverage is 50.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             next    #3336      +/-   ##
==========================================
- Coverage   79.97%   79.07%   -0.91%     
==========================================
  Files          88       88              
  Lines        1663     1701      +38     
  Branches      732      744      +12     
==========================================
+ Hits         1330     1345      +15     
- Misses        328      351      +23     
  Partials        5        5              
Impacted Files Coverage Δ
packages/base/src/Tab/Tab.tsx 56.36% <48.93%> (-38.38%) ⬇️
packages/base/src/Tab/Tab.Item.tsx 86.66% <66.66%> (+2.05%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f2f9422...3239147. Read the comment docs.

@entrymaster entrymaster left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

OKay

@arpitBhalla
arpitBhalla merged commit a0a919b into react-native-elements:next Feb 12, 2022
github-actions Bot pushed a commit that referenced this pull request Feb 12, 2022
Co-authored-by: Arpit Bhalla <55053424+arpitBhalla@users.noreply.github.com>
github-actions Bot pushed a commit that referenced this pull request Feb 12, 2022
Co-authored-by: Arpit Bhalla <55053424+arpitBhalla@users.noreply.github.com>
github-actions Bot pushed a commit that referenced this pull request Feb 12, 2022
Co-authored-by: Arpit Bhalla <55053424+arpitBhalla@users.noreply.github.com>
@arpitBhalla arpitBhalla added this to the v4.0.0 milestone Feb 15, 2022
arpitBhalla added a commit that referenced this pull request Mar 17, 2022
Co-authored-by: Arpit Bhalla <55053424+arpitBhalla@users.noreply.github.com>
arpitBhalla added a commit that referenced this pull request Mar 19, 2022
Co-authored-by: Arpit Bhalla <55053424+arpitBhalla@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

TabItem default purple background Horizontally scrolling tabs

2 participants