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

Add empty state feedback and animation to scanning process - Updates #4 #7

Merged
merged 5 commits into from Oct 12, 2022

Conversation

Le-Roi777
Copy link
Contributor

  • Updates Add blank slate and loading animation #4

  • Added a loading state in the App component and passed it to the table components as prop (loadingRows). In the table component the following have been added to accommodate feedback for the scanning process.

  • Added text feedback and loading indicator to the table component when rows are loading.

  • Added initial state placeholder with an icon and text when no scan has been initialized. (if we don't have a response/rows).

  • Added no results found feedback with an icon and text when the scan results are empty. (if we don't have a response/rows).

  • Updates update README with development steps #3

  • I also updated the README with development instructions : - )

empty-state

loading-state

@ibreakthecloud
Copy link
Member

@Le-Roi777 Good stuff! Thanks for the PR. I'll review it.

@ibreakthecloud ibreakthecloud added enhancement New feature or request extension/ui If the PR has changes related to frontend labels Oct 12, 2022
ui/src/App.tsx Outdated Show resolved Hide resolved
Copy link
Member

@ibreakthecloud ibreakthecloud left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request extension/ui If the PR has changes related to frontend
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants