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

fixedHeader classes when changing absolution/fixed position #52

Open
ArxSargos opened this issue Apr 28, 2015 · 2 comments
Open

fixedHeader classes when changing absolution/fixed position #52

ArxSargos opened this issue Apr 28, 2015 · 2 comments

Comments

@ArxSargos
Copy link

Hi,
recently I was experiencing problem, that elements in table th and fixedHeader th had different height at very specific window width... I think because of browser rounding in table th to align elements to 2 rows, but in fixed headers elements aligned to 3 rows...

The solution was quite simple, that when fixedHeader is in absolute position is not needed to be visible... https://datatables.net/forums/discussion/15516/display-only-when-table-header-is-not-visible-top-header

But i dont want to modify the fixedHeader file and overriding whole function because of 3 rows of code is quite an overhead.

But it would be really great I think, if fixedheader will toggle some class to differentiate it is with fixed or absolute position. That will allows to hide fixedHeader, if he is absolutely positioned..

My question is, if is planned some posiibility to for example parametrize fixed header to use class, or be hidden when is absolutely positioned?

@DataTables
Copy link
Collaborator

Problem with hiding it is that there are some cases I know of where the FixedHeader is user for filtering input controls...

However, I think that is a nightmare in terms of coding (hence all the copyOnDraw etc you will be able to see in the code) and I'm very tempted to drop that from FixedHeader in the next major update and implement your suggestion to only display the FixedHeader when it is floating over the table.

That is unfortunately likely to be a few months away at best though, due to other on going work. In the mean time, a little hack in the code to set a class is probably your best bet.

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

No branches or pull requests

2 participants
@ArxSargos and others