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

implementing some chips functionality #115

Closed

Conversation

mansona
Copy link
Collaborator

@mansona mansona commented Jun 26, 2015

Demo: https://cloudup.com/ckGCOdfqX2S

I've also added custom templates which is not in the gif above ^^

@mansona mansona changed the title implementing basic chips implementing some chips functionality Jun 26, 2015
@mansona
Copy link
Collaborator Author

mansona commented Jul 4, 2015

I've updated this so it works on new master


<h2>Custom Template</h2>
\{{#paper-chips readOnly=readOnly removeItem="removeCustomItem" addItem="addCustomItem" placeholder="Add a tag" content=customFruitNames as |item|}}
<strong>{{item}}</strong>
Copy link
Collaborator

Choose a reason for hiding this comment

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

We need to escape {{item}}.

@miguelcobain
Copy link
Collaborator

Can we include contact chips in this PR?
I'm not opposed to adding them later, but I think they're easy to accomplish from there. Right?

@mansona
Copy link
Collaborator Author

mansona commented Jul 5, 2015

Yea I could probably throw it in there sometime soon, the thing that got me stumped was that I wanted to implement the same functionality as this page: https://material.angularjs.org/latest/#/demo/material.components.chips but we don't have autocomplete implemented yet. I suppose we could implement it without autocomplete 😂

@peec
Copy link
Contributor

peec commented Jul 18, 2015

@mansona When #132 is merged you got paper-autocomplete to use.

@@ -6,7 +6,6 @@
*/

body {
overflow: hidden;
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I don't know why I needed to do this but if I didn't remove this I wasn't able to scroll the dummy app

@mansona
Copy link
Collaborator Author

mansona commented Dec 16, 2015

Ok so I've brought this up to the latest and have even done a quick contact-cards implementation 🎉 If you're not happy with the contact-cards implementation I would suggest taking you up on your offer @miguelcobain to add them later i.e. remove them and worry about them later.

I don't know why this is failing the build, can't figure it out from the output of travis. Can someone help me with this?

@miguelcobain
Copy link
Collaborator

@mansona Sorry for asking you this after so much time.

We're preparing release v1 which will be the big thing in ember-paper.
Please read #249.
We're already automatically importing angular material sass styles.

It would be great if you could bring this to be on par with latest v1 branch and angular material 1.
I have no idea if a lot has changed or not.

@mansona
Copy link
Collaborator Author

mansona commented Jan 13, 2016

I'll see if I can take a look sometime over the next few days 👍

@mansona
Copy link
Collaborator Author

mansona commented Jan 25, 2016

Ok so now I'm going to close this in favour of #267 I have quite a few issues with this new branch but I'll start the conversation there 👍

@mansona mansona closed this Jan 25, 2016
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.

3 participants