Control sorting through props for iTwin and iModel tables

[arome]

Added the ability to control the sorting of the tables through props.

[alexdunae]

Working great in my local testing 💪

[ben-polinsky]

While we're the only consumers of MUI it feels like a rare opportunity to make API changes.

How are we going to know when someone else starts consuming the MUI components?

@alexdunae

[ben-polinsky]

Sort breaks when trying to sort by a hidden column.

{
  "hideColumns": [
    "ITwinName"
  ]
}

I'm not sure the sort column chip should appear if that column is being hidden, that could be implementation specific.

But aside from that, the records are not sorted by name in the above case, if I unhide the column:

Not sure if this is just the storybook implementation, or if it traces back to the component.

[alexdunae]

While we're the only consumers of MUI it feels like a rare opportunity to make API changes.

How are we going to know when someone else starts consuming the MUI components?

@ben-polinsky Grrr... actually a tough question. I've done a search internally and found one consumer, but of course there is no actual way to know about public usage.

The internal consumer would be unaffected by this change.

[ben-polinsky]

While we're the only consumers of MUI it feels like a rare opportunity to make API changes.

How are we going to know when someone else starts consuming the MUI components?

@ben-polinsky Grrr... actually a tough question. I've done a search internally and found one consumer, but of course there is no actual way to know about public usage.

It's absolutely a "grr..." :)

Anyway, I think we're okay for these changes - perhaps we could mark the components as beta and subject to change? I don't want to hold up this PR any longer, but do think it's worth discussing moving forward.

[alexdunae]

@ben-polinsky agreed - we added @alpha for all the MUI stuff

admin-components-react/packages/modules/imodel-browser/src/mui/containers/iModelGrid/IModelGridMUI.tsx

Line 68 in a104fcf

/** @alpha */

[arome]

Sort breaks when trying to sort by a hidden column.

{
  "hideColumns": [
    "ITwinName"
  ]
}

I'm not sure the sort column chip should appear if that column is being hidden, that could be implementation specific.

The column chips are meant to simulate sorting controlled from an external component in that story, they're not part of the iTwinGrid, so hiding the column in the grid shouldnt affect the chips.

But aside from that, the records are not sorted by name in the above case, if I unhide the column:

Not sure if this is just the storybook implementation, or if it traces back to the component.

hmm, I wasnt able to reproduce that, if i hide a column and sort by that column using the chips, im still getting the proper sorting.

[ben-polinsky]

hmm, I wasnt able to reproduce that, if i hide a column and sort by that column using the chips, im still getting the proper sorting.

Fair enough. I was on the fence about holding this up for this "issue" anyway. Not sure what I was doing wrong, but appreciate you checking it out.

[arome]

Fixed the flaky test in manage-versions that was failing unrelated to my changes in this PR

[DanishMehmood-bit]

This should've been marked as a major change since you do remove props from the Grid components