Port the MultiSelect API from FW8

[gilberto-torrezan]

This change is 

[ZheSun88]

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, 8 unresolved discussions.

---

a discussion (no related file):
Do you want to remove all the redundant public? not a blocker

---

flow-data/src/main/java/com/vaadin/flow/data/selection/MultiSelect.java, line 151 at r1 (raw file):

    }

    @Override

just curious, in the FW8 file, there are no these two overrides. how do they deal with it? why do we need it?

---

Comments from Reviewable

[ZheSun88]

Review status: all files reviewed at latest revision, 8 unresolved discussions.

---

flow-data/src/main/java/com/vaadin/flow/data/selection/MultiSelect.java, line 151 at r1 (raw file):

Previously, ZheSun88 (Sun Zhe) wrote…

just curious, in the FW8 file, there are no these two overrides. how do they deal with it? why do we need it?

oh, I see, those changes come from Grid. As Grid is the only component to support MultiSelection so far, it is okay.

---

Comments from Reviewable

[gilberto-torrezan]

Review status: 0 of 1 files reviewed at latest revision, 8 unresolved discussions.

---

a discussion (no related file):

Previously, ZheSun88 (Sun Zhe) wrote…

Do you want to remove all the redundant public? not a blocker

Done

---

Comments from Reviewable

[ZheSun88]

---

Reviewed 1 of 1 files at r2.
Review status: all files reviewed at latest revision, 8 unresolved discussions.

---

Comments from Reviewable