Skip to content

feat: hide columns cell override - #185

Merged
leo-belanger merged 14 commits into
mainfrom
leobelanger/hide-columns
Nov 12, 2025
Merged

feat: hide columns cell override#185
leo-belanger merged 14 commits into
mainfrom
leobelanger/hide-columns

Conversation

@leo-belanger

@leo-belanger leo-belanger commented Oct 31, 2025

Copy link
Copy Markdown
Contributor

We have the requirement to hide a column in a specific scenario. I've added the possibility to hide columns while in cells view for the iModelGrid.

I've added an enum that represents each columns in cell view.

I've also updated the storybook so that we can play with that option.

@leo-belanger
leo-belanger marked this pull request as ready for review October 31, 2025 19:19
@aruniverse
aruniverse requested review from Copilot and removed request for DanishMehmood-bit October 31, 2025 19:55

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds the capability to hide specific columns in the iModel grid's "cells" view mode. The feature enables developers to selectively hide columns like name, description, created date, or options column based on specific requirements.

  • Introduced an enum to standardize column identifiers for better type safety
  • Added a new hideColumns property to the cell overrides configuration
  • Updated table configuration to filter out hidden columns dynamically

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
packages/modules/imodel-browser/src/types.ts Added IModelCellColumn enum and hideColumns property to IModelCellOverrides
packages/modules/imodel-browser/src/containers/iModelGrid/useIModelTableConfig.tsx Updated column IDs to use enum values and implemented column filtering logic
packages/apps/storybook/src/imodel-browser/IModelGrid.stories.tsx Added example usage of hideColumns feature in storybook

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread packages/apps/storybook/src/imodel-browser/IModelGrid.stories.tsx Outdated
@DanishMehmood-bit

Copy link
Copy Markdown
Contributor

Can you also add this option to ITwinGrid? Since I can see it being useful for both iModel and iTwin table

@leo-belanger

Copy link
Copy Markdown
Contributor Author

Can you also add this option to ITwinGrid? Since I can see it being useful for both iModel and iTwin table

@DanishMehmood-bit Good idea, I'll do that!

@leo-belanger

Copy link
Copy Markdown
Contributor Author

Can you also add this option to ITwinGrid? Since I can see it being useful for both iModel and iTwin table

@DanishMehmood-bit Done!

Comment thread packages/modules/imodel-browser/src/types.ts
@leo-belanger
leo-belanger force-pushed the leobelanger/hide-columns branch from a0a9c9a to e3c65ab Compare November 5, 2025 14:15
…lumns_2025-11-05-14-20.json

Co-authored-by: Arun George <11051042+aruniverse@users.noreply.github.com>
@leo-belanger
leo-belanger merged commit a55806b into main Nov 12, 2025
5 checks passed
@leo-belanger
leo-belanger deleted the leobelanger/hide-columns branch November 12, 2025 15:31
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.

5 participants