Revision: 8340
Author: [email protected]
Date: Wed Jun 30 10:07:06 2010
Log: tr...@r8339 was merged into this branch
Revert the IE7 layout implementation back to the IE6 implementation to
work around some quirks.
svn merge -c8339 --ignore-ancestry
https://google-web-toolkit.googlecode.com/svn/trunk
http://code.google.com/p/google-web-toolkit/source/detail?r=8340
Modified:
/releases/2.0/branch-info.txt
/releases/2.0/user/src/com/google/gwt/layout/client/LayoutImplIE6.java
/releases/2.0/user/src/com/google/gwt/layout/client/LayoutImplIE8.java
=======================================
--- /releases/2.0/branch-info.txt Mon Jun 28 10:46:22 2010
+++ /releases/2.0/branch-info.txt Wed Jun 30 10:07:06 2010
@@ -1524,3 +1524,7 @@
Fixes List.subList adds.
svn merge -c8308 --ignore-ancestry
https://google-web-toolkit.googlecode.com/svn/trunk
+tr...@r8339 was merged into this branch
+ Revert the IE7 layout implementation back to the IE6 implementation to
work around some quirks.
+ svn merge -c8339 --ignore-ancestry
https://google-web-toolkit.googlecode.com/svn/trunk
+
=======================================
--- /releases/2.0/user/src/com/google/gwt/layout/client/LayoutImplIE6.java
Tue Jan 26 08:30:07 2010
+++ /releases/2.0/user/src/com/google/gwt/layout/client/LayoutImplIE6.java
Wed Jun 30 10:07:06 2010
@@ -38,30 +38,6 @@
*/
class LayoutImplIE6 extends LayoutImplIE8 {
- private static boolean isIE6 = isIE6();
-
- // Stolen and modified from UserAgent.gwt.xml.
- // TODO(jgw): Get rid of this method, and switch to using soft
permutations
- // once they land in trunk.
- private static native boolean isIE6() /*-{
- function makeVersion(result) {
- return (parseInt(result[1]) * 1000) + parseInt(result[2]);
- }
-
- var ua = navigator.userAgent.toLowerCase();
- if (ua.indexOf("msie") != -1) {
- var result = /msie ([0-9]+)\.([0-9]+)/.exec(ua);
- if (result && result.length == 3) {
- var v = makeVersion(result);
- if (v < 7000) {
- return true;
- }
- }
- }
-
- return false;
- }-*/;
-
private static Element createStyleRuler(Element parent) {
DivElement styleRuler = Document.get().createDivElement();
DivElement styleInner = Document.get().createDivElement();
@@ -139,10 +115,6 @@
@Override
public Element attachChild(Element parent, Element child, Element
before) {
- if (!isIE6) {
- return super.attachChild(parent, child, before);
- }
-
DivElement container = Document.get().createDivElement();
container.appendChild(child);
@@ -165,21 +137,11 @@
@Override
public void fillParent(Element elem) {
- if (!isIE6) {
- super.fillParent(elem);
- return;
- }
-
fillParentImpl(elem);
}
@Override
public void finalizeLayout(Element parent) {
- if (!isIE6) {
- super.finalizeLayout(parent);
- return;
- }
-
resizeRelativeToParent(parent);
resizeHandler(parent, true);
}
@@ -187,39 +149,27 @@
@Override
public void initParent(Element parent) {
super.initParent(parent);
-
- if (isIE6) {
- setPropertyElement(parent, "__styleRuler", createStyleRuler(parent));
- }
+ setPropertyElement(parent, "__styleRuler", createStyleRuler(parent));
}
public void layout(Layer layer) {
- if (!isIE6) {
- super.layout(layer);
- return;
- }
-
Element elem = layer.getContainerElement();
setLayer(elem, layer);
}
@Override
public void onAttach(Element parent) {
- if (isIE6) {
- // No need to re-connect layer refs. This will be taken care of
- // automatically in layout().
- initResizeHandler(parent);
- initUnitChangeHandler(parent, relativeRuler);
- }
+ // No need to re-connect layer refs. This will be taken care of
+ // automatically in layout().
+ initResizeHandler(parent);
+ initUnitChangeHandler(parent, relativeRuler);
}
@Override
public void onDetach(Element parent) {
- if (isIE6) {
- removeLayerRefs(parent);
- removeResizeHandler(parent);
- removeUnitChangeHandler(relativeRuler);
- }
+ removeLayerRefs(parent);
+ removeResizeHandler(parent);
+ removeUnitChangeHandler(relativeRuler);
}
private native void fillParentImpl(Element elem) /*-{
=======================================
--- /releases/2.0/user/src/com/google/gwt/layout/client/LayoutImplIE8.java
Tue Nov 10 10:32:15 2009
+++ /releases/2.0/user/src/com/google/gwt/layout/client/LayoutImplIE8.java
Wed Jun 30 10:07:06 2010
@@ -28,6 +28,7 @@
*/
public class LayoutImplIE8 extends LayoutImpl {
+ @Override
public void layout(Layer layer) {
Style style = layer.container.getStyle();
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
