[JAVASCRIPT] functie retouneert geen string in FF

[Spiral]

Ik wil de achtergrond aanpassen. In IE7 werkt deze javascript functie maar in FF3 niet.

JavaScript:

function getBackgroundUrl(_width, _height, _rgb)
{
    if(_rgb == undefined)
    {
        var _rgb = "0, 119, 255"; 
    }
    
    return "url(background.php?width=" + _width + "&height=" + _height + "&rgb=" + _rgb + ")";
}

function setSiteBackground()
{
    var width = 0;
    var height = 0;
    
    if (window.innerWidth && window.innerHeight)
    { 
        width = window.innerWidth; 
        height = window.innerHeight; 
    } 
    //IE Mozilla 
    if (document.body.clientWidth && document.body.clientHeight)
    { 
        width = document.body.clientWidth; 
        height = document.body.clientHeight; 
    } 

    document.getElementById("container").style.backgroundImage = getBackgroundUrl(width,height, "12,12,12");
}

als ik regel 28 verander in het volgende dan werkt het in zowel IE als FF3:

JavaScript:

document.getElementById("container").style.backgroundImage = "url(background.php?width=" + width + "&height=" + height + "&rgb=12,12,12)";

Waarom werkt het eerste niet? Ik heb via alert(getBackgroundUrl()) al geverifieerd dat hij de goede output genereerd. Ik weet ook niet hoe ik dit probleem moet benoemen om te googlen

[UltimateB]

werkt het wel als je regel 28 maakt:

JavaScript:

    document.getElementById("container").style.backgroundImage = "" + getBackgroundUrl(width,height, "12,12,12"); 

[Snake]

En als je debugt in Firefox met Firebug? Wat het type is enzo? Dit is eigelijk basic debugwerk.

[Spiral]

UltimateB schreef op zondag 02 november 2008 @ 19:15:
werkt het wel als je regel 28 maakt:

JavaScript:

    document.getElementById("container").style.backgroundImage = "" + getBackgroundUrl(width,height, "12,12,12"); 

Nee dat werkt helaas ook niet Ik heb ook meteen maar " ' " geprobeerd en dat uiteraard ook toegevoegd aan het einde. Zoals hieronder. Dit mocht ook niet baten

JavaScript:

    document.getElementById("container").style.backgroundImage = "'" + getBackgroundUrl(width,height, "12,12,12") + "'"; 

Ik vind het heel vreemd! Dit is wel mijn eerste dagje met javascript btw

[Spiral]

Snake schreef op zondag 02 november 2008 @ 19:20:
En als je debugt in Firefox met Firebug? Wat het type is enzo? Dit is eigelijk basic debugwerk.

Ik weet niet wat je met type bedoeld maar het geldt voor een <div id="container">

[Snake]

-

[ Voor 100% gewijzigd door Snake op 02-11-2008 19:54 ]

[Haan]

Probeer anders eens:

JavaScript:

var background = getBackgroundUrl(width,height, "12,12,12");
document.getElementById("container").style.backgroundImage = background;

[Spiral]

AARRGGHH Opgelost!

JavaScript:

function getBackgroundUrl(_width, _height, _rgb)
{
    if(_rgb == undefined)
    {
        var _rgb = "0, 119, 255"; 
    }
    
    return "url(background.php?width=" + _width + "&height=" + _height + "&rgb=" + _rgb + ")";
}

Het waren de spaties in _rgb die het 'm deden. IE7 verwijderd ze waarschijnlijk.
In background.php doe ik o.a. het volgende

PHP:

$rgb_values = explode(",",$_REQUEST["rgb"]);

Waarschijnlijk gaan de spaties verloren in IE7