Safari 4.0 beta + supports native window button. So there is no need to keep CSS hacks for buttons

zhourenjian · zhourenjian · commit 628d354521ec · 2009-03-01T17:10:00.000Z
diff --git a/sources/net.sf.j2s.java.org.eclipse.swt/src/org/eclipse/swt/widgets/Button.css b/sources/net.sf.j2s.java.org.eclipse.swt/src/org/eclipse/swt/widgets/Button.css
@@ -44,20 +44,18 @@
 	font-size:1em;
 }
 
-@media all and (-webkit-min-device-pixel-ratio:0){/* safari only */
-.button-push, .button-toggle {
+.button-push-safari, .button-toggle-safari {
 	height:auto !important;
 	top:-0.9em;
 	margin-top:50%;
 	padding:0.25em 0 0.35em 0;
 }
-.button-arrow {
+.button-arrow-safari {
 	height:auto !important;
 	top:-0.7em;
 	margin-top:50%;
 	padding:0.25em 0 0.35em 0;
 }
-}
 
 body:nth-of-type(1) .button-push-chrome, 
 body:nth-of-type(1) .button-toggle-chrome, 
diff --git a/sources/net.sf.j2s.java.org.eclipse.swt/src/org/eclipse/swt/widgets/Button.java b/sources/net.sf.j2s.java.org.eclipse.swt/src/org/eclipse/swt/widgets/Button.java
@@ -492,6 +492,21 @@ void createHandle() {
 		}
 		if (OS.isChrome) {
 			btnHandle.className += " " + btnHandle.className + "-chrome"; 
+		} else if (OS.isSafari) {
+			boolean isSafari4Plus = false;
+			/**
+			 * @j2sNative
+			 * var ua = navigator.userAgent;
+			 * var verIdx = ua.indexOf ("Version/");
+			 * if (verIdx  != -1) {
+			 * 	var verStr = ua.substring (verIdx + 8);
+			 * 	var verNumber = parseFloat (verStr);
+			 * 	isSafari4Plus = verNumber >= 4.0
+			 * }
+			 */ {}
+			if (!isSafari4Plus) { // There is no need of CSS hack for Safari 4.0+
+				btnHandle.className += " " + btnHandle.className + "-safari"; 
+			}
 		}
 	}
 	btnHandle.onmouseover = new RunnableCompatibility() {
