[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#406020: debbugs: better toggle_infmessages function



Package: debbugs
Severity: normal
Tags: patch


The current toggle_infmessages function does not work in at least
konqueror.
(konqueror do not seem to have
document.styleSheets.styles.cssRules.cssText )

This proposed function does it a bit different:

function toggle_infmessages()
{
        allDivs=document.getElementsByTagName("div");
        for (var i = 0 ; i < allDivs.length ; i++ )
        {
                if (allDivs[i].className == "infmessage")
                {
                        allDivs[i].style.display=(allDivs[i].style.display == 'none') ? 'block' : 'none';
                }
        }
}

but it is at least from my point of view a bit cleaner - and it works in
at least iceweasel and konqueror.

It would also be nice if it the default css was display: block - and in
the end of the document, toggle_infmessages() would be called.
Then people with javascript disabled was able to see the 'useless'
messages anyway, and people with javascript enabled wouldn't see any
difference.

/Sune


-- System Information:
Debian Release: unstable/experimental
  APT prefers unstable
  APT policy: (500, 'unstable'), (500, 'testing'), (200, 'experimental')
Architecture: i386 (i686)
Shell:  /bin/sh linked to /bin/dash
Kernel: Linux 2.6.18-1-vserver-k7
Locale: LANG=en_GB, LC_CTYPE=en_GB (charmap=ISO-8859-1)



Reply to: