Conversation
I'm not override frozen to left | right | null, I just add rightFrozen property. It will not break original version.
robert-luoqing
requested review from
amanmahajan7 and
nstepien
as code owners
|
this is a necessary feature |
|
@amanmahajan7 any news on this? |
|
Whats needed to get this across the finish line? willing to put resources on it |
|
Any update on this feature merging? |
| /** Determines whether column is frozen or not */ | ||
| readonly frozen?: Maybe<boolean>; | ||
| /** Determines whether column is right frozen or not */ | ||
| readonly rightFrozen?: Maybe<boolean>; |
There was a problem hiding this comment.
I would rather not add a new prop that can cause conflicts. What happens when both rightFrozen and frozen props are set? We can instead change frozen to boolean | 'left' | 'right'.
| const defaultDraggable = defaultColumnOptions?.draggable ?? false; | ||
|
|
||
| const { columns, colSpanColumns, lastFrozenColumnIndex, headerRowsCount } = useMemo((): { | ||
| const { columns, colSpanColumns, lastFrozenColumnIndex, rightFrozenColumnCount, headerRowsCount } = useMemo((): { |
There was a problem hiding this comment.
All the new functionality needs extensive tests
|
robert seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account. You have signed the CLA already but the status is still pending? Let us recheck it. |
|
This PR has been stuck since July 2025. Right-side column pinning is a feature I need — happy to take it over with a fresh PR. Before I invest time:
If there's genuine interest, I'll open a PR within ~2 weeks. If not, please say so — I'll plan accordingly. Thanks. |
8 participants
I'm not override frozen to left | right | null, I just add rightFrozen property. It will not break original version.