-
Notifications
You must be signed in to change notification settings - Fork 15
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
Comments
cc. @jmakowski1123 FYI |
@jmakowski1123 @ali-hugo and I were chatting about this, so just to confirm...
|
@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:
5.5. The most granular tag in the hierarchy becomes the explicit tag with all its parent tags becoming implicit
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. |
@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? |
@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? |
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). |
@jmakowski1123 @cassiezamparini @ali-hugo: could you confirm whether this is the intended behavior? Thanks! |
@jmakowski1123 So do you agree with the updated description here: #378 (comment) cc. @mariajgrimaldi |
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
Additional information
No response
The text was updated successfully, but these errors were encountered: