Skip to content
This repository has been archived by the owner on Mar 4, 2020. It is now read-only.

fix(dropdown): change header and content types from string to shorthand #1054

Merged
merged 13 commits into from
Apr 2, 2019

Conversation

silviuaavram
Copy link
Collaborator

@silviuaavram silviuaavram commented Mar 12, 2019

Requested by People team.

Also to be in sync with the changes of the prop types in list.

made content and header props from DropdownItem and header from DropdownSelectedItem to be shorthand.

@codecov
Copy link

codecov bot commented Mar 12, 2019

Codecov Report

Merging #1054 into master will increase coverage by 0.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1054      +/-   ##
==========================================
+ Coverage   82.35%   82.37%   +0.01%     
==========================================
  Files         733      733              
  Lines        8712     8720       +8     
  Branches     1233     1168      -65     
==========================================
+ Hits         7175     7183       +8     
  Misses       1522     1522              
  Partials       15       15
Impacted Files Coverage Δ
...t/src/components/Dropdown/DropdownSelectedItem.tsx 74.46% <100%> (+1.13%) ⬆️
...ges/react/src/components/Dropdown/DropdownItem.tsx 96% <100%> (+1.26%) ⬆️

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 e8632cd...f3c8651. Read the comment docs.

@silviuaavram silviuaavram force-pushed the fix-change-prop-types-dropdown branch from af4471b to ff9c099 Compare March 12, 2019 23:24
@silviuaavram silviuaavram force-pushed the fix-change-prop-types-dropdown branch from ff9c099 to bf3188f Compare March 27, 2019 09:29
@silviuaavram silviuaavram merged commit 71996a4 into master Apr 2, 2019
@silviuaavram silviuaavram deleted the fix-change-prop-types-dropdown branch April 2, 2019 10:01
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.

3 participants