MultiGrid 9.2.1 shifts last header cell with fixedColumns = 0

[aronhelser]

Hi!
With the fix for headers being offset in 9.2.1, it introduced a bug.

It can be seen in the MultiGrid example, by setting fixedColumnCount to 0, and scrolling all the way to the right. The last header item is rendered above the scrollbar, instead of in the last column.

It is also a bit weird to get the extra cellRenderer call with column 0 and row 1 past the declared size of your table. Not sure what the requirements are to solve the original scroll-offset problem, though.

[bvaughn]

Ah! I see what you're describing. Let me push a quick fix. Sorry!

[bvaughn]

Fixed in 9.2.2!

[aronhelser]

Great, thank you! I tried it and it works for me, too.

Related, but not the same issue:
If I have a fixed column, should I be seeing a call to cellRenderer in that column with a row index one past the size of the table?

[bvaughn]

No. MultiGrid renders a placeholder for that cell for you.

[aronhelser]

Yes, I see that's what's supposed to be happening. I'll see if I can track down if something is different in my use-case and open another issue if needed.

…

On Tue, Mar 7, 2017 at 11:21 AM, Brian Vaughn ***@***.***> wrote: No. MultiGrid renders a placeholder <https://github.com/bvaughn/react-virtualized/blob/master/source/MultiGrid/MultiGrid.js#L222-L232> for that cell for you. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#609 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AS_pnazqPgpOyvheV-oC4wszymMXByhTks5rjYPvgaJpZM4MUxRc> .