209518 - [layers] [akonet] poland.gov.pl, pologne.gov.pl, polonia.gov.pl, polen.gov.pl etc.

Status: RESOLVED FIXED

(Tech Evangelism Graveyard :: Polish, defect)

Description

[Marek Stępień [:marcoos, inactive]]

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.4) Gecko/20030611
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.4) Gecko/20030611

These websites, all made by akonet.pl, use document.layers and document.all to
display a navigation menu.

It is the same website, but its language version depends on the domain name. 

This bug is slightly connected with bug  209428  (see this comment:
http://bugzilla.mozilla.org/show_bug.cgi?id=209428#c15 )


Reproducible: Always

Steps to Reproduce:
1. Go to http://www.poland.gov.pl , http://www.pologne.gov.pl ,
http://www.polonia.gov.pl or http://www.polen.gov.pl
2. Hover the links on the left

Actual Results:  
Nothing happens.

Expected Results:  
Display the navigation menu.

Also, Mozilla does not display the kite, which in Opera and MSIE follows the mouse.
But this is good, I think, because it's less annoying. ;)

Comment 1

[Marek Stępień [:marcoos, inactive]]

There's some progress.  The page now works correctly in Mozilla (well, the kite
does not work ;) but causes the browser to hang for a while. This is probably
due to the fact that they used the fixed layers.js from bug 209428. I changed
some functions there to work ok in Mozilla, so I added innerHTML+=something to
replace MSIE's insertAdjacentHTML. It wasn't used on the MSZ website, so I
didn't realize how bad it was. On poland.gov.pl it's executed lots of times, and
this is the cause for the hang...

Probably we need a better replacement for insertAdjacentHTML than innerHTML+=.
DOM methods will be good, but it will cause a complete rewrite of the functionsg.

I'm now looking at http://jscript.dk/faq/mozinsertadjacenthtml.html which has
probably a nice replacement for this.

Comment 2

[Marek Stępień [:marcoos, inactive]]

Attachment: layers.js for poland.gov.pl

This is a new version of layers.js that is compatible with both Mozilla and
MSIE, and doesn't cause the hang. It uses the replacement for
insertAdjacentHTML mentioned above.

Comment 3

[Marek Stępień [:marcoos, inactive]]

Just tested the new layers.js in other browsers (MSIE and Opera). It works ok in
them, too, so I  e-mailed [e-mail] and [e-mail] about the
better version of the script.

Comment 4

[Marek Stępień [:marcoos, inactive]]

They silently uploaded my version of layers.js to all pol*.gov.pl websites. 
RESOLVED FIXED.

(I wonder why they didn't fix bug 209428, cause this is exactly the same problem...)