Griddle icon indicating copy to clipboard operation
Griddle copied to clipboard

useFixedHeader - columns are misaligned

Open cosminnicula opened this issue 10 years ago • 8 comments

When you use useFixedHeader=true, the table columns are misaligned with the header columns, just like in the image below.

1

cosminnicula avatar Apr 27 '15 16:04 cosminnicula

My columns are even more broken than this. I would suggest that the plugin move over to using

and for sizing the columns on both tables.

aphillipo avatar Jun 05 '15 12:06 aphillipo

I fixed this in my branch. It was that the TR used to space out the table did not contain any TDs - i've adapted the griddle source to support this in my branch on github.

It's not intended for pull requests as Griddle's authors seem to have abandoned this project in favour of a rewrite.

aphillipo avatar Jul 29 '15 14:07 aphillipo

@aphillipo I'm sorry if I miscommunicated the state of this project. It is true that we are working to break apart the functionality into other modules (so rendering isn't so tied to grid logic, etc). These modules will be used in the main griddle project (this will allow for other enhancements that we'll be documenting a little later on). That all said we are not abandoning this project.

We are being a bit cautious about approving larger PRs because we are treating a lot of the open issues as acceptance criteria for the new stuff. What we are trying to avoid is having someone spend a lot of effort implementing a feature only to have it replaced a little later on.

This sounds like it would be a good candidate for a PR if you already have the work done.

ryanlanciaux avatar Jul 30 '15 03:07 ryanlanciaux

Sorry for being so rude! I know you haven't abandoned it but I didn't think you were taking pull requests.

The code I have semi-duplicates the generation of a row code so while it fixes this bug it definitely is not perfect. I have no time to pull apart the row generation to avoid this duplication.

I would send a pull request but I've made so many other small changes I doubt either of us would want it merged.

Seeing as you have put in so much effort on Griddle I'll try my very best to find time to get only the things that fix this bug into a pull request.

aphillipo avatar Jul 31 '15 10:07 aphillipo

Oh you weren't being rude at all. You bring up a totally valid concern -- I just wanted to clarify because I realize I may have been less than clear in the gitter chat when we talked on this.

I think this weekend we're going to start opening issues on the griddle-core / griddle-render projects and closing the ones here that we're working in those projects (that will make their way back here -- it's confusing, I know). That way we'll have a more accurate view of what is being worked on and what needs attention.

ryanlanciaux avatar Jul 31 '15 16:07 ryanlanciaux

So when will I be able to replace old griddle with the new work you guys have been doing? Be nice to have a rough guess of when I can start testing it...

aphillipo avatar Jul 31 '15 17:07 aphillipo

I have the same problem.

Shachar16290 avatar Oct 27 '15 13:10 Shachar16290

image

Seems like there are 2

generated is you put useFixedHeader to true. Which cause the miss alignement, since on of them has no data just the header.

Are there any progress on this?

ThePainnn avatar Apr 04 '16 19:04 ThePainnn