Add API for freezing grid columns

[ahie]

This change is 

[gilberto-torrezan]

Reviewed 3 of 3 files at r1.
Review status: all files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.

---

flow-components-parent/demo-flow-components/src/test/java/com/vaadin/flow/demo/views/GridViewIT.java, line 221 at r1 (raw file):

                grid.getAttribute("columnReorderingAllowed"));

        String idColumnFrozenStatusScript = "return arguments[0].shadowRoot.querySelectorAll('th')[0].getAttribute('frozen')";

This can break if the internals of the component changes over time. One way of testing this is just to evaluate the frozen property of the vaadin-grid-column element, outside the shadow root. That's public API, so it shouldn't change unless a major version is released.

---

Comments from Reviewable

[ahie]

Review status: all files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.

---

flow-components-parent/demo-flow-components/src/test/java/com/vaadin/flow/demo/views/GridViewIT.java, line 221 at r1 (raw file):

Previously, gilberto-torrezan (Gilberto Torrezan) wrote…

This can break if the internals of the component changes over time. One way of testing this is just to evaluate the frozen property of the vaadin-grid-column element, outside the shadow root. That's public API, so it shouldn't change unless a major version is released.

Thing is, it is not actually applied to the vaadin-grid-column element.

---

Comments from Reviewable

[ahie]

Review status: all files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.

---

flow-components-parent/demo-flow-components/src/test/java/com/vaadin/flow/demo/views/GridViewIT.java, line 221 at r1 (raw file):

Previously, ahie (Aleksi Hietanen) wrote…

Thing is, it is not actually applied to the vaadin-grid-column element.

At least selenium didn't correctly pick it up from the column element.

---

Comments from Reviewable

[ahie]

Review status: 0 of 3 files reviewed at latest revision, 1 unresolved discussion.

---

flow-components-parent/demo-flow-components/src/test/java/com/vaadin/flow/demo/views/GridViewIT.java, line 221 at r1 (raw file):

Previously, ahie (Aleksi Hietanen) wrote…

At least selenium didn't correctly pick it up from the column element.

Improved the test.

---

Comments from Reviewable

[gilberto-torrezan]

Reviewed 3 of 3 files at r2.
Review status: all files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.

---

Comments from Reviewable