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

Enable Frozen Columns with Column Grouping #429

Merged
merged 2 commits into from
Sep 2, 2019
Merged

Conversation

gavingallagher
Copy link
Contributor

@gavingallagher gavingallagher commented Aug 30, 2019

Created the preheader panel, scroller and spacer elements for the right side panel, and bound scrollLeft to the right panel when frozen columns are set.

Exposed the right preheader panel via public getter for constructing the column group headers.

Added example of usage.

Fixes #340

Created the preheader panel, scroller and spacer elements for the right side panel, and bound scrollLeft to the right panel when frozen columns are set.

Exposed the right preheader panel via public getter.

Added example of usage.
slick.grid.js Outdated Show resolved Hide resolved
Also renamed the function to get the right preheader panel to be more descriptive.

Updated the example to use correct link.
@ghiscoding
Copy link
Collaborator

Great, looks good to me with the changes 👍

At first I thought this PR was to fix the Row Grouping but it's actually Column (Header/Titles) Grouping. I wonder if that could help in fixing that other Row Grouping bug though 😄

@gavingallagher
Copy link
Contributor Author

Ah, yeah, my bad. Forgot to link the issue when I created the PR.

@6pac 6pac merged commit 1c4da6b into 6pac:master Sep 2, 2019
@ghiscoding
Copy link
Collaborator

@gavingallagher
Thanks again for the fix, I added your new Example for the Wiki list of examples as well, the very last one in this case. Really glad you fixed this one. 😃

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.

Column grouping headers does not work with frozen columns
3 participants