Positionnement div

Question

Je donne un coup de poignard aux instructions if else, je ne sais pas si j'écris correctement ... Essayant de l'obtenir de sorte que si la dimension popupHeight est plus grande que la dimension windowHeight, alors il la positionnerait en haut de la fenêtre + 10px ...

$("#data").load("/content/" + htmlName + ".html", null, function(){ 
    //Set Variables 
    var container = $(".container"); 
    var project = $(".project"); 
    var popupWidth = container.find(".project img:first").width(); 
    var popupHeight = container.find(".project img:first").height()+35; 
    var windowWidth = document.documentElement.clientWidth; 
    var windowHeight = document.documentElement.clientHeight; 
    var x = ($(window).width()/2 - popupWidth/2) + $(window).scrollLeft(); 
    var y = ($(window).height()/2 - popupHeight/2) + $(window).scrollTop(); 

    //Set popup dimensions 
    container.css("width" , popupWidth); 
    container.css("height" , popupHeight); 

    //Set popup CSS 
    container.css({"position": "absolute", "left": x + "px", "z-index": "2" }); 
    project.css({"width": (popupWidth), "height": (popupHeight) }); 

    //Determine Position 
    if(popupHeight>windowHeight) { 
     container.css{("top": $(window).scrollTop(); + 10 + "px") 
     }else{ 
     container.css({"top": y + "px"}); 
     return; 
     } 
}); 

Answer 1

Débarrassez-vous de la demi-virgule après scrollTop() et le mettre à la fin de la ligne-

container.css{("top": $(window).scrollTop(); + 10 + "px") 

Si forme-

container.css({"top": $(window).scrollTop() + 10 + "px"}); 

Votre fonction css doit également être reformatée, elle devrait ressembler à ce qui précède.

Answer 2

Je n'ai jamais utilisé jQuery auparavant, donc je peux me tromper, mais le point-virgule de cette ligne n'est-il pas au mauvais endroit?

container.css{("top": $(window).scrollTop(); + 10 + "px")