Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

Combine onload and onresize (jQuery)

I want to call the function on load as well as on resize.

Is there a better way to rewrite this more compactly?

$('.content .right').width($(window).width() - (480));
$(window).resize(function(e) {
    $('.content .right').width($(window).width() - (480));
});
like image 800
eozzy Avatar asked Dec 29 '09 14:12

eozzy


4 Answers

You can bind to the resize event alone, and trigger this event automatically upon load:

// Bind to the resize event of the window object
$(window).on("resize", function () {
    // Set .right's width to the window width minus 480 pixels
    $(".content .right").width( $(this).width() - 480 );
// Invoke the resize event immediately
}).resize();

The last .resize() call will run this code upon load.

like image 143
Sampson Avatar answered Nov 17 '22 23:11

Sampson


I think the best solution is just to bind it also to the load event:

$(window).on('load resize', function () {
    $('.content .right').width( $(this).width() - 480 );
});
like image 20
Hativ Avatar answered Nov 17 '22 23:11

Hativ


It is nice to spot repeating logic and break that out to a function instead:

function sizing() {
  $('.content .right').width($(window).width() - 480);
}

$(document).ready(sizing);
$(window).resize(sizing);
like image 18
Emil Vikström Avatar answered Nov 18 '22 00:11

Emil Vikström


I'm going to combine the best parts of two other answers. First, move repeated code into a function. Second, don't pollute the global namespace.

$(document).ready(function() {
  var adjust_size = function() {
    $('.content .right').width($(window).width() - 480);
  };
  adjust_size();
  $(window).resize(adjust_size);
});

I named the function adjust_size because I prefer verbs for action-oriented functions.

like image 4
David J. Avatar answered Nov 18 '22 01:11

David J.