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

Display full hierarchy for taxonomy metadata #283

Closed
leogermani opened this issue Jul 19, 2019 · 6 comments

Comments

@leogermani
Copy link
Contributor

commented Jul 19, 2019

When displaying the value for a taxonomy metadata, display the whole tree, and not just the seleced term, in a breadcrumb format:

Musical Instruments > Percussion > Tamborim

@leogermani leogermani added this to Backlog in tainacan/tainacan Jul 19, 2019

@leogermani leogermani self-assigned this Jul 19, 2019

@leogermani leogermani added the 0.11 label Jul 19, 2019

@mateuswetah

This comment has been minimized.

Copy link
Contributor

commented Jul 19, 2019

I'm wondering how would this go for an item that has many terms, being most of them from the same parent... Won't it make it too polluted? Shouldn't we display this only in a tooltip or in something collapsable?

@leogermani

This comment has been minimized.

Copy link
Contributor Author

commented Jul 19, 2019

I think we should give it a try and see how it goes in the real world

@leogermani leogermani moved this from Backlog to To Do in tainacan/tainacan Jul 29, 2019

@leogermani leogermani moved this from To Do to In Progress in tainacan/tainacan Jul 29, 2019

leogermani added a commit that referenced this issue Jul 29, 2019

@leogermani leogermani moved this from In Progress to Validation in tainacan/tainacan Jul 29, 2019

@leogermani

This comment has been minimized.

Copy link
Contributor Author

commented Jul 29, 2019

Captura de tela de 2019-07-29 16-16-12

@mateuswetah

This comment has been minimized.

Copy link
Contributor

commented Jul 29, 2019

Do they have a <span> tag around with a class so that a theme can customize the way we did with the separators?

@leogermani

This comment has been minimized.

Copy link
Contributor Author

commented Jul 30, 2019

No, will add it. What you suggest?

  • add a HTML element with a class in the separator

Also, I will add a filter so the CSV exporter writes the same format the importer accepts, which uses ">>"

  • add a filter to change the separator and make CSV exporter use it
@mateuswetah

This comment has been minimized.

Copy link
Contributor

commented Jul 30, 2019

Great!
Well, we used multivalue-separator for the previous, so maybe hierarchy-indicator?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
3 participants
You can’t perform that action at this time.