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

feat(Menu): added shape prop and changed the meaning of the type prop #68

Merged
merged 10 commits into from
Jul 17, 2018

Conversation

mnajdova
Copy link
Collaborator

@mnajdova mnajdova commented Jul 10, 2018

Introduced shape prop with one of the following values:

  • pills
  • pointing
  • underlined

The type prop is now used in a similar way to the button and divider (for emphasizing).

Here is how all different Menu variations can appear in default and primary style:
image

manajdov added 2 commits July 10, 2018 16:59
@codecov
Copy link

codecov bot commented Jul 10, 2018

Codecov Report

Merging #68 into master will increase coverage by 0.59%.
The diff coverage is 91.66%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #68      +/-   ##
==========================================
+ Coverage   68.37%   68.96%   +0.59%     
==========================================
  Files          66       67       +1     
  Lines        1056     1073      +17     
  Branches      198      204       +6     
==========================================
+ Hits          722      740      +18     
+ Misses        330      329       -1     
  Partials        4        4
Impacted Files Coverage Δ
src/components/Menu/menuRules.ts 100% <100%> (+25%) ⬆️
src/components/Menu/menuVariables.ts 100% <100%> (ø)
src/themes/teams/siteVariables.ts 100% <100%> (ø) ⬆️
src/components/Menu/MenuItem.tsx 100% <100%> (ø) ⬆️
src/components/Menu/Menu.tsx 100% <100%> (ø) ⬆️
src/components/Menu/menuItemRules.ts 86.36% <85.71%> (+7.79%) ⬆️

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 f954452...588aee4. Read the comment docs.

levithomason
levithomason previously approved these changes Jul 11, 2018
Copy link
Member

@levithomason levithomason left a comment

Choose a reason for hiding this comment

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

This seems to be an improvement to me. As we formalize our APIs and review our designs goals we may revisit this, but, I think it is good to merge for now. 👍

manajdov added 4 commits July 11, 2018 14:22
# Conflicts:
#	src/components/Menu/Menu.tsx
#	src/components/Menu/MenuItem.tsx
#	src/components/Menu/menuItemRules.ts
#	src/components/Menu/menuRules.ts
@mnajdova mnajdova changed the title [WIP] Menu/added shape prop and changed the meaning of the type prop feat(Menu): added shape prop and changed the meaning of the type prop Jul 12, 2018
manajdov added 2 commits July 16, 2018 11:11
# Conflicts:
#	docs/src/examples/components/Menu/Types/index.tsx
-added shorthand examples
kuzhelov
kuzhelov previously approved these changes Jul 16, 2018
Copy link
Collaborator

@kuzhelov kuzhelov left a comment

Choose a reason for hiding this comment

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

proposed API looks consistent 👍

@mnajdova mnajdova merged commit 97ca1af into master Jul 17, 2018
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.

None yet

3 participants