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

[Test failure] TC_AUTHOR_REDWOOD_42: Deleting third level child tags should remove parent and grandparent tags #378

Open
cassiezamparini opened this issue Jun 5, 2024 · 8 comments
Labels
bug Report of or fix for something that isn't working as intended needs triage redwood release testing Affects the upcoming release (attention needed)
Milestone

Comments

@cassiezamparini
Copy link

cassiezamparini commented Jun 5, 2024

Release

Redwood

Expected behavior

Removing an explicit child that is the only child of its implicit parent should remove the implicit parent too and grandparent too.

*Note: The most granular tag in the hierarchy becomes the explicit tag with all its parent tags becoming implicit

Actual behavior

See this video for the actual behaviour: https://www.loom.com/share/aea0455461b34bc98732c77805f46aa3?sid=1cafa5ad-7be0-4c6c-9658-7a43a71783c5

Steps to reproduce

  1. Add a parent tag
  2. Add a second level and third level child tag
  3. Click save
  4. Reopen the tag drawer and delete the third level tag. The second level tag disappears but the first does not. They should both disappear

Additional information

No response

@cassiezamparini cassiezamparini added bug Report of or fix for something that isn't working as intended release testing Affects the upcoming release (attention needed) needs triage labels Jun 5, 2024
@cassiezamparini
Copy link
Author

cc. @jmakowski1123 FYI

@cassiezamparini
Copy link
Author

@jmakowski1123 @ali-hugo and I were chatting about this, so just to confirm...

  1. The Level 2 tag shouldn't delete if I've explicitly selected it (if I delete a level 3 tag) as mentioned above.
  2. BUT if I don't explicitly select a Level 2 tag, but explicitly select a Level 3 tag and then delete it, the Level 2 tag should automatically delete. It works in this way already.

@ali-hugo
Copy link

ali-hugo commented Jun 5, 2024

@cassiezamparini Although I agree that "chordophone" disappearing feels strange, I don't think it's actually a bug. I found this ticket describing the intended functionality. See the following points:

  1. Display an "x" for explicit tags in the tag drawer. Clicking the "x" will remove the tag from the content item.

5.5. The most granular tag in the hierarchy becomes the explicit tag with all its parent tags becoming implicit

  1. Removing an explicit child that is the only child of its implicit parent will remove the implicit parent too (and grandparent etc.)

Let me know if you agree that the actual behaviour is in line with what is described in that ticket.

Hopefully we'll get the opportunity to improve the tagging user experience in the future.

@cassiezamparini
Copy link
Author

@ali-hugo Yes, it seems to be inline with these two points especially: The most granular tag in the hierarchy becomes the explicit tag with all its parent tags becoming implicit; Removing an explicit child that is the only child of its implicit parent will remove the implicit parent too (and grandparent etc.) Thanks Ali.

Confusing though. I agree we need to do some work on the overall UX here. There were one or two things I was confused by during testing again.

@jmakowski1123 Should I archive this issue? Anything you'd like to add?

@jmakowski1123
Copy link

@cassiezamparini Although I agree that "chordophone" disappearing feels strange, I don't think it's actually a bug. I found this ticket describing the intended functionality. See the following points:

  1. Display an "x" for explicit tags in the tag drawer. Clicking the "x" will remove the tag from the content item.

5.5. The most granular tag in the hierarchy becomes the explicit tag with all its parent tags becoming implicit

  1. Removing an explicit child that is the only child of its implicit parent will remove the implicit parent too (and grandparent etc.)

Let me know if you agree that the actual behaviour is in line with what is described in that ticket.

Hopefully we'll get the opportunity to improve the tagging user experience in the future.

@cassiezamparini @ali-hugo I think the bug is of a different nature then. This is the intended behavior:

Removing an explicit child that is the only child of its implicit parent will remove the implicit parent too (and grandparent etc.)

But in the example video Cassie posted, when the grandchild tag (piano) was deleted, chordophone was also deleted, but instrument was not. In the story, I would assume that chordophone and instrument would both become implicit tags. Do you interpret it the same way?

@ali-hugo
Copy link

ali-hugo commented Jun 6, 2024

@jmakowski1123

In the story, I would assume that chordophone and instrument would both become implicit tags. Do you interpret it the same way?

I do.

It's strange though, because it looks like "percussion instruments" does become implicit after @cassiezamparini selects the "add" button (i.e. there is no longer a grey "x" next to it), but it is not treated as an implicit tag when its children tags are deleted (i.e. it does not get deleted).

@mariajgrimaldi
Copy link
Member

@jmakowski1123 @cassiezamparini @ali-hugo: could you confirm whether this is the intended behavior? Thanks!

@cassiezamparini cassiezamparini changed the title [Test failure] TC_AUTHOR_REDWOOD_42: Deleting third level child tags shouldn't remove second level child tags [Test failure] TC_AUTHOR_REDWOOD_42: Deleting third level child tags should remove parent and grandparent tags Jun 11, 2024
@cassiezamparini
Copy link
Author

cassiezamparini commented Jun 11, 2024

@jmakowski1123 So do you agree with the updated description here: #378 (comment)

cc. @mariajgrimaldi

@mariajgrimaldi mariajgrimaldi added this to the Redwood.3 milestone Sep 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Report of or fix for something that isn't working as intended needs triage redwood release testing Affects the upcoming release (attention needed)
Projects
Status: No status
Development

No branches or pull requests

4 participants