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 chips demo #23
Add chips demo #23
Conversation
- Make tags selectable - Use a different resource for selected tags - When the 'Enter' key is pressed add the text as a tag and don't hide the keyboard
|
||
@Override | ||
public void afterTextChanged(Editable s) { | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
comment empty methods
Ready for a re-review 👀 @blundell @florianmski 🎉 |
|
||
repositories { | ||
maven { | ||
url 'http://dl.bintray.com/novoda/maven' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
temporary - waiting for the lib to be accepted in jcenter
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
might be worth a code comment in case we forget 😄
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we've been accepted 🎉 will change
@Override | ||
protected void onTextChanged(Editable text) { | ||
String rawText = text.toString(); | ||
if (!textContainsHint(text) && (rawText.isEmpty() || containsSpansOnly(rawText))) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can be simplified at all? in name
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 done in the last commit (or at least tried!)
legit 👍 |
} | ||
|
||
/** | ||
* According to the documentation it's modifying the text from this method is an error so we don't need to worry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Something seems wrong in the wording, same for above
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 will fix
Small nits otherwise LGTM 👍 |
Comments addressed! 🎉 |
Adds a demo showing how to implement a 'chips' view using 2 libraries: