-
Notifications
You must be signed in to change notification settings - Fork 112
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update kedro-viz lite banner icon #2128
Comments
Or maybe ℹ️ not |
Updated latest designs can be found here - https://www.figma.com/design/3kSpvIO1veLKfy9qHxuXwF/Kedro-WIP?node-id=2970-63490&t=9d2kW8aU1hd9wVYE-4 |
This is definitely an improvement, but I think this still sends the wrong message.
I would argue:
With the above in mind I'd like to pitch:
|
Hi @datajoely , Thank you for the suggestions. I agree with the informative approach. I have few issues with the text though:
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 |
I think your suggestions are great, @ravi-kumar-pilla - I'd also push for the banner to be less aggressive still, possibly more transparent :) |
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 - #2058
The text was updated successfully, but these errors were encountered: