atom icon indicating copy to clipboard operation
atom copied to clipboard

Problem: Long custom levels of description prevent sidebar treeview nodes from being clicked

Open fiver-watson opened this issue 10 months ago • 0 comments

Current Behavior

First reported in the user forum, 2024-04-05 - link. Not sure what version they were using, but they specified the BS5 plugin, so either 2.7.x or 2.8.x.

Reproduced locally in the Docker env, running the latest qa/2.x as of 2023-04-05 - settings read version as: 2.8.2 - 193

Steps to reproduce the behavior

  1. Log in as an administrator
  2. Navigate to Admin > Settings > Treeview
  3. Set the treeview to use the Sidebar treeview, and Save
  4. Navigate to Manage > Taxonomies > Levels of description
  5. Add a new level of description: Sub-sub-sub-sub-sub-sub-subseries. Save
  6. Find or create a top-level description
  7. Add a new child description. Give it the new sub-sub level of description. Save
  8. Navigate back to the top level description
  9. Try to use the treeview to return to the child record

Resulting error

Because the level of description is not part of the hyperlink, a long custom level of description will prevent users from clicking on the node.

Expected Behavior

Given that users can create their own custom levels of description, AND given that even the default levels may be longer in other languages, a long LoD should not prevent a user from clicking on a sidebar treeview node.

Possible Solution

I would like to propose that for the sidebar treeview, the entire node should be a hyperlink - I am not sure what advantage it adds to keep the LoD outside of the link. If we made the entire span / treeview node a link, then it shouldn't matter how much or how little of the tree is being displayed

Another possible solution:

On hover, remove the "text-truncate" class, so the entire title of the node will display, and users can click on the part of the node that is a hyperlink. When I tested in my browser, without this class the title does still wrap properly in the treeview.

Context and Notes

Likely an issue in BS2 as well, but given its imminent removal and current deprecation, I don't think we need to fix it there.

Version used

2.8.2 - 193

Operating System and version

Ubuntu 20.04

Default installation culture

en

PHP version

PHP 7.4

Contact details

No response

fiver-watson avatar Apr 05 '24 13:04 fiver-watson