vscode-dvc icon indicating copy to clipboard operation
vscode-dvc copied to clipboard

Show a notification if the user trying to toggle exp and 7 expS are already toggled

Open maxagin opened this issue 3 years ago • 4 comments

I think we can show a notification You can toggle maximum 7 experiments if the user trying to toggle a new exp (we have a max 7). Because I kinda thought that this is a bug.

https://user-images.githubusercontent.com/98249521/175383474-d21f3912-e450-4240-b584-fb840fdbc585.mov

maxagin avatar Jul 15 '22 04:07 maxagin

@mattseddon Once a user is aware that they can only select 7 this no longer becomes an issue. Following the UX guidelines diagram I would take this path: image FYI https://code.visualstudio.com/api/ux-guidelines/notifications contains full guidelines on notifications.

Very good, but I actually was aware of the fact that only 7 can be selected and like a fool was trying again and again to select a new one :) We need to make sure to give the correct response to any action of the user I think.

maxagin avatar Jul 15 '22 04:07 maxagin

Once a user is aware that they can only select 7

How can we make them aware?

Can there be some very light-weight / non-intrusive reaction (like show a tooltip similar to the one we show on hover) - I don't think it can do any harm

shcheklein avatar Jul 15 '22 05:07 shcheklein

@maxagin any specific reason it was assigned to me? :)

shcheklein avatar Jul 25 '22 22:07 shcheklein

Do not remember now @shcheklein . Probably by mistake.

maxagin avatar Jul 25 '22 23:07 maxagin

👍 I find this confusing. It always takes me a minute to remember why clicking another experiment doesn't work and feels like it's broken until I remember this -- how is a new user supposed to know?

dberenbaum avatar Nov 03 '22 16:11 dberenbaum