Skip to content
This repository was archived by the owner on Oct 2, 2019. It is now read-only.

BugFix: removing the label when new tag is clicked on #1994

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

shmuel-web
Copy link

TL;DR

When creating a new tag and clicking to add it to multi tag select, the tag remains named "Tag(new)" with the label even after it's created.
How to reproduce:
go to this plunker
Create a new tag by typing something then you will see the label added in the select options if you select it via keyboard enter the label vanishes, & the tag is added without the label
but ...
if you select via mouse click the label stays...

description

Desired outcome: When you type in "Tag" the selection screen will show "Tag(new)". When you select "Tag(new)", the actual tag will be called "Tag".
Actual outcome: The created tag is still called "Tag(new)".

images

the bug!
image
the desired outcome!
image
the label added to the select option.
image

thanks!

@shmuel-web
Copy link
Author

thanks!

… when clicking on it

now the all the test are passing sorry ;)
@Jefiozie
Copy link
Contributor

Jefiozie commented Jun 5, 2017

Hi , thanks for submitting this PR. Could you clear the dist folder? Then I can have a good view on your changes. Thanks.

asiacomeau pushed a commit to Mixpo/ui-select that referenced this pull request Apr 4, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants