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

Added a Select All checkbox option #65

Merged
merged 2 commits into from
Feb 6, 2025
Merged

Conversation

RealRTorres
Copy link
Contributor

Description

Added a checkbox at the top to the left of the search bar, where you can select all clusters in the table.

Related Issue

Fixes #50

Changes Made

  • Added a Select All checkbox above the table, to the left of the search bar
  • Added padding to both Select All checkbox and table checkboxes

Checklist

Please ensure the following before submitting your PR:

  • I have reviewed the project's contribution guidelines.
  • I have written unit tests for the changes (if applicable).
  • I have updated the documentation (if applicable).
  • I have tested the changes locally and ensured they work as expected.
  • My code follows the project's coding standards.

Screenshots or Logs (if applicable)

image

Additional Notes

N/A

@clubanderson clubanderson merged commit cc0ea6d into kubestellar:dev Feb 6, 2025
1 check passed
@clubanderson
Copy link
Contributor

@RealRTorres you might consider putting the checkbox inside the table header - that is where it is usually found in standard table selection dialogs

@RealRTorres
Copy link
Contributor Author

Okay, thanks for the feedback! I'll fix this tomorrow

@RealRTorres RealRTorres mentioned this pull request Feb 6, 2025
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

LFX mentorship (2025/term1): Implement an ITS frontend supported by ITS backend endpoints
2 participants