ЙажаСценарий / Говнокод #1877 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  10. 10
  11. 11
function SetStyleForFilterOver(this2) {
    this2.style.border = "1px solid #ea0000";
    this2.style.color = "white";
    this2.style.background = "#404040";
}
function SetStyleForFilterOut(this2) {
    this2.style.border = "1px solid #404040";
    this2.style.color = "#ea0000";
    this2.style.color = "#ea0000";
    this2.style.background = "#eeeeee";
}

Тс-с-с-с... Только не говорите автору про css и селектор ":hover" :о)

Запостил: Ordos Ordos, (Updated )

Комментарии (18) RSS

  • ava guest # Ссылка на оригинал 0

    угу, и ie6 который ховер понимает только для ссылок
    Ответить

  • ava Oleg_quadro # Ссылка на оригинал 0

    Только что проверил на IE 8, он поддерживает ":hover" только для ссылок
    Ответить

    • ava Ordos # Ссылка на оригинал 0 ⇈

      Не правда, только что проверил:
      div{background:#f70;}
      div:hover{background:#07f;}
      Ответить

      • ava guest # Ссылка на оригинал 0 ⇈

        кроссбраузерно только для ссылок. Но не говнокодер сделал бы два класса.
        Ответить

        • ava Sadie # Ссылка на оригинал 0 ⇈

          я так и делаю - но это тоже наверно говно-код... есть еще какой способ?
          Ответить

          • ava guest # Ссылка на оригинал 0 ⇈

            Если hover не для ссылок, то только классы + js. Другие варианты от лукавого. Судя по коду, автор не умеет применять классы, да и просто строки повторяет зачем-то, видимо для надежности.
            Ответить

        • ava Ordos # Ссылка на оригинал 0 ⇈

          Если не учитывать старые браузеры, то вполне кроссбраузерно. (ну подумаешь, у пользователей ie6 мигать ничего не будет :о)
          Ответить

      • ava Oleg_quadro # Ссылка на оригинал 0 ⇈

        На IE 8 проверял?
        твой код проверил только что на IE 8, не работает.
        Ответить

        • ava Ordos # Ссылка на оригинал 0 ⇈

          Да, я проверил на IE8. Сделал минимальную хтмл-страничку (доктайп, теги хтмл, хеад,боди), в тело поместил <div>Hello</div>. Всё работает.
          Ответить

          • ava striker # Ссылка на оригинал 0 ⇈

            на этом сайте через :hover сдалана подсветка блока комментария. Так вот проверил в ie8, эта зараза ест :hover только при наведении на текст, тогда как другие браузеры отрабатывают как надо.
            Ответить

          • ava Oleg_quadro # Ссылка на оригинал 0 ⇈

            Спасибо, с доктайпом <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">

            всё работает. Я то делал страничку без доктайпа. Спасибо.
            Ответить

  • ava Sadie # Ссылка на оригинал 0

    пасиба... я так часто говнокодю... теперь буду стараться :hover 🙂
    Ответить

  • ava guest # Ссылка на оригинал 0

    кстати в ие прямое присваивание отрабатывает в разы(!!!) быстрее, чем присваивание стиля. в свое время столкнулись с этим в проекте.
    Ответить

    • ava guest # Ссылка на оригинал 0 ⇈

      что такое прямое присваивание и присваивание стиля?
      Ответить

    • ava Oleg_quadro # Ссылка на оригинал 0 ⇈

      тут такое дело, если много стилей так устанавливать:
      "this2.style.border = "1px solid #404040";"
      то может быть замедление, потому что после каждой такой команды, вызывается перерисовка, которая может особенно долгой если меняется длина или высота объекта, ведь нужно подвинуть и другие объекты (в случае если position:absolute, то перерисовка особо не замедлится).

      Кто хочет почитать об этом отсылаю сюда: http://dev.opera.com/articles/view/efficient-javascript/
      статья написана разработчиками Opera, но не только для браузера Опера.
      Ответить

Добавить комментарий

Я, guest, находясь в здравом уме и твердой памяти, торжественно заявляю:

    А не использовать ли нам bbcode?

    ↑
    8
    ↓