Respect client side "hidden" attribute handling setVisible functionality

[denis-anisimov]

Fix for #3334

---

This change is 

[gilberto-torrezan]

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

---

flow-tests/test-root-context/src/main/java/com/vaadin/flow/uitest/ui/template/HiddenTemplateView.java, line 45 at r2 (raw file):

    }

}

New line here

---

Comments from Reviewable

[denis-anisimov]

Review status: 8 of 9 files reviewed at latest revision, 1 unresolved discussion.

---

flow-tests/test-root-context/src/main/java/com/vaadin/flow/uitest/ui/template/HiddenTemplateView.java, line 45 at r2 (raw file):

Previously, gilberto-torrezan (Gilberto Torrezan) wrote…

New line here

Done.

---

Comments from Reviewable

[gilberto-torrezan]

Reviewed 1 of 1 files at r3.
Review status: all files reviewed at latest revision, all discussions resolved.

---

Comments from Reviewable

[vaadin-bot]

SonarQube analysis reported 2 issues

Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

1. SimpleElementBindingStrategy.java#: This file has 817 lines, which is greater than 750 authorized. Split it into smaller files.
2. SimpleElementBindingStrategy.java#L737: Refactor this method to reduce its Cognitive Complexity from 17 to the 15 allowed.