Skip to content

2.x Update documentation for menus#2197

Merged
jarednova merged 12 commits into2.x-docs-apifrom
2.x-docs-api-menus
May 4, 2020
Merged

2.x Update documentation for menus#2197
jarednova merged 12 commits into2.x-docs-apifrom
2.x-docs-api-menus

Conversation

@gchtr
Copy link
Copy Markdown
Member

@gchtr gchtr commented Feb 16, 2020

Ticket: #2073, #2089

Menus aren’t documented yet in #2073. This pull request adds documentation for how menus are handled in 2.0.

Copy link
Copy Markdown
Member

@jarednova jarednova left a comment

Choose a reason for hiding this comment

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

Looks good @gchtr. The one change I made was to make all of your examples with MenuPrimary and MenuItemPrimary. Before one of the examples before was using MenuFooter and MenuItemFooter, which took me a few minutes to decode. This way the examples are consistent in diff't pages throughout the documentation

@codecov
Copy link
Copy Markdown

codecov bot commented Feb 17, 2020

Codecov Report

Merging #2197 into 2.x-docs-api will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@               Coverage Diff               @@
##             2.x-docs-api    #2197   +/-   ##
===============================================
  Coverage           96.04%   96.04%           
  Complexity           1530     1530           
===============================================
  Files                  50       50           
  Lines                3892     3892           
===============================================
  Hits                 3738     3738           
  Misses                154      154

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 7c4f510...b3582e6. Read the comment docs.

@codecov-io
Copy link
Copy Markdown

codecov-io commented May 4, 2020

Codecov Report

Merging #2197 into 2.x-docs-api will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@               Coverage Diff               @@
##             2.x-docs-api    #2197   +/-   ##
===============================================
  Coverage           94.18%   94.18%           
  Complexity           1615     1615           
===============================================
  Files                  53       53           
  Lines                4079     4079           
===============================================
  Hits                 3842     3842           
  Misses                237      237           

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 369f45b...ca80e5a. Read the comment docs.

@jarednova jarednova merged commit 1030f59 into 2.x-docs-api May 4, 2020
@jarednova jarednova deleted the 2.x-docs-api-menus branch May 4, 2020 02:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants