1. JavaScript / Говнокод #1877

    +157.3

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    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, 21 Сентября 2009

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

    • ava guest 21.09.2009 15:46 # 0

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

    • ava Oleg_quadro 21.09.2009 16:05 # 0

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

      • ava Ordos 21.09.2009 16:14 # 0

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

        • ava guest 21.09.2009 16:31 # 0

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

          • ava Sadie 21.09.2009 21:25 # 0

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

            • ava Oleg_quadro 21.09.2009 21:31 # 0

              Думаю не говнокод. Когда в классах, легче управлять
              Ответить

            • ava guest 22.09.2009 08:39 # 0

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

          • ava Ordos 22.09.2009 07:50 # 0

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

        • ava Oleg_quadro 21.09.2009 21:29 # 0

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

          • ava Ordos 22.09.2009 07:47 # 0

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

            • ava striker 22.09.2009 09:59 # 0

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

            • ava Oleg_quadro 22.09.2009 13:24 # 0

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

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

    • ava Smoke 21.09.2009 16:07 # 0

      css, да
      а вот :hover нет
      Ответить

    • ava Sadie 21.09.2009 16:24 # +2

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

    • ava guest 22.09.2009 14:34 # 0

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

      • ava guest 22.09.2009 14:56 # +1

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

      • ava Oleg_quadro 22.09.2009 17:47 # 0

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

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

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