feat(connectToggleRefinement): add support for array values

[Kocal]

Hi 👋

This PR is a proposal for #2182 which add support for multiples values on on/off for the toggle refinement connector.

See #2182 for more information.

Result

It allow us to use the toggle refinement connector with multiple values:

connectToggleRefinement()({
  attribute: 'whatever',
  on: ['my', 'values'],
  off: ['something', 'else']
});

and the toggle refinement widget:

search.addWidget(instantsearch.widgets.toggleRefinement({
  container: '#toggle',
  attribute: 'whatever',
  on: ['my', 'values'],
  off: ['something', 'else']
}));

On the playground, I've used:

  toggleRefinement({
    container: '#toggle',
    attribute: 'brand',
    on: ['Sony', 'Incipio'],
  }),

and you can see it toggle both values.

What do you think?
Thanks!

[codesandbox-ci]

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 444aa37:

Sandbox	Source
InstantSearch.js	Configuration

[Haroenv]

Relevant sandbox: https://codesandbox.io/s/instantsearchjs-mp4zs?file=/src/app.js

[eunjae-lee]

Relevant sandbox: codesandbox.io/s/instantsearchjs-mp4zs?file=/src/app.js

I clicked "branch" and "Metra", so got this url:

https://mp4zs.csb.app/?instant_search%5BrefinementList%5D%5Bbrand%5D%5B0%5D=Sony&instant_search%5BrefinementList%5D%5Bbrand%5D%5B1%5D=Incipio&instant_search%5BrefinementList%5D%5Bbrand%5D%5B2%5D=Metra&instant_search%5Btoggle%5D%5Bbrand%5D=true

but opening the url won't get Metra selected.

[Haroenv]

That's probably since they're on the same attribute, which isn't something we officially support I think. If the widgets are mounted in the other order, the result would be correct. I don't think we can really find a solution for that, except if toggle refinement doesn't remove other refinements in getWidgetSearchParameters

[eunjae-lee]

You're right. I changed the order of the widgets, and the result is correct. Since it's a "toggle" refinement, I guess the current behavior probably makes sense. We can ship it and later get feedback.

[Haroenv]

for me this makes sense as a feature, and the code & tests look sound

[eunjae-lee]

I agree. It looks good to me.

[Kocal]

Thanks!