Показать сообщение отдельно
  #4 (permalink)  
Старый 20.07.2011, 15:57
Профессор
Отправить личное сообщение для ваый Посмотреть профиль Найти все сообщения от ваый
 
Регистрация: 29.06.2011
Сообщений: 445

вместо десятка одинаковых функций сделайте одну для начала:
вместо ShowBackElement1, ShowBackElement2, ...
function ShowBackElement(i) {
    var backgroung = document.getElementById('price' + i);
    backgroung.style.backgroundImage = background;
    
    var red = document.getElementById('under_price' + i);
    red.className = 'class2'; // '#430606';

    var yellow = document.getElementById('old_price' + i);
    yellow.className = 'class3'; // color = '#ffc601';

    var backImg = document.getElementById('wallpaper');
    backImg.style.backgroundImage = imgArr[i];
}
HideBackElement(i) по аналогии
function Wallpaper() {
    for (var i=1; i<5; i++) {
        var element1 = document.getElementById('price' + i);
        element1.onmouseover = function() {
            ShowBackElement(i);
        };
        element1.onmouseout = function() {
            HideBackElement(i);
        };
    }
}
но даже так - это отвратительно. но это нормально, вы ж учитесь
Ответить с цитированием