Skip to content

Conversation

@lotrien
Copy link
Member

@lotrien lotrien commented Jan 14, 2018

To avoid duplications for tags we've forbid tags duplications
on component level, so user wouldn't be able to add two duplicate
tags on new Snippet form.
In addition, fixed onKeyPress issue so now it's not bubbling up
to submit form element and thus do not cause form subbmitting
when user tries to add new tag by clicking Enter button.

@lotrien lotrien requested review from ikalnytskyi and malor January 14, 2018 11:02
To avoid duplications for tags we've forbid tags duplications
on component level, so user wouldn't be able to add two duplicate
tags on new Snippet form.
In addition, fixed onKeyPress issue so now it's not bubbling up
to submit form element and thus do not cause form subbmitting
when user tries to add new tag by clicking Enter button.
Copy link
Member

@malor malor left a comment

Choose a reason for hiding this comment

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

Nice! 👍

@lotrien lotrien merged commit f4b0ccd into master Jan 14, 2018
@lotrien lotrien deleted the keypress-unique branch January 14, 2018 11:14
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