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

feat(drawer): Improved navigation drawer #3417

Merged
merged 32 commits into from
Aug 25, 2018
Merged

Conversation

abhiomkar
Copy link
Contributor

@abhiomkar abhiomkar commented Aug 24, 2018

  • Introduced new component MDCDrawer (Removed MDCPersistentDrawer & MDCTemporaryDrawer).
  • Support for Dismissible, Permanent & Modal variants.
  • Support for full-height & clipped drawer.
  • More SASS customizations.
  • Improved accessibility including focus management.
  • More simplified structure.

BREAKING CHANGE: Drawer variants have new DOM structure, mixins, and JS. MDCPersistentDrawer and MDCTemporaryDrawer components are replaced with a single MDCDrawer component which supports both.

lynnmercier and others added 25 commits June 4, 2018 12:51
…ts/material-components-web into feat/navigation-drawer
chore: water feat/navigation-drawer branch w/ master
@codecov-io
Copy link

codecov-io commented Aug 24, 2018

Codecov Report

Merging #3417 into master will decrease coverage by 0.09%.
The diff coverage is 98.63%.

Impacted file tree graph

@@           Coverage Diff            @@
##           master   #3417     +/-   ##
========================================
- Coverage   98.39%   98.3%   -0.1%     
========================================
  Files         123     119      -4     
  Lines        5185    5012    -173     
  Branches      640     612     -28     
========================================
- Hits         5102    4927    -175     
- Misses         83      85      +2
Impacted Files Coverage Δ
packages/mdc-drawer/modal/foundation.js 100% <100%> (ø)
packages/mdc-drawer/dismissible/foundation.js 100% <100%> (ø)
packages/mdc-top-app-bar/index.js 100% <100%> (ø) ⬆️
packages/mdc-drawer/constants.js 100% <100%> (ø)
packages/mdc-top-app-bar/foundation.js 94.11% <100%> (+1.01%) ⬆️
packages/mdc-drawer/util.js 100% <100%> (ø) ⬆️
packages/mdc-drawer/index.js 96.82% <96.82%> (ø)

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 108f9ea...a8c68fe. Read the comment docs.

Copy link
Contributor

@kfranqueiro kfranqueiro left a comment

Choose a reason for hiding this comment

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

I just realized (and discussed w/ Will) that we should change the open and close event names to opened and closed, to be consistent with Menu.

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.

None yet

5 participants