Improve how SVG empty state illustrations are passed to dashboard and it's empty states

Background

Currently we must support 5 different illustrations for our empty states, and more will be added over time.

Each of those are added as a separate property to both the dashboard props and empty-state component props.

:empty-getting-started-svg-path="emptyGettingStartedSvgPath"
:empty-loading-svg-path="emptyLoadingSvgPath"
:empty-no-data-svg-path="emptyNoDataSvgPath"
:empty-no-data-small-svg-path="emptyNoDataSmallSvgPath"
:empty-unable-to-connect-svg-path="emptyUnableToConnectSvgPath"

This makes the code hard to read and could be more simple.

Proposal

Looking for ideas...

Original Discussion

The following discussion from !20900 (merged) should be addressed:

  • @kushalpandya started a discussion: (+1 comment)

    I wonder if we can combine all SVG related props into a single object, would it make sense?

Edited Dec 11, 2019 by Miguel Rincon
Assignee Loading
Time tracking Loading