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

Frozen cell contents not appended when > 15 cols #68

Open
wants to merge 2 commits into
base: 2.0-frozenRowsAndColumns
Choose a base branch
from

Conversation

gcko
Copy link

@gcko gcko commented Feb 7, 2014

This fixes an issue where the frozen cells show up blank when there is a large number of columns and the user has scrolled horizontally.

to reproduce:

  1. on one of the example files add ~ 50 columns programatically with perhaps 100 rows (you have to scroll to trigger the virtual rendering).
  2. load the page and scroll horizontally towards the end of the list of columns
  3. scroll vertically. Note that the frozen column cells remain blank

Jared Scott added 2 commits February 7, 2014 14:32
This fixes an issue where the frozen cells show up blank when there is a large number of columns and the user has scrolled horizontally. 

to reproduce: 
  1. on one of the example files add ~ 50 columns programatically with perhaps 100 rows (you have to scroll to trigger the virtual rendering). 
  2. load the page and scroll horizontally towards the end of the list of columns
  3. scroll vertically. Note that the frozen column cells remain blank
@gcko
Copy link
Author

gcko commented Feb 18, 2014

I made a change to one of the example files (you can see it below). It looks as though there are some slight rendering issues still with frozen rows, however it is heavily mitigated by my fix.

http://rawgithub.com/gcko/SlickGrid/patch-2/examples/example-frozen-columns-and-rows.html

@spidergeuse
Copy link

Awesome. I have this issue too. Good you figured it out. I've been looking into it for a while.

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.

4 participants