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

Add Classes to Column Header to optimize the Responsive behavior #988

Open
atheosangel opened this issue May 28, 2021 · 4 comments
Open

Comments

@atheosangel
Copy link

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;
}
}

@tonytomov
Copy link
Owner

tonytomov commented May 28, 2021

Good idea - will see what I can do.

tonytomov added a commit that referenced this issue May 31, 2021
…s to the labels in the grid. Fix adding classes property to the invisible firstrow. See #988
@tonytomov
Copy link
Owner

Hello,

First of of all Thank you very much for the participation. Unfortunately I will not accept it since:

  1. It is needed to make the changes only on the separate module and not and the generated file
  2. 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

@atheosangel
Copy link
Author

atheosangel commented May 31, 2021 via email

@tonytomov
Copy link
Owner

tonytomov commented Jun 1, 2021

Hello,

Can you please open another issue and explain in detail the problem.
If there is no problem with this one you may want to close it.

Kind Regards,
Tony

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants