Skip to content

Conversation

@kimanhou
Copy link
Contributor

@kimanhou kimanhou commented Oct 1, 2025

Done

  • Create ColumnSelector component that toggles/untoggles columns of a table

QA

Pinging @canonical/react-library-maintainers for a review.

Storybook

To see rendered examples of all react-components, run:

yarn start

QA in your project

from react-components run:

yarn build
npm pack

Install the resulting tarball in your project with:

yarn add <path-to-tarball>

QA steps

  • In Storybook, check that toggling/untoggling columns in the ColumnSelector makes them appear/disappear in the associated table
  • Columns hidden because of size should be disabled

Percy steps

  • UI should look like in LXD UI, see screenshots below
Screenshot from 2025-10-01 17-39-10 image

Screenshot after Cluster Member column was untoggled:
image

@webteam-app
Copy link

Copy link
Contributor

@edlerd edlerd 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 raising this upstream. Some comments and ideas for the structure below.

Copy link
Contributor

@edlerd edlerd left a comment

Choose a reason for hiding this comment

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

Tests, Stories and QA LGTM

Please extend on docs and some small nitpicks on naming.

… table

Signed-off-by: Kim Anh Nguyen <4783194+kimanhou@users.noreply.github.com>
@kimanhou kimanhou force-pushed the WD-4728-lxd-ui-column-toggle branch from 193072a to 46d0c82 Compare October 3, 2025 07:38
@kimanhou kimanhou marked this pull request as ready for review October 3, 2025 08:18
Copy link
Contributor

@edlerd edlerd left a comment

Choose a reason for hiding this comment

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

QA and Code LGTM, thanks for upstreaming :)

@kimanhou kimanhou merged commit 5552db5 into canonical:main Oct 6, 2025
8 checks passed
@github-actions
Copy link

github-actions bot commented Oct 6, 2025

🎉 This PR is included in version 3.3.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@kimanhou kimanhou deleted the WD-4728-lxd-ui-column-toggle branch October 27, 2025 10:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants