CC-177 Give neuroglancer layers a tag dependent on their type #67
Conversation
src/ui/layer_list_panel.css
Outdated
| @@ -33,6 +48,13 @@ | |||
| stroke: #999; | |||
| } | |||
|
|
|||
| .neuroglancer-layer-side-panel-name { | |||
There was a problem hiding this comment.
It would be better if we modify the CSS only for the name item when it is in the layer list panel. This is too broad and affects the name when it is in the layer side panel, and when it is part of a tool.
For example, we could add a class for this. Or update the selector
| .neuroglancer-layer-list-panel-item input[type="checkbox"] { | ||
| width: 1rem; | ||
| height: 1rem; | ||
| margin: 4px; |
There was a problem hiding this comment.
I think this margin is too much, might need to review design
There was a problem hiding this comment.
I see, in the design the checkbox itself is 16x16 and has margin of 4px and a gap between items is 4px:
but if it looks too much we can ask @syamace-metacell to reduce the size
What do you think @seankmartin?
src/ui/layer_list_panel.css
Outdated
| margin: 4px; | ||
| } | ||
|
|
||
| .neuroglancer-layer-list-panel-item .neuroglancer-icon { |
There was a problem hiding this comment.
Is this targeting just the eye icon? If so it would be better to give that a class to make this more explicit
Issue #CC-177
Problem: Give neuroglancer layers a tag dependent on their type
Solution:
Result:
Recording.2025-01-24.130409.mp4