kiali icon indicating copy to clipboard operation
kiali copied to clipboard

Debug information autoscroll when copying from the text field

Open matejnesuta opened this issue 2 years ago • 3 comments

Found this one during the "Help" dropdown testing.

When I open the Help dropdown and choose the "View Debug Info", I am presented with a modal, which includes two text fields. It's entire content gets copied after a single click or with the "Copy" button.

The issue is that it scrolls automatically all the way to the bottom when the content is copied either with the button, or with the click. It's confusing visually and if I wanted to read it's content, one accidental click could ruin it.

Before copying: image

After copying: image

The "Health config" behaves differently even though it looks exactly the same as the bottom field, which is a little confusing as well.

Not sure, if bug or a feature.

matejnesuta avatar Aug 16 '22 16:08 matejnesuta

I have always hated that separate "health config" text box at the top. Why is "health config" special? It is already in the debug yaml. This just duplicates the output and makes this page more complicated and confusing than it needs to be.

I say we take this opportunity to collapse this dialog window so it only shows the full "Please include this information" text area and removes that health yaml which, as I understand, is nothing more than a subset of what you see in the full yaml text area.

jmazzitelli avatar Aug 16 '22 16:08 jmazzitelli

I agree about those two sections, I never really understood that. Also, don't we use a nice YAML editor in other places, why not here? Having said all that, this is pretty low priority. I'll add to the backlog but also a "Good First issue", maybe we can get a contribution.

jshaughn avatar Aug 16 '22 17:08 jshaughn

Also, don't we use a nice YAML editor in other places, why not here?

+1... that's a good idea.

jmazzitelli avatar Aug 16 '22 17:08 jmazzitelli

Closing as this page has now been redesigned.

jshaughn avatar Jan 13 '23 18:01 jshaughn