onSelect and onSelectAll callbacks parameters

[gabrycaos]

Callbacks for onSelect and onSelectAll accept similar parameters, but "reversed":

from the storyBook

handleOnSelect = (row, isSelect)

and

handleOnSelectAll = (isSelect, rows)

swapping parameters in one of these functions (e.g handleOnSelectAll = (rows, isSelect)) can be easily for developers to manage both cases, maybe with a single function, depending on use case. I know that row is an object and rows is an array, but they can be managed with a single function in a simple scenario

[AllenFang]

@gabrycaos I'm sorry that i didn't notice this mistake, but if I swap this parameter, I should do a major version bump. So I will consider to fix this issue in next time major release.

[gabrycaos]

i can work on a pull request for that

[AllenFang]

sure, it's welcome, but like I mention, this patch will be major release, I will choose a good timing to merge your PR, which mean your PR will not be merge in near future. because we just have a new major bump last week

[juancafe4]

I passed a onSelect from a component to another as a prop where is using the table. But it doesn't change the color of the row when I called setState in the callback function.

[AllenFang]

@juancafe4 what react-bootstrap-table-next version you use now?

[AllenFang]

@juancafe4 your issue should be fixed in last year #378