Changeset 1266 for trunk/web-app

Timestamp:

Dec 15, 2010, 12:39:03 PM (12 years ago)

Author:

robert@…

Message:

Fixed several IE7 bugs. See tickets #231, #232, #234 and #237

Location:

trunk/web-app

Files:

• css/default_style.css (modified) (5 diffs)
• css/wizard.css (modified) (2 diffs)
• js/importer.js (modified) (1 diff)
• js/wizard.js (modified) (1 diff)

trunk/web-app/css/default_style.css

@@ -74 +74 @@
         font-size: 1.2em;
         background: url(../images/default_style/topnav/topnav_bg.gif) repeat-x;
-        z-index: 1000;
+        z-index: 1005; /** Important: keeps submenu's on top of other elements in IE7. See http://www.shawnpreisz.com/css/z-index-internet-explorer-7-ie7#comment-547 */
 }
 
@@ -82 +82 @@
         padding: 0 15px 0 0;
         position: relative; /*--Declare X and Y axis base--*/
+        z-index: 1004; /** Important: keeps submenu's on top of other elements in IE7. See http://www.shawnpreisz.com/css/z-index-internet-explorer-7-ie7#comment-547 */
 }
 
@@ -130 +131 @@
         -webkit-border-bottom-right-radius: 5px;
         border: 1px solid #111;
+        z-index: 1001; /** Important: keeps submenu's on top of other elements in IE7. See http://www.shawnpreisz.com/css/z-index-internet-explorer-7-ie7#comment-547 */
 }
 
@@ -139 +141 @@
         clear: both;
         width: 170px;
+        position:relative;
+        z-index: 1000; /** Important: keeps submenu's on top of other elements in IE7. See http://www.shawnpreisz.com/css/z-index-internet-explorer-7-ie7#comment-547 */
 }
 
@@ -440 +444 @@
 #facebookConnect {
         padding:0px 0px 10px 0px;
-        z-index: -1;
-}
+        position:relative;
+        z-index: 0;
+}

trunk/web-app/css/wizard.css

@@ -242 +242 @@
     -o-transform: rotate(180g);
     cursor: pointer;
+    zoom: 1; /* IE 6 & 7 hack */
+    *display: inline; /* IE 6 & 7 hack */
 }
 
@@ -325 +327 @@
 }
 
-.wizard .table .highlight, .wizard .table .highlight input, .wizard .table .highlight select, .wizard .table .highlight b {
+#wizard div.row:hover, #wizard div.row:hover b{
     background-color: #006DBA;
     color: #fff;
 }
 
-.wizard .table .highlight input, .wizard .table .highlight select {
+#wizard div.row:hover input, #wizard div.row:hover select {
     background-color: #6e99d4;
     color: #fff;
 }
+
 /* END :: TABLE */
 

trunk/web-app/js/importer.js

@@ -217 +217 @@
 // attach subject events
 function attachTableEvents() {
-    $('div#wizard').find('div.row').each(function() {
-        $(this).hover(
-                function() {
-                    $(this).addClass('highlight');
-                },
-                function() {
-                    $(this).removeClass('highlight');
-                }
-        );
-    });
+        // This method handled the background hover color on table rows
+        // Since this broke select boxes in IE7, this is now handled by css
+        // See wizard.css and #234 and #237
 }
 

trunk/web-app/js/wizard.js

@@ -222 +222 @@
 // attach subject events
 function attachTableEvents() {
-    $('div#wizard').find('div.row').each(function() {
-        $(this).hover(
-                function() {
-                    $(this).addClass('highlight');
-                },
-                function() {
-                    $(this).removeClass('highlight');
-                }
-        );
-    });
+        // This method handled the background hover color on table rows
+        // Since this broke select boxes in IE7, this is now handled by css
+        // See wizard.css and #234 and #237
 }