-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Add Classes to Column Header to optimize the Responsive behavior #988
Comments
Good idea - will see what I can do. |
…s to the labels in the grid. Fix adding classes property to the invisible firstrow. See #988
Hello, First of of all Thank you very much for the participation. Unfortunately I will not accept it since:
Please see my commit - it add a property labelClasses, which should do the job. Please let me know if this resolves your problem. Thank again. Best Regards, |
Hi Tony,
thanks for your reply.
It is needed to make the changes only on the separate module and not and the generated file
I don't know this.. sorry.
You add class classesResponsive to the data cells, but this is not needed - we already have classes property
I know that their are this properties. But I added it for more functionality. e.g. you want to style the header in a fancy way(now you may see the first row depending on the style.
But your solution do the job! thanks for the really fast changes.
Since I try the Version 5.5.5 instead of 5.5.0 the height of the grid will differently calculated if the option $.jgrid.defaults.responsive is true. The Property "height" doesn't seems to work like it do in the version before.
Let me know if you need some support on the PHP implementation of the grid.
Best Regards
Mario
Gesendet: Montag, 31. Mai 2021 um 14:56 Uhr
Von: "Tony Tomov" ***@***.***>
An: "tonytomov/jqGrid" ***@***.***>
Cc: "atheosangel" ***@***.***>, "Author" ***@***.***>
Betreff: Re: [tonytomov/jqGrid] Add Classes to Column Header to optimize the Responsive behavior (#988)
Hello,
First of of all Thank you very much for the participation. Unfortunately I will not accept it since:
It is needed to make the changes only on the separate module and not and the generated file
You add class classesResponsive to the data cells, but this is not needed - we already have classes property
Please see my commit - it add a property labelClasses, which should do the job.
I have fixed adding the classes to the first invisible row.
Please let me know if this resolves your problem.
Thank again.
Best Regards,
Tony
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
|
Hello, Can you please open another issue and explain in detail the problem. Kind Regards, |
It will be very nice to be able to add a custom CSS class through the colModel to the colum header like to the rows to optimize the Responsive behavior of the grid.
For Example. If we add the class "hidden-xs" then i also want be able to add this or another class to the header. Please also add the Class to the Toolbarfilter or add an separate class for this.
/Responsive Design/
@media (max-width: 767.98px) {
.hidden-xs {
display:none !important;
}
}
The text was updated successfully, but these errors were encountered: