Bug #5404

Collapsing term in Places context treeview (Browse Places) causes treeview to disappear

Added by Dan Gillean almost 9 years ago. Updated over 8 years ago.

Status:VerifiedStart date:07/24/2013
Priority:MediumDue date:
Assignee:José Raddaoui Marín% Done:

100%

Category:Taxonomy / TermEstimated time:4.00 hours
Target version:Release 2.0.0
Google Code Legacy ID: Tested version:
Sponsored:No Requires documentation:

Description

To reproduce
1) Navigate to Browse > places and select a term that is used in a description, and which has a parent (in dev site - choose Vancouver)
2) Collapse top-level term to see other top-level terms in treeview (i.e., Canada)

Resulting error
Treeview disappears

Expected result
All top level terms are viewable in the places treeview.

History

#1 Updated by Jesús García Crespo almost 9 years ago

  • Estimated time set to 4.00

#2 Updated by José Raddaoui Marín over 8 years ago

  • Status changed from New to In progress
  • Assignee changed from Jesús García Crespo to José Raddaoui Marín

#3 Updated by José Raddaoui Marín over 8 years ago

  • Status changed from In progress to QA/Review
  • % Done changed from 0 to 100

Applied in changeset atom|commit:4d1321b7d85501a7f4c2aa817147310f176617fb.

#4 Updated by José Raddaoui Marín over 8 years ago

Hi Dan,

The behaviour you're expecting was changed, at least in the information objects treeview. Now you won't be able to collapse the first element in the terms treeview, like the other treeview.

#5 Updated by Dan Gillean over 8 years ago

  • Status changed from QA/Review to Feedback

Hi Radda,

What if there are other terms at the same level - ex: I can't collapse Canada in the AC data if I enter by clicking on Ontario in Browse > Places, but what if there is Cameroon, Cambodia, etc - other taxonomies for other places at the same top-level as Canada?

Also: try this in the 2x test site:

1) Browse > Places
2) Click on Africa
3) Expand treeview until you can see Canada
4) Try to expand Canada

Resulting error
Treeview disappears

#6 Updated by José Raddaoui Marín over 8 years ago

  • Status changed from Feedback to QA/Review

Applied in changeset atom|commit:ad2de1596b00b67e6ae8fa564858a89054781812.

#7 Updated by José Raddaoui Marín over 8 years ago

Hi Dan,

I agree with you, we shouldn't use the same behaviour in the terms treeview. I think I finally got the behaviour you're expecting. Also the second problem you said should be solved.

#8 Updated by Dan Gillean over 8 years ago

  • Status changed from QA/Review to Verified

Nice! I tried, I couldn't break it :D

Also available in: Atom PDF