Skip to content

feat(ws): Add Empty State to Workspace list #268

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

Open
wants to merge 1 commit into
base: notebooks-v2
Choose a base branch
from

Conversation

liavweiss
Copy link

@liavweiss liavweiss commented Apr 28, 2025

fix #259
image

Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign ederign for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@paulovmr
Copy link

/ok-to-test

Copy link

@paulovmr paulovmr left a comment

Choose a reason for hiding this comment

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

Hi @liavweiss , thanks for the PR. Please consider extracting the empty state from the WorkspaceKinds.tsx file as a reusable component and use it here, also including the CLEAR ALL FILTERS button.

))}
{sortedWorkspaces.length === 0 && (
<Tr>
<Td colSpan={11} id="empty-state">
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
<Td colSpan={11} id="empty-state">
<Td colSpan={12} id="empty-state">

The table has 12 columns including the columnNames and the two actions columns.

Copy link
Member

@jenny-s51 jenny-s51 left a comment

Choose a reason for hiding this comment

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

Thanks for implementing the empty state in the workspace table @liavweiss! Great work.

A couple of suggestions for improvement:

  • I agree with @paulovmr - we should extract the empty state component into a dedicated reusable component to ensure consistency when we need similar empty states elsewhere.

  • Can we also include a "Clear all filters" button in the empty state so the users can easily click the action button to recover from the empty state? https://www.patternfly.org/components/empty-state/#no-match-found is a good example of this in action.

@ederign
Copy link
Member

ederign commented May 1, 2025

/retitle feat(ws): Add Empty State to Workspace list

@google-oss-prow google-oss-prow bot changed the title feat(ws): Notebooks 2.0 // Frontend // Workspace table // Add Empty State #259 feat(ws): Add Empty State to Workspace list May 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Needs Triage
Development

Successfully merging this pull request may close these issues.

4 participants