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

Toolbar Notification Component #148

Closed
kyleledbetter opened this issue Nov 10, 2016 · 5 comments
Closed

Toolbar Notification Component #148

kyleledbetter opened this issue Nov 10, 2016 · 5 comments
Labels
Milestone

Comments

@kyleledbetter
Copy link
Contributor

Feature Request

I'm seeing a trend that many of our apps with need a notification count/icon menu item w/ a dropdown menu w/ list items, etc

Screenshots or link to CodePen/Plunker/JSfiddle

image

@kyleledbetter
Copy link
Contributor Author

image

@emoralesb05
Copy link
Contributor

emoralesb05 commented Nov 11, 2016

md-menu-trigger now accepts a MdMenuPanel interface in alpha.10.
https://github.com/angular/material2/pull/1564/files#diff-863c4e6ab704390217dec7361906bec8R30

So this might be just a matter of creating a menu like component and leverage material2 menu-trigger.
e.g.

<td-notification-menu #menu="tdNotificationMenu">
    <div td-notification-header>...</div>
    <button td-notification-menu-item> Refresh </button>
    <div td-notification-footer>...</div>
</td-notification-menu>

where TdNotificationMenu implements MdMenuPanel (maybe even extend MdMenu)

OR just creating a directive to style the md-menu differently, and apply our theme on it.

e.g.

<md-menu td-notification-menu #menu="mdMenu">
    <div td-notification-header>...</div>
    <button md-menu-item> Refresh </button>
    <button md-menu-item> Settings </button>
    <button md-menu-item> Help </button>
    <button md-menu-item disabled> Sign Out </button>
    <div td-notification-footer>...</div>
</md-menu>

Im a little more inclined to the first one though.

@asaph26
Copy link
Contributor

asaph26 commented Dec 28, 2016

Is this on the roadmap still? Beta.0 or later?

@emoralesb05
Copy link
Contributor

We are doing the initial design, if all goes well, we will include it in alpha.10

@emoralesb05
Copy link
Contributor

Initial implementation of notifications and menu core submodules with #213

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants