Skip to content

fix (DataTable): Columns not getting dynamically frozen#7860

Merged
melloware merged 1 commit intoprimefaces:masterfrom
acc-cassio:dtable-fix-frozencol
Apr 4, 2025
Merged

fix (DataTable): Columns not getting dynamically frozen#7860
melloware merged 1 commit intoprimefaces:masterfrom
acc-cassio:dtable-fix-frozencol

Conversation

@acc-cassio
Copy link
Copy Markdown
Contributor

Defect Fixes

Fixed bug #7545 which was re-introduced after after PR #7420 was merged.

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 4, 2025

Deployment failed with the following error:

Creating the Deployment Timed Out.

@melloware melloware merged commit dc0160c into primefaces:master Apr 4, 2025
3 of 4 checks passed
@balavignesh111
Copy link
Copy Markdown

balavignesh111 commented May 2, 2025

Hi team,

I kind of face an issue like frozen column is not working for more than two columns. Kindly give me an solution for this.
@acc-cassio @melloware @Sharmila-g @DurgaSivakumar

Issue description :
Actuallly the issue is in css. When giving alignFrozen="right" or "left", based on it right or left css property will apply to tag. It is working for two columns which i pinned in right/ left, but when it crosses more than two columns the css right/left property is not calculating correctly. So the automatic calculation is the problem here.

Version:
PrimeReact- 10.9.5
react- 18 above

https://stackblitz.com/edit/tu17mfd2-ngjschbq?file=package.json

@melloware
Copy link
Copy Markdown
Member

@balavignesh111 did this work before 10.9.5? I didn't think you were allowed to freeze "non contiguous" columns?

@balavignesh111
Copy link
Copy Markdown

@melloware currently i am using 10.8.1 it was working perfectly for that version. but i dont know that it was not working for 10.9.1 version

@melloware
Copy link
Copy Markdown
Member

@balavignesh111 put together a stackblitz showing it was working before so we can debug.

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.

3 participants