Javascript darken Couleur de fond

Question

Une idée de pourquoi ça ne marchera pas? Il est censé assombrir la couleur d'arrière-plan d'une page lorsque l'on clique sur CTRL + flèche vers le bas. (Il commence à « #CCCCCC ».)

var color="cccccc"; 

    var isCtrl = false; 
    document.onkeyup=function(e){ 
    if(e.which == 17) isCtrl=false; 
    } 
    document.onkeydown=function(e){ 
    if(e.which == 17) isCtrl=true; 

    if(e.which == 40 && isCtrl == true) { 


    if (color.length >6) { color= color.substring(1,color.length)} 
    var rgb = parseInt(color, 16); 
    var r = Math.abs(((rgb >> 16) & 0xFF)+1); if (r>255) r=r-(r-255); 
    var g = Math.abs(((rgb >> 8) & 0xFF)+1); if (g>255) g=g-(g-255); 
    var b = Math.abs((rgb & 0xFF)+1); if (b>255) b=b-(b-255); 
    r = Number(r < 0 || isNaN(r)) ? 0 : ((r > 255) ? 255 : r).toString(16); 
    if (r.length == 1) r = '0' + r; 
    g = Number(g < 0 || isNaN(g)) ? 0 : ((g > 255) ? 255 : g).toString(16); 
    if (g.length == 1) g = '0' + g; 
    b = Number(b < 0 || isNaN(b)) ? 0 : ((b > 255) ? 255 : b).toString(16); 
    if (b.length == 1) b = '0' + b; 
    var color=r + g + b; 

    document.body.style.backgroundColor="#"+color; 

    } 


    } 

Answer 1

Essayez ceci:

var color = 204; // "cc" in decimal; 

document.documentElement.onkeydown = function(e) { 
    e = e || window.event; 
    var c = e.which || e.keyCode; 
    if(c == 40 && e.ctrlKey) { 
     color = Math.max(color-8,0); 
     document.body.style.backgroundColor = "rgb("+color+","+color+","+color+")"; 
    } 
} 

Answer 2

C'est ce méchant petit morceau de code:

var color=r + g + b; 

il change la portée de la couleur qui en fait toujours pas défini. Cela génère une erreur lorsque vous appelez color.length car undefined n'a pas de propriété length. Changer à

color=r + g + b; 

EDIT:

oh ouais. demo: http://jsfiddle.net/EuShR/ (appuyez sur ctrl FYI vers le bas pour voir l'effet)