zui icon indicating copy to clipboard operation
zui copied to clipboard

Right-click "Open details" feels like a no-op

Open philrz opened this issue 1 year ago • 2 comments

Repro is with Brim commit 124d658.

I was clicking through some current workflows in the app with intent to start recording videos as documentation. As I was going through all the current options on the right-click menu, I was struck by the sense that "Open details" felt like a no-op. This actually led me back to go back to GA Brim v0.31.0 and remind myself what it looked like in the past. As shown in the attached video, I was quickly reminded that it simply opened the right-side "Log Details" panel, and of course the details for the clicked record would be shown there.

https://user-images.githubusercontent.com/5934157/190536876-616cc293-13e9-49e3-b1c1-2c6e989f2a20.mp4

However, compare this with what I experienced while pretending to be a brand new Zui user at commit 124d658 in this next video. Since the right panel is now open by default, clicking it after importing my first data appears to do nothing. Knowing what I know as an insider about what it did in the past, I then hit the hotkey to close the right panel and selected "Open details" once again. Indeed, this opened the right panel, but didn't even switch to the "Detail" tab within that panel. I had to click that manually as an extra step. Therefore, it's effectively a straight-up bug at this point.

https://user-images.githubusercontent.com/5934157/190537041-c35089e8-9455-4e16-9b49-9eca0bb56935.mp4

In terms of what we might make "Open details" do instead as a part of a fix, my first thought is that the separate "Details" window could be popped out, just like what happens when a record is double-clicked. Indeed, I suspect that the original panel-centric functionality of the right-click "Open details" might date all the way back to before the popped out "Details" window even existed, so this change might be overdue.

philrz avatar Sep 16 '22 01:09 philrz

I think if the detail pane is open, this option should be disabled. But if the detail pane is not open, it should open the right pane and move to the detail tab.

I don't like buttons that can be clicked but don't do anything.

jameskerr avatar Sep 21 '22 18:09 jameskerr

@jameskerr: That proposal works for me!

philrz avatar Sep 21 '22 21:09 philrz