From 0652f2abb8bcdbfa76ec135025284115bcb71e2f Mon Sep 17 00:00:00 2001 From: Anna Miroshnik Date: Thu, 30 Oct 2014 14:30:48 +0300 Subject: [PATCH] Scrolling down submenus via scrollbar (MenuBar.isAutoOpen()) (#10456) full name of defect: Can't scroll down submenus using the scrollbar when MenuBar.isAutoOpen() == true Tests. Change-Id: I2cbe797c96dc6a780df147e165babb23f7fbdef3 --- .../java/com/vaadin/client/ui/VMenuBar.java | 18 ++++- .../menubar/MenuBarIsAutoOpenScrolling.java | 50 +++++++++++++ .../MenuBarIsAutoOpenScrollingTest.java | 72 +++++++++++++++++++ 3 files changed, 139 insertions(+), 1 deletion(-) create mode 100644 uitest/src/main/java/com/vaadin/tests/components/menubar/MenuBarIsAutoOpenScrolling.java create mode 100644 uitest/src/test/java/com/vaadin/tests/components/menubar/MenuBarIsAutoOpenScrollingTest.java diff --git a/client/src/main/java/com/vaadin/client/ui/VMenuBar.java b/client/src/main/java/com/vaadin/client/ui/VMenuBar.java index d08301ec93..2f1b93994b 100644 --- a/client/src/main/java/com/vaadin/client/ui/VMenuBar.java +++ b/client/src/main/java/com/vaadin/client/ui/VMenuBar.java @@ -33,6 +33,10 @@ import com.google.gwt.event.dom.client.KeyDownEvent; import com.google.gwt.event.dom.client.KeyDownHandler; import com.google.gwt.event.dom.client.KeyPressEvent; import com.google.gwt.event.dom.client.KeyPressHandler; +import com.google.gwt.event.dom.client.MouseOutEvent; +import com.google.gwt.event.dom.client.MouseOutHandler; +import com.google.gwt.event.dom.client.MouseOverEvent; +import com.google.gwt.event.dom.client.MouseOverHandler; import com.google.gwt.event.logical.shared.CloseEvent; import com.google.gwt.event.logical.shared.CloseHandler; import com.google.gwt.user.client.Command; @@ -54,7 +58,7 @@ import com.vaadin.shared.ui.menubar.MenuBarConstants; public class VMenuBar extends SimpleFocusablePanel implements CloseHandler, KeyPressHandler, KeyDownHandler, - FocusHandler, SubPartAware { + FocusHandler, SubPartAware, MouseOutHandler, MouseOverHandler { // The hierarchy of VMenuBar is a bit weird as VMenuBar is the Paintable, // used for the root menu but also used for the sub menus. @@ -605,6 +609,8 @@ public class VMenuBar extends SimpleFocusablePanel popup.setWidget(item.getSubMenu()); popup.addCloseHandler(this); popup.addAutoHidePartner(item.getElement()); + popup.addDomHandler(this, MouseOutEvent.getType()); + popup.addDomHandler(this, MouseOverEvent.getType()); // at 0,0 because otherwise IE7 add extra scrollbars (#5547) popup.setPopupPosition(0, 0); @@ -1718,4 +1724,14 @@ public class VMenuBar extends SimpleFocusablePanel public CustomMenuItem getMenuItemWithElement(Element element) { return getMenuItemWithElement(DOM.asOld(element)); } + + @Override + public void onMouseOver(MouseOverEvent event) { + LazyCloser.cancelClosing(); + } + + @Override + public void onMouseOut(MouseOutEvent event) { + LazyCloser.schedule(); + } } diff --git a/uitest/src/main/java/com/vaadin/tests/components/menubar/MenuBarIsAutoOpenScrolling.java b/uitest/src/main/java/com/vaadin/tests/components/menubar/MenuBarIsAutoOpenScrolling.java new file mode 100644 index 0000000000..d3cdacab17 --- /dev/null +++ b/uitest/src/main/java/com/vaadin/tests/components/menubar/MenuBarIsAutoOpenScrolling.java @@ -0,0 +1,50 @@ +/* + * Copyright 2000-2014 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.menubar; + +import com.vaadin.server.VaadinRequest; +import com.vaadin.tests.components.AbstractTestUI; +import com.vaadin.ui.MenuBar; +import com.vaadin.ui.MenuBar.MenuItem; + +public class MenuBarIsAutoOpenScrolling extends AbstractTestUI { + + @Override + protected void setup(VaadinRequest request) { + MenuBar menuBar = new MenuBar(); + menuBar.setAutoOpen(true); + menuBar.addStyleName("menu-bar"); + + MenuItem item = menuBar.addItem("Item", null); + + for (int i = 1; i < 100; i++) { + item.addItem("SubItem" + i, null); + } + + addComponent(menuBar); + } + + @Override + protected String getTestDescription() { + return "SubMenu (when MenuBar.isAutoOpen()) should not dissapear after a couple of seconds when the mouse pointer is over the scrollbar."; + } + + @Override + protected Integer getTicketNumber() { + return 10456; + } + +} diff --git a/uitest/src/test/java/com/vaadin/tests/components/menubar/MenuBarIsAutoOpenScrollingTest.java b/uitest/src/test/java/com/vaadin/tests/components/menubar/MenuBarIsAutoOpenScrollingTest.java new file mode 100644 index 0000000000..51fca00eda --- /dev/null +++ b/uitest/src/test/java/com/vaadin/tests/components/menubar/MenuBarIsAutoOpenScrollingTest.java @@ -0,0 +1,72 @@ +/* + * Copyright 2000-2014 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.menubar; + +import java.util.List; + +import org.junit.Test; +import org.openqa.selenium.By; +import org.openqa.selenium.WebElement; +import org.openqa.selenium.interactions.Actions; +import org.openqa.selenium.remote.DesiredCapabilities; +import org.openqa.selenium.support.ui.ExpectedConditions; + +import com.vaadin.testbench.elements.MenuBarElement; +import com.vaadin.testbench.parallel.Browser; +import com.vaadin.tests.tb3.MultiBrowserTest; + +public class MenuBarIsAutoOpenScrollingTest extends MultiBrowserTest { + + @Override + public List getBrowsersToTest() { + List list = super.getBrowsersToTest(); + // test is unreliable on Firefox + list.remove(Browser.FIREFOX.getDesiredCapabilities()); + return list; + } + + @SuppressWarnings("deprecation") + @Test + public void testIsAutoOpenSubmenuScrolling() { + openTestURL(); + Actions actions = new Actions(driver); + + MenuBarElement menu = $(MenuBarElement.class).get(0); + + actions.moveToElement(menu).perform(); + + waitForElementPresent(By.className("v-menubar-popup")); + + WebElement subMenuPopup = driver + .findElement(By.className("v-menubar-popup")); + + // here we have to use pause() because LazyCloser in VMenuBar auto + // closes submenus popup in 750 ms. + actions.moveToElement(subMenuPopup, subMenuPopup.getSize().width - 7, + 100).clickAndHold().pause(1000).moveByOffset(0, 200).release() + .perform(); + + // subMenuPopup should still be presented + waitUntil(ExpectedConditions + .visibilityOfElementLocated(By.className("v-menubar-popup"))); + + actions.moveByOffset(100, 0).perform(); + // subMenuPopup should disappear + waitUntil(ExpectedConditions + .not(ExpectedConditions.invisibilityOfElementLocated( + By.className("v-menubar-popup")))); + } +} -- 2.39.5