| Status | pending |
 |
|
Rating
|
    (1)
|
|
Ticket ID
|
2112
|
|
Subject
|
IE7 problem with the "Show/Hide menu " javascript (patch now attached as a workaround)
|
|
Submitted by
|
eromneg
|
|
Priority
|
9 high
|
|
Category
|
Bug: Error Patch
|
|
Tiki Version
|
2.x
|
|
Feature
|
Browser Compatibility
|
|
Description
|
With version 2.2 (not checked previous versions) using the "Show/Hide Menus" option to collapse a left or right hand column with IE7 gives page errors and does not execute. Error message says:
Line 409
Char 4
Error: Could not get the display property. Invalid argument.
Code:0
URL: http://dev.tikiwiki.org/tiki-index.php
Function works fine in FF3 and (surprisingly) IE6 but gives exactly the same problem across a number of different tiki sites with IE7, including dev.tikiwiki.org.
The error message (it is assumed) refers to the function flip(foo,style) in lib/tiki-js.js
|
|
Workaround
|
A colleague who understands javascript (since I certainly don't!) has resolved this issue for me and I have created a patch that works. Its a bit crude/inelegant and does not look at any consequences across other areas of the tiki-js.js or the imminent arrival of IE8, but ...
... the issue is that the variable iewin is only initialised in the function browser to only detect IE5 and IE6, using an intermediate variable ie56. The patch therefore simply adds a new variable ie567 and iewin7 and uses iewin7 for the conditional test in the flip function. I've attached an updated copy of tiki-js.js for anyone that wants it.
|
|
Technical Area
|
Javascript
|
|
Resolution status (legacy)
|
Fixed
|
|
Lastmod by
|
luci
|
| Created |
Thursday 30 October, 2008 17:58:46 UTC |
| LastModif |
Monday 23 February, 2009 16:20:53 UTC |
Last Comments