diff options
author | Artur Signell <artur.signell@itmill.com> | 2012-08-13 10:11:42 +0000 |
---|---|---|
committer | Artur Signell <artur.signell@itmill.com> | 2012-08-13 10:11:42 +0000 |
commit | 856a68cbdb93d8ca8b6b45b604f606bfbb138b28 (patch) | |
tree | e91f63d72faa444167ca7947dc14e7c1f554c0a7 | |
parent | 131c3db7abb86c7518e5826f74bdd31b729de94e (diff) | |
download | vaadin-framework-856a68cbdb93d8ca8b6b45b604f606bfbb138b28.tar.gz vaadin-framework-856a68cbdb93d8ca8b6b45b604f606bfbb138b28.zip |
Renames based on review (#9284)
svn changeset:24156/svn branch:6.8
-rw-r--r-- | src/com/vaadin/terminal/gwt/client/ui/VOverlay.java | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/src/com/vaadin/terminal/gwt/client/ui/VOverlay.java b/src/com/vaadin/terminal/gwt/client/ui/VOverlay.java index 92f2d6540b..6efedda93c 100644 --- a/src/com/vaadin/terminal/gwt/client/ui/VOverlay.java +++ b/src/com/vaadin/terminal/gwt/client/ui/VOverlay.java @@ -190,7 +190,7 @@ public class VOverlay extends PopupPanel implements CloseHandler<PopupPanel> { return shadow != null; } - private void removeShim() { + private void removeShimElement() { if (shimElement != null) { shimElement.removeFromParent(); } @@ -209,7 +209,7 @@ public class VOverlay extends PopupPanel implements CloseHandler<PopupPanel> { return isShadowEnabled() && shadow.getParentElement() != null; } - private boolean isShimAttached() { + private boolean isShimElementAttached() { return shimElement != null && shimElement.hasParentElement(); } @@ -244,7 +244,7 @@ public class VOverlay extends PopupPanel implements CloseHandler<PopupPanel> { } private IFrameElement getShimElement() { - if (shimElement == null && useShimIframe()) { + if (shimElement == null && needsShimElement()) { shimElement = Document.get().createIFrameElement(); // Insert shim iframe before the main overlay element. It does not @@ -349,7 +349,7 @@ public class VOverlay extends PopupPanel implements CloseHandler<PopupPanel> { // Always ensure shadow is removed when the overlay is removed. removeShadowIfPresent(); - removeShim(); + removeShimElement(); } @Override @@ -463,7 +463,7 @@ public class VOverlay extends PopupPanel implements CloseHandler<PopupPanel> { DOM.setStyleAttribute(shadow, "display", progress < 0.9 ? "none" : ""); } - if (useShimIframe()) { + if (needsShimElement()) { updatePositionAndSize((Element) Element.as(getShimElement()), positionAndSize); } @@ -490,9 +490,9 @@ public class VOverlay extends PopupPanel implements CloseHandler<PopupPanel> { RootPanel.get().getElement().insertBefore(shadow, getElement()); sinkShadowEvents(); } - if (useShimIframe() && !isShimAttached()) { + if (needsShimElement() && !isShimElementAttached()) { RootPanel.get().getElement() - .insertBefore(shimElement, getElement()); + .insertBefore(getShimElement(), getElement()); } } @@ -504,7 +504,7 @@ public class VOverlay extends PopupPanel implements CloseHandler<PopupPanel> { * * @return true if a shim iframe should be added, false otherwise */ - protected boolean useShimIframe() { + protected boolean needsShimElement() { BrowserInfo info = BrowserInfo.get(); return info.isIE() && info.isBrowserVersionNewerOrEqual(8, 0); } |