Home/ Questions/Q 756075
In Process

The Archive Base Latest Questions

Editorial Team
  • 0
Asked: 2026-05-14T15:12:13+00:00

I now understand that the following code will not work because I’m assigning window.onload

  • 0

I now understand that the following code will not work because I’m assigning window.onload to the result of the function, not the function itself. But if I remove the parens, I suspect that I have to explicitly call a separate function to process the config before the onload. So, where I now have:

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<HEAD>
<script type="text/javascript" src="lb-core.js"></script>
<script type="application/javascript">
var lbp = {
    defaults: {
        color: "blue"
    },
    init: function(config) {
        if(config) {
            for(prop in config){
                setBgcolor.defaults[prop] = config[prop];
            }
        }
        var bod = document.body;
        bod.style.backgroundColor = setBgcolor.defaults.color;
    }
}
var config = {
    color: "green"
}
window.onload = lbp.init(config);
</script>
</HEAD>
<body>
<div id="container">test</div>
</body>
</HTML>

I imagine I would have to change it to:

var lbp = {
    defaults: {
        color: "blue"
    },
    configs: function(config){
            for(prop in config){
                setBgcolor.defaults[prop] = config[prop];
            }
    },
    init: function() {
        var bod = document.body;
        bod.style.backgroundColor = setBgcolor.defaults.color;
    }
}
var config = {
    color: "green"
}

lbp.configs(config);
window.onload = lbp.init;

Then, for people to use this script and pass in a configuration, they would need to call both of those bottom lines separately (configs and init). Is there a better way of doing this?

Note: If your answer is to bundle a function of window.onload, please also confirm that it is not hazardous to assign window.onload within scripts. It’s my understanding that another script coming after my own could, in fact, overwrite what I’d assigned to onload. With that in consideration, it’s best to expect your script user to call the function on initialization himself.

Share

Leave an answer

You must login to add an answer.

Need An Account,

1 Answer

  1. Editorial Team

    Use an anonymous function to pass options, like this:

    window.onload = function() { lbp.init(config); };

    If you’re worried about getting overwritten take the unobtrusive route, like this.

    Alternatively you can use a library for your javascript needs, for example I use jQuery for this, and it would look like this:

    $(function() { lbp.init(config); });

    Getting a library for only this is overkill, but if you have lots of javascript going on, I’d suggest you take a look at what’s available.

    • 0