From 3e8c87939b2ec7fd25f96bfacfbdf915a14b2344 Mon Sep 17 00:00:00 2001 From: Olli Tietäväinen Date: Mon, 12 Feb 2018 15:14:49 +0200 Subject: Implement focus handing in RadioButtonGroup and CheckboxGroup (#10440) Fixes #10429 * implement focus handing in RadioButtonGroup, fixes #10429 * Merge branch 'master' of https://github.com/vaadin/framework into radiobuttongroup-focus * merge * Merge branch 'master' of https://github.com/vaadin/framework into radiobuttongroup-focus * fix initial focus handling also on CheckBoxGroup and add tests * add license headers * Merge branch 'master' of https://github.com/vaadin/framework into radiobuttongroup-focus * changed client to use lambdas and refactored focus testing to parent class * made FocusTest abstract * Merge branch 'master' of https://github.com/vaadin/framework into radiobuttongroup-focus * don't allow focusing on disabled items & refactor focusing first item --- .../checkboxgroup/CheckBoxGroupFocus.java | 40 ++++++++++++++++++ .../radiobuttongroup/RadioButtonGroupFocus.java | 44 ++++++++++++++++++++ .../com/vaadin/tests/components/FocusTest.java | 47 ++++++++++++++++++++++ .../checkboxgroup/CheckBoxGroupFocusTest.java | 45 +++++++++++++++++++++ .../RadioButtonGroupFocusTest.java | 45 +++++++++++++++++++++ 5 files changed, 221 insertions(+) create mode 100755 uitest/src/main/java/com/vaadin/tests/components/checkboxgroup/CheckBoxGroupFocus.java create mode 100755 uitest/src/main/java/com/vaadin/tests/components/radiobuttongroup/RadioButtonGroupFocus.java create mode 100755 uitest/src/test/java/com/vaadin/tests/components/FocusTest.java create mode 100755 uitest/src/test/java/com/vaadin/tests/components/checkboxgroup/CheckBoxGroupFocusTest.java create mode 100755 uitest/src/test/java/com/vaadin/tests/components/radiobuttongroup/RadioButtonGroupFocusTest.java (limited to 'uitest') diff --git a/uitest/src/main/java/com/vaadin/tests/components/checkboxgroup/CheckBoxGroupFocus.java b/uitest/src/main/java/com/vaadin/tests/components/checkboxgroup/CheckBoxGroupFocus.java new file mode 100755 index 0000000000..7d845e96a4 --- /dev/null +++ b/uitest/src/main/java/com/vaadin/tests/components/checkboxgroup/CheckBoxGroupFocus.java @@ -0,0 +1,40 @@ +/* + * Copyright 2000-2016 Vaadin Ltd. + * + * Licensed under the Apache License, Version 2.0 (the "License"); you may not + * use this file except in compliance with the License. You may obtain a copy of + * the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT + * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the + * License for the specific language governing permissions and limitations under + * the License. + */ +package com.vaadin.tests.components.checkboxgroup; + +import com.vaadin.annotations.Widgetset; +import com.vaadin.server.VaadinRequest; +import com.vaadin.tests.components.AbstractTestUIWithLog; +import com.vaadin.ui.Button; +import com.vaadin.ui.CheckBoxGroup; + +@Widgetset("com.vaadin.DefaultWidgetSet") +public class CheckBoxGroupFocus extends AbstractTestUIWithLog { + + @Override + protected void setup(VaadinRequest request) { + CheckBoxGroup cbg = new CheckBoxGroup<>("CheckBoxes"); + cbg.setItems("Test1", "Test2", "Test3"); + cbg.select("Test2"); + cbg.setItemCaptionGenerator(item -> "Option " + item); + cbg.focus(); + CheckBoxGroup cbg2 = new CheckBoxGroup<>("No selection"); + cbg2.setItems("Foo1", "Foo2", "Foo3"); + Button button = new Button("focus second group", e -> cbg2.focus()); + addComponents(cbg, cbg2, button); + } + +} diff --git a/uitest/src/main/java/com/vaadin/tests/components/radiobuttongroup/RadioButtonGroupFocus.java b/uitest/src/main/java/com/vaadin/tests/components/radiobuttongroup/RadioButtonGroupFocus.java new file mode 100755 index 0000000000..4342a32860 --- /dev/null +++ b/uitest/src/main/java/com/vaadin/tests/components/radiobuttongroup/RadioButtonGroupFocus.java @@ -0,0 +1,44 @@ +/* + * Copyright 2000-2016 Vaadin Ltd. + * + * Licensed under the Apache License, Version 2.0 (the "License"); you may not + * use this file except in compliance with the License. You may obtain a copy of + * the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT + * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the + * License for the specific language governing permissions and limitations under + * the License. + */ +package com.vaadin.tests.components.radiobuttongroup; + +import com.vaadin.annotations.Widgetset; +import com.vaadin.server.VaadinRequest; +import com.vaadin.tests.components.AbstractTestUIWithLog; +import com.vaadin.ui.Button; +import com.vaadin.ui.RadioButtonGroup; + +/** + * @author Vaadin Ltd + * + */ +@Widgetset("com.vaadin.DefaultWidgetSet") +public class RadioButtonGroupFocus extends AbstractTestUIWithLog { + + @Override + protected void setup(VaadinRequest request) { + RadioButtonGroup rbg = new RadioButtonGroup<>("Radios"); + rbg.setItems("Test1", "Test2", "Test3"); + rbg.setSelectedItem("Test2"); + rbg.setItemCaptionGenerator(item -> "Option " + item); + rbg.focus(); + RadioButtonGroup rbg2 = new RadioButtonGroup<>("No selection"); + rbg2.setItems("Foo1", "Foo2", "Foo3"); + Button button = new Button("focus second group", e -> rbg2.focus()); + addComponents(rbg, rbg2, button); + } + +} diff --git a/uitest/src/test/java/com/vaadin/tests/components/FocusTest.java b/uitest/src/test/java/com/vaadin/tests/components/FocusTest.java new file mode 100755 index 0000000000..ae7f5301d2 --- /dev/null +++ b/uitest/src/test/java/com/vaadin/tests/components/FocusTest.java @@ -0,0 +1,47 @@ +/* + * Copyright 2000-2016 Vaadin Ltd. + * + * Licensed under the Apache License, Version 2.0 (the "License"); you may not + * use this file except in compliance with the License. You may obtain a copy of + * the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT + * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the + * License for the specific language governing permissions and limitations under + * the License. + */ +package com.vaadin.tests.components; + +import static org.junit.Assert.assertNotNull; +import static org.junit.Assert.assertTrue; + +import java.util.List; + +import org.openqa.selenium.By; +import org.openqa.selenium.WebElement; +import org.openqa.selenium.remote.DesiredCapabilities; + +import com.vaadin.testbench.TestBenchElement; +import com.vaadin.tests.tb3.MultiBrowserTest; + +public abstract class FocusTest extends MultiBrowserTest { + + protected boolean isFocusInsideElement(TestBenchElement element) { + WebElement focused = getFocusedElement(); + assertNotNull(focused); + String id = focused.getAttribute("id"); + assertTrue("Focused element should have a non-empty id", + id != null && !"".equals(id)); + return element.isElementPresent(By.id(id)); + } + + @Override + public List getBrowsersToTest() { + // Focus does not move when expected with Selenium/TB and Firefox 45 + return getBrowsersExcludingFirefox(); + } + +} diff --git a/uitest/src/test/java/com/vaadin/tests/components/checkboxgroup/CheckBoxGroupFocusTest.java b/uitest/src/test/java/com/vaadin/tests/components/checkboxgroup/CheckBoxGroupFocusTest.java new file mode 100755 index 0000000000..4d6c52e22b --- /dev/null +++ b/uitest/src/test/java/com/vaadin/tests/components/checkboxgroup/CheckBoxGroupFocusTest.java @@ -0,0 +1,45 @@ +/* + * Copyright 2000-2016 Vaadin Ltd. + * + * Licensed under the Apache License, Version 2.0 (the "License"); you may not + * use this file except in compliance with the License. You may obtain a copy of + * the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT + * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the + * License for the specific language governing permissions and limitations under + * the License. + */ +package com.vaadin.tests.components.checkboxgroup; + +import static org.junit.Assert.assertTrue; + +import org.junit.Test; + +import com.vaadin.testbench.elements.ButtonElement; +import com.vaadin.testbench.elements.CheckBoxGroupElement; +import com.vaadin.tests.components.FocusTest; + +public class CheckBoxGroupFocusTest extends FocusTest { + + @Test + public void focusOnInit() { + openTestURL(); + CheckBoxGroupElement checkBoxGroup = $(CheckBoxGroupElement.class) + .first(); + assertTrue(isFocusInsideElement(checkBoxGroup)); + } + + @Test + public void moveFocusAfterClick() { + openTestURL(); + $(ButtonElement.class).first().click(); + CheckBoxGroupElement checkBoxGroup = $(CheckBoxGroupElement.class) + .last(); + assertTrue(isFocusInsideElement(checkBoxGroup)); + } + +} diff --git a/uitest/src/test/java/com/vaadin/tests/components/radiobuttongroup/RadioButtonGroupFocusTest.java b/uitest/src/test/java/com/vaadin/tests/components/radiobuttongroup/RadioButtonGroupFocusTest.java new file mode 100755 index 0000000000..37916abf90 --- /dev/null +++ b/uitest/src/test/java/com/vaadin/tests/components/radiobuttongroup/RadioButtonGroupFocusTest.java @@ -0,0 +1,45 @@ +/* + * Copyright 2000-2016 Vaadin Ltd. + * + * Licensed under the Apache License, Version 2.0 (the "License"); you may not + * use this file except in compliance with the License. You may obtain a copy of + * the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT + * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the + * License for the specific language governing permissions and limitations under + * the License. + */ +package com.vaadin.tests.components.radiobuttongroup; + +import static org.junit.Assert.assertTrue; + +import org.junit.Test; + +import com.vaadin.testbench.elements.ButtonElement; +import com.vaadin.testbench.elements.RadioButtonGroupElement; +import com.vaadin.tests.components.FocusTest; + +public class RadioButtonGroupFocusTest extends FocusTest { + + @Test + public void focusOnInit() { + openTestURL(); + RadioButtonGroupElement radioButtonGroup = $( + RadioButtonGroupElement.class).first(); + assertTrue(isFocusInsideElement(radioButtonGroup)); + } + + @Test + public void moveFocusAfterClick() { + openTestURL(); + $(ButtonElement.class).first().click(); + RadioButtonGroupElement radioButtonGroup2 = $( + RadioButtonGroupElement.class).last(); + assertTrue(isFocusInsideElement(radioButtonGroup2)); + } + +} -- cgit v1.2.3