Update kedro-viz lite banner icon

kedro-org / kedro-viz

Visualise your Kedro data and machine-learning pipelines and track your experiments.

https://demo.kedro.org

Apache License 2.0

682 stars 113 forks source link

Update kedro-viz lite banner icon #2128

Closed stephkaiser closed 5 days ago

stephkaiser commented 1 month ago

Description

We've gotten feedback that the user warning banner for kedro-viz lite is too aggressive and gives the impression that something is broken or there is an issue.

Possible Implementation

A quick fix to this would be to change the icon on the banner from a X to a warning icon instead.

datajoely commented 1 month ago

Or maybe ℹ️ not ⚠️ ?

stephkaiser commented 1 week ago

Updated latest designs can be found here - https://www.figma.com/design/3kSpvIO1veLKfy9qHxuXwF/Kedro-WIP?node-id=2970-63490&t=9d2kW8aU1hd9wVYE-4

datajoely commented 1 week ago

This is definitely an improvement, but I think this still sends the wrong message.

Missing dependencies - For the best experience with full functionality, please install the missing Kedro project dependencies.

I would argue:

The banner should still be less prominent, more information than warning.
If the user has intentionally run in --lite mode (which we're already seeing user value for - great job team) I think they don't need to be told they're not getting the "best" expereince. It's an intentional action.

With the above in mind I'd like to pitch:

"ℹ️ Features such as dataset previews are disabled in --lite mode as Python dependencies are not required [Learn More]`

The user is being informed of the limitations of their chosen approach
There is not suggestion what they're doing is wrong, just differen
It's slightly less mysterious about what constitutes the full experience

ravi-kumar-pilla commented 1 week ago

Hi @datajoely , Thank you for the suggestions. I agree with the informative approach. I have few issues with the text though:

Regarding the term Python dependencies in the message is a bit confusing to me.
The feature dataset preview is not entirely disabled in lite mode. Example if someone installed csv dataset, they can preview csv dataset but cannot preview an excel or other datasets whose dependencies are missing. Lite mode only mocks dependencies that are missing.

Something along the below lines would be fine -

"ℹ️ Some features might be disabled in --lite mode due to missing dependencies. [Learn More]`

-> User can click on Learn More to find out the limitations, but we show an informative generic text. What do you think ?

Thank you

datajoely commented 1 week ago

I think your suggestions are great, @ravi-kumar-pilla - I'd also push for the banner to be less aggressive still, possibly more transparent :)