Always callback setSelected.

[exabugs]

I think this check is not necessary.

    if (index === this.props.selectedIndex) return;

Because we can't handle re-selected event.

If you want to do nothing when same tab is selected, you can check it at user-side.

    onSelect(index, lastIndex, event) {
        if (index === lastIndex) return;

[danez]

I see, that makes sense. I will test it.

This is a breaking change though.

[SaifAsad]

Hi @danez, any updates on the status of this pull request ? really need this feature to be implemented, can you please approve it ?

I am trying to implement closing tab functionality in my project and this check if (index === this.props.selectedIndex) return; is breaking my code,

Many thanks!!

[danez]

Thank you very much