Add description to explorative annotations view #3035
Conversation
…plorative annotations view
There was a problem hiding this comment.
Nice 👍 I made a suggestion to improve the UX a bit, since name and description is a bit intertwined now.
Maybe, also only show the popover when clicking the icon, and on hover show a tooltip saying "Show description". That way, it becomes even more clear what the button does. Since the column title reads "Name", we should be careful that the user still understands what is going on.
| placement="topLeft" | ||
| > | ||
| <div> | ||
| <EditableTextLabel |
There was a problem hiding this comment.
Works well! However, I find it a bit weird from a UX perspective, that the popover always shows when you hover over the edit button or the name input. I'd change the structure as follows:
<div>
<EditableTextLabel ... />
<Popover>
<i className="fa fa-align-justify" />
</Popover>
</div>
That way, the description is a separate thing.
CHANGELOG.md
Outdated
| @@ -13,6 +13,7 @@ For upgrade instructions, please check the [migration guide](MIGRATIONS.md). | |||
|
|
|||
| - Added two new properties to mapping json files. The `colors: [<hsvHueValue1>, <hsvHueValue2>, ...]` property can be used to specify up to 256 custom colors for the first 256 equivalence classes of the mapping. The `hideUnmappedIds: <true|false>` property indicates whether segments that were not mapped should be rendered transparently or not. [#2965](https://github.com/scalableminds/webknossos/pull/2965) | |||
| - Added a button for refreshing the dataset in the backend cache. [#2975](https://github.com/scalableminds/webknossos/pull/2975) | |||
| - Added a popover to the explorative annotations view that displays when hovered over the name of an annotation. [#3035](https://github.com/scalableminds/webknossos/pull/3035) | |||
There was a problem hiding this comment.
I think, this is a bit too detailed. I'd change it to "Added the possibility to see the description of a tracing within the dashboard."
MichaelBuessemeyer
changed the title
|
@philippotto Please have a look at the newest changes that added the suggested tooltip. |
URL of deployed dev instance (used for testing):
Steps to test:
Issues: