Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: #13781 || dataTable: frozenColumn header overlap with frozen body #13791

Conversation

ashikjs
Copy link
Contributor

@ashikjs ashikjs commented Oct 1, 2023

Defect Fixes

Fix: #13781

Explanation

I apologize for any oversight in my previous pull request. In this new pull request, I've taken steps to ensure that the changes won't adversely affect any other parts of the codebase.

The primary change in this PR involves adjusting the z-index values for various elements. Specifically, I've set the z-index of datatable-thead and rowgroup-header to level 2, and the datatable-loading-overlay now has a z-index of 3.

I also considered raising the z-index of datatable-tfoot to level 2, but after thorough testing, I couldn't identify any issues with the current setup. I'm somewhat uncertain about whether this adjustment is necessary, so I'd appreciate any guidance or suggestions in this regard.

Please review the changes, and I'm open to any feedback or additional adjustments to ensure the code remains robust. Thank you for your time and consideration.

@vercel
Copy link

vercel bot commented Oct 1, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
primeng-ssr-test ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 1, 2023 3:10pm
1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
primeng ⬜️ Ignored (Inspect) Visit Preview Oct 1, 2023 3:10pm

@ashikjs
Copy link
Contributor Author

ashikjs commented Oct 1, 2023

@SoyDiego I will be happy if you check this PR.

@ashikjs
Copy link
Contributor Author

ashikjs commented Oct 1, 2023

@gucal hi, can you please check this PR. Thanks in advance.

Please feel free give me any feedback .

@Pixelaters
Copy link

Solution worked for me thanks

@CurstinJR
Copy link

CurstinJR commented Oct 24, 2023

Solution worked for me thanks

Yea seems like the z index on the tables are abit finicky but changing the values of it seemed to work. Hopefully the PR can get tested and approved

@ashikjs
Copy link
Contributor Author

ashikjs commented Nov 6, 2023

@gucal hi, can you please check this PR. Thanks in advance.

@cetincakiroglu cetincakiroglu merged commit f70d685 into primefaces:master Nov 22, 2023
5 checks passed
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.

dataTable : <th> is hidden by <td> with frozenColumn
4 participants