Skip to content
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

EmptyState implementation inconsistency with Vanilla Framework #1071

Closed
vladimir-cucu opened this issue Apr 18, 2024 · 6 comments
Closed

EmptyState implementation inconsistency with Vanilla Framework #1071

vladimir-cucu opened this issue Apr 18, 2024 · 6 comments
Labels
Bug 🐛 Something isn't working

Comments

@vladimir-cucu
Copy link
Contributor

Vanilla Framework's design of EmptyState component can be found here and the react-components library's design can be found here. The image is rendered to the left of the text in Vanilla Framework, while it is rendered on top of the text in react-components library.

@vladimir-cucu
Copy link
Contributor Author

Had a small chat today with @lukasSerelis and he specified that the Empty State design from Vanilla Framework is the one we should use in the web apps as well.

@vladimir-cucu vladimir-cucu added the Bug 🐛 Something isn't working label Apr 26, 2024
@bartaz
Copy link
Contributor

bartaz commented Apr 26, 2024

Thanks for confirming that. I'm not sure why this inconsistency was introduced, but will be good to fix that to avoid confusion.

@vladimir-cucu
Copy link
Contributor Author

@bartaz I'm currently looking into how Vanilla Framework implemented the Empty State and the implementations of all 3 types of Empty States differ. Are we only concerned with implementing the No content Empty State in this component?

@bartaz
Copy link
Contributor

bartaz commented Apr 26, 2024

Yes, the Vanilla docs for Empty State pattern are very generic and cover variety of cases (search, 404, etc). For apps the most relevant is "no content" empty state, so that was meant to be covered by this React component.

@bartaz
Copy link
Contributor

bartaz commented Apr 26, 2024

Also note that this is very context specific, so image may be different, text obviously. The action button also should be optional.

image

Overall the purpose of empty state component is to provide information about what's missing, and ideally explain the action to be taken to add something.

@jmuzina
Copy link
Member

jmuzina commented May 17, 2024

Closing as completed in #1076

@jmuzina jmuzina closed this as completed May 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug 🐛 Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants