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

deprecate ListItem subcomponents and avatar.accessory #2494

Merged

Conversation

flyingcircle
Copy link
Collaborator

Fixes: #2459
Remove ListItem sub-components, deprecate props.
Also deprecates Avatar accessory props.

TODO:

  1. double check that example continues to work
  2. write blog post detailing upgrade details

@codecov
Copy link

codecov bot commented Aug 10, 2020

Codecov Report

Merging #2494 into next will decrease coverage by 2.26%.
The diff coverage is 74.39%.

Impacted file tree graph

@@            Coverage Diff             @@
##             next    #2494      +/-   ##
==========================================
- Coverage   95.37%   93.10%   -2.27%     
==========================================
  Files          33       40       +7     
  Lines         692      769      +77     
  Branches       87      115      +28     
==========================================
+ Hits          660      716      +56     
- Misses         28       49      +21     
  Partials        4        4              
Impacted Files Coverage Δ
src/list/ListItemChevron.js 0.00% <0.00%> (ø)
src/list/ListItemButtonGroup.js 50.00% <50.00%> (ø)
src/list/ListItemCheckBox.js 50.00% <50.00%> (ø)
src/list/ListItemInput.js 50.00% <50.00%> (ø)
src/list/ListItemSubtitle.js 66.66% <66.66%> (ø)
src/list/ListItem.js 80.00% <73.68%> (-16.30%) ⬇️
src/avatar/Avatar.js 92.30% <88.88%> (-2.14%) ⬇️
src/list/ListItemContent.js 100.00% <100.00%> (ø)
src/list/ListItemTitle.js 100.00% <100.00%> (ø)
... and 6 more

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 1fbd9ac...8d484bc. Read the comment docs.

@flyingcircle flyingcircle merged commit 7423932 into react-native-elements:next Aug 10, 2020
@flyingcircle flyingcircle deleted the listitem-refactor branch August 19, 2020 03:35
Comment on lines +396 to +402
ThemedListItem.Chevron = ListItemChevron;
ThemedListItem.Content = ListItemContent;
ThemedListItem.Input = ListItemInput;
ThemedListItem.Title = ListItemTitle;
ThemedListItem.Subtitle = ListItemSubtitle;
ThemedListItem.CheckBox = ListItemCheckBox;
ThemedListItem.ButtonGroup = ListItemButtonGroup;
Copy link
Contributor

Choose a reason for hiding this comment

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

These are not added to the TS types

Copy link
Contributor

Choose a reason for hiding this comment

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

Ah: #2523

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.

[Suggestion] ListItem -> Add a new text field
2 participants