1. Куча / Говнокод #17439

    +129

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    .hidden {
    display: block;
}
.visible {
    display: none;
}

    И это не шутка. Я полдня искал ошибку в логике в JS, пока случайно увидел, почему модалка не выпрыгивает. Эх... верстальщики

    Запостил: alexscrat, 13 Января 2015

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

    • ava Анонимус 13.01.2015 21:41 # 0

      На кой чорт вообще такой класс нужен? Сущности плодить?

      А класс 
      .red {
color:red;
}

      у вас тоже есть?
      Ответить

      • ava bot 13.01.2015 22:24 # +3

        .red - есть у меня. Почему бы и нет?
        Ответить

        • ava Анонимус 13.01.2015 22:25 # 0

          А зачем он?
          Ответить

          • ava 1024-- 13.01.2015 22:28 # +1

            Может, надо будет потом разом заменить все эти цвета на #400 или #d22.
            Ответить

            • ava Fike 13.01.2015 22:31 # 0

              ну .red тогда действительно нахуй не нужен, нужен какой-нибудь .accent-main (а еще лучше конечно переменными в каком-нибудь sass заправлять).
              Ответить

            • ava Анонимус 13.01.2015 22:34 # 0

              или на blue, например. правда?

              Семантическая верстка потому и семантическая, что знания о конкретных цветах/ширинах/фонтах сокрыты за бизнес-понятиями.

              Бывает правда так, что элемент нужно скрыть на iphone в рамках респонсивного дизайна, а SSAS/LESS никак вот нельзя использовать. И тогда конечно появляются классты типа hidden-iphone на основе media queries, но во-первых они обычно есть во всяких фреймворках (бутстрап тот же и прочие сеточки) а во-вторых в них есть ХОТЬ КАКОЙ-ТО смысл. А в классе "visible" смысла нет. Он ничем не лучше прямого hide() или как там оно в Jquery
              Ответить

              • ava 1024-- 13.01.2015 22:40 # 0

                Ну, не канонично - так не канонично.

                > или на blue, например. правда?
                Нет, я рассуждал с точки зрения http://stat8.blog.ru/lr/0a144a17ed4a0249189e88990e3d3a33
                Ответить

          • ava bot 13.01.2015 22:43 # 0

            Подсвечивать красненьким, же.
            Ваш КО.
            Ответить

            • ava Анонимус 13.01.2015 22:44 # 0

              почему сразу в код не написать color = red?
              Ответить

              • ava bot 13.01.2015 22:51 # +2

                Потому что, add/removeClass()
                Некоторые штуки бывают удобными и сладкими. Как конфетки. Даже в CSS.
                Ответить

                • ava bormand 13.01.2015 23:20 # 0

                  Но ведь в жуквери уже есть hide() и show().
                  Ответить

                  • ava bot 13.01.2015 23:34 # 0

                    То есть вы предлагаете джва <span> и один из них color=red ?
                    Ответить

                    • ava bormand 13.01.2015 23:56 # 0

                      Да, именно так:
                      <font color=red class=red></font>
<font color=green class=green></font>

$("red").hide();
$("green").show();
                      Ответить

                      • ava 1024-- 14.01.2015 00:38 # +1 

                        <font color=#000000 class=color000000></font>
<font color=#000001 class=color000001></font>
<font color=#000002 class=color000002></font>
...
<font color=#fffffe class=colorfffffe></font>
<font color=#ffffff class=colorffffff></font>

                        $("font").hide();
$('.colorff0000').show()
                        Ответить

                  • ava blablabla2 14.01.2015 08:08 # 0

                    Потому что можно вот так например:

                    .hidden + div {
                    display: block;
                    }

                    Покажет див идущий за скрытым, например с неким тултипом. Можно конечно и $(this).hide().next().show(); но от такого можно очень быстро устать, особенно если начать это все анимировать.
                    Ответить

              • ava bormand 13.01.2015 23:20 # +1

                > color = red
                <font color=red>...</font>
                Ответить

              • ava Lokich 14.01.2015 18:08 # 0

                потому, что когда пишешь разметку, проще написать class="red" чем везде добавлять style="color:red;", и это уже не говоря о том, что inline css это бэд практис. да и вообще, в bootstrap есть .pull-left { float:left;}, (и да, конечно же .pull-right { float:right;} тоже есть) но почему-то никто не жалуется.
                Ответить

                • ava Анонимус 14.01.2015 20:12 # +2

                  >>написать class="red" чем везде добавлять style="color:red;"

                  Ни то ни другое не нужно писать в разметке.

                  >>я о том, что inline css это бэд практис
                  Как и color=red.

                  Дамы и господа, вы только что прослушали лекцию на тему "как догма заменяет здравый смысл"
                  Ответить

      • ava blablabla2 14.01.2015 08:04 # +1

        Класс hidden имеет свой смысл, но не в таком виде как описано здесь.

        .hidden {
        overflow: hidden;
        width:0;
        height: 0;
        position: absolute;
        }

        Позволит загружать изображения и подключаемые шрифты в невидимый контейнер.
        Ответить

        • ava bormand 14.01.2015 08:20 # 0

          > подключаемые шрифты
          Эээ... Пока шрифт реально не поюзан на странице он не загружается?
          Ответить

          • ava blablabla2 14.01.2015 09:14 # 0

            Именно так происходит в хроме, в остальных не проверял, думаю будет тоже самое.
            Ответить

            • ava blablabla2 14.01.2015 09:22 # 0

              This means that WebKit and Blink are smart enough to know that even if a node exists in the document that uses our new font-family but the node is empty—the font does not download. This is great!

              Источник: https://dev.opera.com/articles/better-font-face/
              Ответить

    • ava guest 13.01.2015 21:42 # 0

      Кто-то до сих пор дрочит и кончает в трусы.
      Ответить

    • ava mittorn 15.01.2015 21:06 # +1 

      #define TRUE (0)
#define FALSE (1)
.....
if(TRUE)....
      Ответить

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