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

Add a multiple option to {{md-select}} #621

Open
Gorzas opened this issue Jul 6, 2017 · 1 comment
Open

Add a multiple option to {{md-select}} #621

Gorzas opened this issue Jul 6, 2017 · 1 comment

Comments

@Gorzas
Copy link
Contributor

Gorzas commented Jul 6, 2017

I've seen that MaterializeCSS has a selector with multiple property but we don't allow it in {{md-select}}:

screenshot from 2017-07-06 11-53-02

I think we can added. We have two options:

  • adding the property to the component {{md-select}}.
  • creating a new component called {{md-select-multiple}}.

What do you think? I'm going to send a PR with the changes as soon as I can.

@mike-north
Copy link
Owner

I would love this to be one {{md-select}} component, built with the intent of refactoring away the materialize javascript away and replacing it with some more SPA-friendly implementation

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

No branches or pull requests

2 participants