-
Notifications
You must be signed in to change notification settings - Fork 61
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
Label name uniqueness tweeks #535
Conversation
✅ Deploy Preview for volview-dev ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site configuration. |
a019c4f
to
59c69df
Compare
I think it would be a good idea to enforce name uniqueness and not let the user create labels with duplicate names. Have we encountered a use-case for different labels with the same name? |
No known use case for duplicate names. I'll disable "Done" if name is used. |
59c69df
to
d6ac93c
Compare
d6ac93c
to
3e49354
Compare
Disabled Done button for segment and label editors if name already exists. |
Looks like there are issues with code formatting. |
* remove clear default labels flag from mergeLabels and make explicit function. * set max width to LabelControls label editor
Still allows duplicate names if user presses "Done". New labels are created with unique names suffixed with a number. New labels loop through the confgi.ts:TOOL_COLORS.
Warns if user is editing label name and it maches existing label. "Done" is disabled if name already exists.
Also
Closes #477