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

Add "create tag" in select input #222

Closed
wants to merge 2 commits into from
Closed

Add "create tag" in select input #222

wants to merge 2 commits into from

Conversation

SimonLab
Copy link
Member

Please feel free to discard/close this PR, see #221 (comment)

It is in example how a tag can be created from the select input.
As mentioned in the issue, this is not a prefect UX, this PR can be used for testing and discussion.

Create a tag from the select input
Feel free to discard this commit/PR
see: #221
@@ -85,6 +85,11 @@
</div>
</li>
<% end %>
<%= if Enum.empty?(@tags) and String.length(String.trim(@search_tag)) != 0 do %>
Copy link
Member Author

Choose a reason for hiding this comment

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

Only show the "create" action when the search for a tag returns 0 results

run mix format
@nelsonic
Copy link
Member

@SimonLab feels like we need "review apps" to discuss ... dwyl/learn-devops#85 💭

@SimonLab
Copy link
Member Author

Yes I will look at how we can create review apps with Fly.
In the meantime I've created the PR to allow people to test it on localhost

@SimonLab SimonLab added the discuss Share your constructive thoughts on how to make progress with this issue label Nov 30, 2022
@SimonLab
Copy link
Member Author

image

Clicking on the "create my new tag" will create the tag and select it before resetting the search tag input:
image

@SimonLab SimonLab removed their assignment Nov 30, 2022
@SimonLab SimonLab mentioned this pull request Dec 14, 2022
@nelsonic
Copy link
Member

The concept of having to manually "Create new tag" is a UX antipattern that takes the person out of the flow of "Capture" mode and into "admin". We aren't going to do this as outlined in: #221 (comment)

@nelsonic nelsonic closed this Jan 26, 2023
@nelsonic nelsonic deleted the create-tag-#221 branch January 26, 2023 00:12
@SimonLab
Copy link
Member Author

To clarify if we need to search code from this PR, the feature added here doesn't redirect to another page. The create link call some liveview code which creates the tag and update the UI with the tag pill

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discuss Share your constructive thoughts on how to make progress with this issue
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants