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

feat: add grid view option for systems table #315

Merged
merged 2 commits into from
Dec 8, 2024
Merged

Conversation

0xMMMMMM
Copy link
Contributor

@0xMMMMMM 0xMMMMMM commented Dec 6, 2024

  • Add toggle button to switch between table and grid layouts
  • Implement card-based grid view with system metrics
  • Add sort dropdown menu for grid view
  • Display system status, metrics with icons and labels
  • Improve handling of long system names
  • Maintain consistent sorting and filtering between views
  • Persist view preference in localStorage

- Add toggle button to switch between table and grid layouts
- Implement card-based grid view with system metrics
- Add sort dropdown menu for grid view
- Display system status, metrics with icons and labels
- Improve handling of long system names
- Maintain consistent sorting and filtering between views
- Persist view preference in localStorage
@henrygd
Copy link
Owner

henrygd commented Dec 6, 2024

Thanks, I took a quick look and I like it a lot! Much better layout for mobile devices especially.

Maybe we should just have one button called "View" or "Options", which opens a dropdown menu to manage layout / fields / sorting. That would also make it easier to add more options in the future.

I don't have time today, but I'll merge tomorrow and play around with it. Thanks again!

Combines view type, sort by and visible fields into a single dropdown menu for better organization.
@0xMMMMMM
Copy link
Contributor Author

0xMMMMMM commented Dec 7, 2024

Hey, see if the new Options button suit your taste!

@henrygd henrygd merged commit 5110eaf into henrygd:main Dec 8, 2024
@henrygd
Copy link
Owner

henrygd commented Dec 8, 2024

Merged and made some small tweaks.

Great contribution, thank you! 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants