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

Deselecting a key with no value in the tag editor removes the key #10145

Closed
tohaklim opened this issue Mar 3, 2024 · 7 comments
Closed

Deselecting a key with no value in the tag editor removes the key #10145

tohaklim opened this issue Mar 3, 2024 · 7 comments
Labels
usability An issue with ease-of-use or design

Comments

@tohaklim
Copy link
Contributor

tohaklim commented Mar 3, 2024

Description

Hello everyone, at first I thought this was some sort of validation improvement, but didn't see it on the changelogs.

Steps to reproduce:

  1. Open the tag editor pane (the UI one, issue is not present in the plain text editor)
  2. Press the + Add button to add a new line
  3. Input a new key, either using the dropdown autocomplete or manually
  4. Without inputting the tag value, switch to another line in the editor or click anywhere else out of it

Expected behaviour (how it was previously):
The key with no tag value set is retained for the editing session. Discarded during upload if no value is added.

Current behaviour:
Key line with no value is cleared as soon as deselected. In the recording I am using shift/tab keys to switch between fields.

Is this by design?

Screenshots

gif

@danieldegroot2
Copy link
Contributor

Workaround: Use the raw tag editor

image

@tohaklim
Copy link
Contributor Author

tohaklim commented Mar 3, 2024

Yes, but I do like the autofill suggestions

@tyrasd
Copy link
Member

tyrasd commented Mar 14, 2024

This is in principle by design: As empty-valued tags would be deleted silently anyway, removing them immediately from the tag list makes it more clear to the user that these tags will be discarded.

The key with no tag value set is retained for the editing session.

Do you have a use case that would benefit from this?

@tyrasd tyrasd added the question Not Actionable - just a question about something label Mar 14, 2024
@tohaklim
Copy link
Contributor Author

For example I start typing in a tag value pair, but switch to a wiki tab to check a definition, or simply shift focus from the text field. The whole line is removed, and has to be retyped.
Looks like #10151 might actually fix this

@tyrasd tyrasd added usability An issue with ease-of-use or design and removed question Not Actionable - just a question about something labels Mar 15, 2024
@tyrasd
Copy link
Member

tyrasd commented Mar 15, 2024

Ah, I see. Yes, that's a good argument in favor of restoring the previous behavior. 👍

@tyrasd
Copy link
Member

tyrasd commented Mar 15, 2024

should be fixed now with #10151

@maro-21
Copy link

maro-21 commented Apr 24, 2024

Thanks, it helps.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
usability An issue with ease-of-use or design
Projects
None yet
Development

No branches or pull requests

4 participants