css jquery doesn't work

Question

First apologize for my English. I have a popup when I do click in triggerPop this one it appears, I'm trying to show this one to the right but the css doesn't work. What am I doing wrong? Thank you in advance.

var monscreen = $(window).width();
var mondoc = $(document).width();

if (mondoc > monscreen) {
    var dif = mondoc - monscreen;
    $("div#Navigation_Popup").css({
        'right': dif + 'px'
    });
}

$(function() {
    $('#triggerPop').bind('click', function(e) {
        $("#Navigation_Popup").slideFadeToggle(function() {});
        return false;
    });

    $.fn.slideFadeToggle = function(easing, callback) {
        return this.animate({
            opacity: 'toggle',
            height: 'toggle'
        }, "fast", easing, callback);
    }
});​

Answer 1

I'm going off of the limited information. You mentioned css ...

I would move all of this code:

var monscreen = $(window).width();
var mondoc = $(document).width();

if (mondoc > monscreen) {
  var dif = mondoc - monscreen;
  $("div#Navigation_Popup").css({
    'right': dif + 'px'
  });
}

Inside of

$(function(){
  .
  .
  .
});

As it is now, the if (mondoc > monscreen) code block is getting executed before the document is ready.

Answer 2

Not sure how jquery treat these values but, isnt var monscreen = $(window).width(); var mondoc = $(document).width(); returning strings? How about an alert to check this?

Answer 3

Bind is deprecated, try on instead. I also think everything should be inside $(function()) handler, so, try this:

$(function() {
    var monscreen = $(window).width();
     var mondoc = $(document).width();

    if (mondoc > monscreen) {
        var dif = mondoc - monscreen;
        $("div#Navigation_Popup").css({
           'right': dif + 'px'
        });
    }
    $('#triggerPop').on('click', function(e) {
        $("#Navigation_Popup").slideFadeToggle(function() {});
        return false;
    });

    $.fn.slideFadeToggle = function(easing, callback) {
        return this.animate({
            opacity: 'toggle',
            height: 'toggle'
        }, "fast", easing, callback);
    }
});