Open pmq opened 9 years ago
It's not the best solution, if there are a small number of columns then the select column would be huge for no reason. Maybe a "1%" or "2%" value would be better, but still not good enough... It's not a problem when you know the number of columns and you can change the value to what works for you, but when the no of columns change dynamically...
Hi @vladud, you're right it's not an exact fix, but I thought it would be a saner default value. I think the percentage one is even more error-prone.
In that case, maybe we could change ngReactGridCheckbox with a syntax like this : new ngReactGridCheckbox($scope.expensesSelected, { batchToggle: true, width: '20' })
to let everyone override the width without having to subclass? Or maybe it's already there and I'm misreading the source; how do you set it?
No, I don't think you can set it that way, it would be ok to export the setting.. Personally I preferred to implement my own selection without a checkbox column, but it's a quick and dirty implementation .. I've only used the library for a few tests so far
@pmq @vladud so what do you guys think we should do to deal with this?
I'm mentioning @ewu02 also, who can probably provide more informed feedback. From my point of view, we should :
I can provide a patch if we agree on that, and probably also provide an example for documentation.
Yeah, that seems ok. Maybe you could also change the default "10%" cell width to 100% / noOfColumns ... I guess that would provide the perfect fit. (no of column except select column) Actually changing it to "100%" would work the same way :/
Just recently encountered the same issue, which seems to show up when there are many columns. I would favor having an override option with % over table width (or noOfColumns), with the default being 10% of table width as well.
Created pull request https://github.com/josebalius/ngReactGrid/pull/60 that should solve this issue.
Like https://github.com/josebalius/ngReactGrid/pull/33, not sure if it is only on my browser, but the default checkbox column width is too small. Feel free to close abruptly if I'm the only one :)
Tested using a grid setup like this :
Before :
After :