Brain GUI modernization (Phase 5) - #14149
Conversation
a1516a7 to
c44b9e4
Compare
|
@wmvanvliet do you have time to look? If not, I can |
|
I love the trace list! I would love it even more if you could name the traces. Why did you remove the legend though? That was a useful feature and I don't remember us discussing its removal. "Set Atlas to None to see vertex and RMS traces here." <-- a bit weird message I think, because the traces work fine with an atlas selected. |
|
if the new trace list is meant to replace the old legend, then the vertex numbers and MNI coordinates need to be there. |
|
The text size of the trace list is a bit bigger than the size of the other text labels. It's probably better if they were the same size. |
Do you mean user-editable names? If you add
It was too small text for me, also when moving the time (dashed line) it jumps to left and right, but it's easy to return it (
agree, I will go for
I see, it should be "annotations" not "atlas" in the txt, becasue when you select an annotation, traces will be gone.
agree
agree. P.S. I know my eyesight isn't very good :) |
|
I use this short script for testing, I thought maybe it could be also useful for you for testing (also for next phases) @wmvanvliet |
Reference issue (if any)
Phase 5 of #14042 and following #14102
What does this implement/fix?
Additional information
Docs for the functions are written by Claude. Also test is written by Claude, but I made it minimal and faster.