kedro-viz icon indicating copy to clipboard operation
kedro-viz copied to clipboard

Update kedro-viz lite banner icon

Open stephkaiser opened this issue 1 year ago • 1 comments

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.

Related - https://github.com/kedro-org/kedro-viz/issues/2058

Image

stephkaiser avatar Oct 07 '24 15:10 stephkaiser

Or maybe ℹ️ not ⚠️ ?

datajoely avatar Oct 07 '24 15:10 datajoely

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

Image

stephkaiser avatar Nov 12 '24 19:11 stephkaiser

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

datajoely avatar Nov 13 '24 10:11 datajoely

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

  1. Regarding the term Python dependencies in the message is a bit confusing to me.
  2. 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

ravi-kumar-pilla avatar Nov 15 '24 16:11 ravi-kumar-pilla

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

datajoely avatar Nov 15 '24 19:11 datajoely