Bruno Busco wrote:
> FF 3.5.8

It won't work in all IEs, sometimes.

You can't manipulate or search the dom in some IEs until the dom is
loaded.  It may work fine locally, but that's because you have super
fast access to the server.

There are plenty of google searches about this.

If the menu needs to be hidden at page load time, then doing it thru
css is the correct way.

If you want to start using the menu before the page is loaded, then
doing it thru css is the correct approach, and not doing it at all
with javascript.  Of course, in this case, then the sub-menus have to
be children of the parent, so that you can match on the :hover of the
parent, and change the display.

> 
> 2010/2/24 Adam Heath <[email protected]>:
>> [email protected] wrote:
>>> Author: buscob
>>> Date: Wed Feb 24 14:53:40 2010
>>> New Revision: 915812
>>>
>>> URL: http://svn.apache.org/viewvc?rev=915812&view=rev
>>> Log:
>>> A patch from Sascha Rodekamp, "Tomahawk Menu Loading", 
>>> https://issues.apache.org/jira/browse/OFBIZ-3494
>>>
>>> Modified:
>>>     ofbiz/trunk/themes/tomahawk/includes/appbarClose.ftl
>>>     ofbiz/trunk/themes/tomahawk/webapp/tomahawk/js/dropdown.js
>>>
>>> Modified: ofbiz/trunk/themes/tomahawk/includes/appbarClose.ftl
>>> URL: 
>>> http://svn.apache.org/viewvc/ofbiz/trunk/themes/tomahawk/includes/appbarClose.ftl?rev=915812&r1=915811&r2=915812&view=diff
>>> ==============================================================================
>>> --- ofbiz/trunk/themes/tomahawk/includes/appbarClose.ftl (original)
>>> +++ ofbiz/trunk/themes/tomahawk/includes/appbarClose.ftl Wed Feb 24 
>>> 14:53:40 2010
>>> @@ -88,3 +88,8 @@
>>>  </div>
>>>  <div class="clear">
>>>  </div>
>>> +
>>> +<script>
>>> +  var mainmenu = new DropDownMenu($('main-navigation'));
>>> +  var appmenu = new DropDownMenu($('app-navigation'));
>>> +</script>
>>>
>>> Modified: ofbiz/trunk/themes/tomahawk/webapp/tomahawk/js/dropdown.js
>>> URL: 
>>> http://svn.apache.org/viewvc/ofbiz/trunk/themes/tomahawk/webapp/tomahawk/js/dropdown.js?rev=915812&r1=915811&r2=915812&view=diff
>>> ==============================================================================
>>> --- ofbiz/trunk/themes/tomahawk/webapp/tomahawk/js/dropdown.js (original)
>>> +++ ofbiz/trunk/themes/tomahawk/webapp/tomahawk/js/dropdown.js Wed Feb 24 
>>> 14:53:40 2010
>>> @@ -39,9 +39,4 @@
>>>        }
>>>      });
>>>    }
>>> -};
>>> -
>>> -document.observe('dom:loaded', function(){
>>> -  var mainmenu = new DropDownMenu($('main-navigation'));
>>> -  var appmenu = new DropDownMenu($('app-navigation'));
>>> -});
>>> +};
>>> \ No newline at end of file
>> What browsers did you test this in?
>>
>>

Reply via email to