8000 Wrap mdc-menu · Issue #18 · mpetuska/kmdc · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Wrap mdc-menu #18

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

Closed
mpetuska opened this issue Nov 26, 2021 · 2 comments · Fixed by #67
Closed

Wrap mdc-menu #18

mpetuska opened this issue Nov 26, 2021 · 2 comments · Fixed by #67
Assignees
Milestone

Comments

@mpetuska
Copy link
Owner

Wrap mdc-menu js module in new kmdc-menu module.

@mpetuska mpetuska added this to the 1.0.0 milestone Nov 26, 2021
@pocharlebois
Copy link
Contributor

@mpetuska I'll be looking into this one.

@mpetuska
Copy link
Owner Author

Sweet, thanks!

pocharlebois referenced this issue in innertech/kmdc Jan 5, 2022
Provides support for MDCMenu and MDCMenuSurface
Demonstrate use as a dropdown in Sandbox

Fixes #18 and #19
mpetuska pushed a commit that referenced this issue Jan 9, 2022
* kmdc-menu and kmdc-menu-surface

Provides support for MDCMenu and MDCMenuSurface
Demonstrate use as a dropdown in Sandbox

Fixes #18 and #19

* Refactor MDCMenu based on newer MDC components

* kmdc-menu - remove text opts for MenuItem

* kmdc-menu - correct use of scope

* kmdc-menu - Separate SideEffect for each opts

* kmdc-menu - Remove unnecessary MenuSurfaceAnchorAttrsScope

* kmdc-menu - Fix setAnchorCorner

* kmdc-menu - setAnchorCorder as Byte

* kdmc-menu - Import Corner byte definition from Material API
pocharlebois referenced this issue in innertech/kmdc Mar 14, 2022
Provides support for MDCMenu and MDCMenuSurface
Demonstrate use as a dropdown in Sandbox

Fixes #18 and #19
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 a pull request may close this issue.

2 participants
0