opendev/gerrit
Code Issues Proposed changes

Fix LinkMenuItem blur on older browsers

Browse Source 
We can't use FocusImpl because it assumes the blur should be on the
first child of the anchor element, and the anchor has no child.

Signed-off-by: Shawn O. Pearce <sop@google.com>
This commit is contained in:
Shawn O. Pearce
2009-05-16 10:18:35 -07:00
parent 8192a7dbb8
commit 7ce27a1aaf
1 changed files with 2 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
src/main/java/com/google/gerrit/client/ui/LinkMenuItem.java
View File

@@ -15,17 +15,16 @@
package com.google.gerrit.client.ui;
import com.google.gwt.core.client.GWT;
import com.google.gwt.dom.client.AnchorElement;
import com.google.gwt.user.client.DOM;
import com.google.gwt.user.client.Element;
import com.google.gwt.user.client.Event;
import com.google.gwt.user.client.ui.Accessibility;
import com.google.gwt.user.client.ui.Hyperlink;
import com.google.gwt.user.client.ui.impl.FocusImpl;
import com.google.gwt.user.client.ui.impl.HyperlinkImpl;
public class LinkMenuItem extends Hyperlink {
private static HyperlinkImpl impl = GWT.create(HyperlinkImpl.class);
private static FocusImpl implPanel = GWT.create(FocusImpl.class);
public LinkMenuItem(final String text, final String targetHistoryToken) {
super((Element) null);
@@ -39,7 +38,7 @@ public class LinkMenuItem extends Hyperlink {
public void onBrowserEvent(Event event) {
super.onBrowserEvent(event);
if (DOM.eventGetType(event) == Event.ONCLICK && impl.handleAsClick(event)) {
implPanel.blur(getElement());
AnchorElement.as(getElement()).blur();
}
}
}