Changeset 1290

Timestamp:

09/10/07 16:09:35 (1 year ago)

Author:

benoitg

Message:

• Fix #370: Dropdown Javascript menus didn't retract in Internet Explorer. My thanks to Robin Jones for at least proposing a solution to the problem, even if it didn't work in this case. The problem turned out to be a wrong regex in the IE workaround javascript.
  

Files:

• trunk/wifidog-auth/CHANGELOG (modified) (1 diff)
• trunk/wifidog-auth/wifidog/classes/MainUI.php (modified) (1 diff)
• trunk/wifidog-auth/wifidog/classes/Menu.php (modified) (3 diffs)
• trunk/wifidog-auth/wifidog/media/base_theme/stylesheet.css (modified) (6 diffs)

trunk/wifidog-auth/CHANGELOG

@@ -1 +1 @@
-# $Id$
+2007-09-10 Benoit Grégoire  <bock@step.polymtl.ca>
+        * Fix #370:  Dropdown Javascript menus didn't retract in Internet Explorer.  My thanks to Robin Jones for at least proposing a solution to the problem, even if it didn't work in this case.  The problem turned out to be a wrong regex in the IE workaround javascript.
+        
-2007-09-09 Benoit Grégoire  <bock@step.polymtl.ca>
-        * Early OpenID support.  The auth server can now be used as an identity provider, with your profile page as the identity URL

trunk/wifidog-auth/wifidog/classes/MainUI.php

@@ -559 +559 @@
-            $menu = Menu::getObject();
-            $this->addContent('main_area_top', $menu->getUserUI());
+                        $this->appendHtmlHeadContent(Menu::getIEWorkaroundJS());
-        }
-

trunk/wifidog-auth/wifidog/classes/Menu.php

@@ -200 +200 @@
-    }
-
-
-
-
-
-
-
+
+
+
+
+
+
+
-        <script type="text/javascript"><!--//--><![CDATA[//><!--
-
@@ -215 +216 @@
-                }
-                sfEls[i].onmouseout=function() {
-
+
+
+
-                }
-        }
@@ -223 +226 @@
-//--><!]]></script>
-EOT;
+return $html;
+    }
+    public function getUserUI()
+    {
+        $this->initMenu();
+        $html = '';
+        //Deal with internet explorer's baindeadness.  From http://www.htmldog.com/articles/suckerfish/dropdowns/example/vertical.html
+
-$html .= "<ul id='nav'>\n";
-        $userData=null;

trunk/wifidog-auth/wifidog/media/base_theme/stylesheet.css

@@ -645 +645 @@
-}
-
-
-
+
+
+
+
+
+
+
+
+
+
-        float: left;
-        width: auto;
-        list-style: none;
-        line-height: 1;
-        background: white;
-        font-weight: bold;
-        padding: 0;
-        border: solid #d7d5d5;
-        border-width: 1px 0;
-        margin: 0 0 1em 0;
-        z-index: 99;
-}
-
-#nav {
+        /* WD specific, this must be after the #nav,#nav ul, it will apply only to the top menu items */
-        background: none;
-        margin: 0 0 0 0;
@@ -669 +674 @@
-}
-
-
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
-        border-left: 1px solid #d7d5d5;
-        border-right: 1px solid #d7d5d5;
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
+
+
+
+
+
+
+
+
+
+
+
+
+
+
-        position: absolute;
+        width: 14.4em;
-        left: -999em;
+        /* using left instead of display to hide menus because display: none isn't read by screen readers */
+        /* WD specific */
-        height: auto;
-        width: 14.4em;
-        w\idth: 13.9em;
-        font-weight: normal;
@@ -705 +718 @@
-}
-
-
-
-
-
-
-
-
-
-
-
-
+
+
+
+
+
-        margin: -1.75em 0 0 14em;
-}
@@ -725 +732 @@
-
-#nav li:hover ul,#nav li li:hover ul,#nav li li li:hover ul,#nav li.sfhover ul,#nav li li.sfhover ul,#nav li li li.sfhover ul
-
+ /* lists nested under hovered list items */
-        left: auto;
-}
-
-
+ /* WD specific */
-        background: #f5f5f5;
-}
@@ -800 +807 @@
-        z-index: 100;
-        opacity: 0.90;
-
+  
-        -webkit-border-radius: 5px;
-        -moz-border-radius: 5px;
@@ -1370 +1377 @@
-
-\html
+
- 
+
-div
+
+
-.tableContainer
+
- 
+
-table
-
+
+
-{
-margin
+
+
-:
- 
-0
--16
-px
+
- 
+
-0
-0
-}
-\html
+
- 
+
-div
+
+
-.tableContainer
-
+
+
-{
-padding
+
+
-:
- 
-0
-16
-px
+
- 
+
-0
-0;