1. PHP / Говнокод #4419

    +159

    1. 1
    <a href="<?=$res["URL"]?>" style="font-size: <?=$res["FONT_SIZE"]?>px; color: #<?=$res["COLOR"]?>;px" rel="nofollow"><?=$res["NAME"]?></a>

    Битрикс, компонент "Облако тэгов"
    Последний "px" в тэге style явно лишний.

    Запостил: hdkeeper, 22 Октября 2010

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

    • ava Анонимус 22.10.2010 14:45 # +2

      круто)
      удобно наверное скнизиоваться CSSом)
      Ответить

    • ava Мартин 22.10.2010 18:37 # 0

      Что разработчики битрикса курят?
      Ответить

      • ava telnet 23.10.2010 06:04 # 0

        План, исключительно план. Который они должны выполнить, чтобы не остаться без доширака.
        Ответить

    • ava Arigato 22.10.2010 20:16 # 0

      А еще кто-то доказывает, что вставки <?=$res["URL"]?> понятнее, чем {URL}
      Ответить

      • ava Анонимус 22.10.2010 23:34 # 0

        говно этого кода, если Вы не поняли, заключается не в синтаксисе, а в зашитом цвете и фонт-сайзе
        Ответить

        • ava Arigato 23.10.2010 09:19 # 0

          На сколько я понял, говно тут:
          color: #<?=$res["COLOR"]?>;px
          Что касается зашитого цвета и фонта, то он не зашит, а берется из настроек, т.е. я не вижу тут говна, по логике, админ может менять размер шрифта и цвет элемента из админки. Разве что это стоило вынести в контейнер style и прописать класс.
          Но так же говно и в подобных конструкциях имеется: <?=$res["URL"]?>
          Ответить

          • ava Анонимус 23.10.2010 18:19 # 0

            он берется из настроек, а не из css. В том-то и гавно.
            Менять цвет должен не админ, а верстальщик по требованию дизайнера.
            Админ должен оперировать семантическими понятиями "показать блок кода в стиле "важный"".
            А что такое стиль "важный" задается в css.

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

            • ava Arigato 23.10.2010 18:35 # 0

              Это надо смотреть, в каком конкретно случае применяется данная конструкция.
              К примеру, на форуме админ может создавать новые группы и, при желании, задавать для каждой группы свой цвет, что бы ники выводились разными цветами для идентификации группы. По-моему, такое допустимо.
              Ответить

            • ava Lure Of Chaos 23.10.2010 22:38 # 0

              я тоже хотел было высказаться по поводу того, что это в style, а не css -- но вовремя увидел, что это "облако тегов", т.е. визуально разбросанная по области куча слов, цвет и размер которых отличается незначительно (одинаковых наберется не очень много).

              Поэтому, можно ожидать, что лист стилей получится внушительным, и в данном случае, лучше генерировать встроенный стиль, чем генерированные классы стилей. Не сложно, но сложней, чем первый вариант.
              Ответить

              • ava Анонимус 24.10.2010 03:04 # 0

                в облаках тегов класс изображает размер (size-1, size-2 итд). По нему верстальщик делает нужные размеры и цвета. Если это будет делать админ -- то одни таги будут нечитаемые, потому что мелкие. А другие будут слишком большие, не по делу большие.

                Не говоря уж о том, что с css верстальшик может сверстать тестовую страницу (со всеми размерами) и показать ее дизайнеру.
                В предложенном же варианте дизайнер будет сначала добиваться от CMSки что бы она вывела таги в нужном размере, потом подставлять туда размеры и смотреть как получится.
                Офигенно.

                К счастью, так делать никто не будет. Потому что у большинства сайтов нет дизайнеров. И никого не смущает что пользователь форума сам выбирает размер шрифта и его цвет и чуть ли не HTML туда вставляет.
                Потому большинство сайтов выглядят как говно. Что конечно не мешает им работать, приносить прибыль, и смеятся на фанатиками, которые верят в отделение представления от данных и семантическую верстку)
                Ответить

                • ava Lure Of Chaos 24.10.2010 12:01 # 0

                  а в переносе от сайта к сайту, корректировать размеры в цсс?
                  хотя, что я говорю, в цсс тоже можно вычисляемые размеры
                  Ответить

                • ava Arigato 24.10.2010 14:34 # 0

                  Ну да, к примеру, на говнокоде потрясный дизайн :)))))))
                  Ответить

    • ava mrbig66 28.10.2010 09:40 # 0

      Тут лучше использовать em.
      Ответить

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