Стили нового редактора добавляются прямо в page.content инлайном Исправлено

3

После обновления с новым визуальным редактором обнаружили что в page.html в content добавляется инлайном стили:

<style>.text-center{text-align:center;}.text-right{text-align:right}.text-justify{text-align:justify;}</style>

Код отвечающий за это:

https://github.com/webasyst/webasyst-framework/blo...

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

Разработчики webAsyst:

  • придумали новую идею - хорошо,
  • реализовали - ну как сделали так сделали,
  • застолбили эти классы как стандартные - нет,
  • уведомили об этих нововведениях сторонних разработчиков чтобы они могли внести поддержку новых классов в свои темы - нет

Вот по поводу может создать где-то ленту анонсов нововведений, что будет внесено, изменено. А то получается что некоторые вещи узнаешь чисто случайно. Да и документацию поддерживать в рабочем состояние не помешает, ни тут https://developers.webasyst.ru/features/wysiwyg/ ни тут https://developers.webasyst.ru/templates/pages/ ничего не изменилось.

11 комментариев

  • +2
    J. B. diGriz J. B. diGriz 30 сентября 2017 02:02 #

    сомнительное решение. Я бы в атрибут style форматирование визульного редактора засовывал

  • +2

    На самом деле это не очень удачное решение, потому что теперь $page.content не пустой всегда. Эти стили добавляются автоматом, и это плохо.

  • +1
    fedoric fedoric 9 октября 2017 15:17 #

    А Вы вообще в курсе сколько теперь клиентов повернутых на вот этом https://validator.w3.org/ будут разрушать мозг программистам ?


    • +1
      Eugen Nichikov Eugen Nichikov 9 октября 2017 15:33 #

      Не только, кстати.

      {$page.content|stip_tags|truncate:300} для превью теперь будет хреново работать.

      • 0
        fedoric fedoric 9 октября 2017 15:53 #

        Да вообще. Что это за решение такое. Если уж это так необходимо то

        <head> СЮДА И ТОЛЬКО СЮДА </head>

        {$wa->css()} -- мне казалось вот это специально для таких целей было сделано!

      • +1
        Eugen Nichikov Eugen Nichikov 4 января 2018 13:41 #

        Всплыл ещё один баг. ненужный инлайновый тег style ломает AMP-страницы



      • +1
        info@ravencode.ru info@ravencode.ru 4 января 2018 16:32 #

        Я вообще не удивлен) профессиАнальные решения в духе wa, то что логика моделей зачастую раскидана по контроллерам никого не смущает?) про дублирование кода и его анальную огороженность и невозможность повторного использования я вообще молчу.

        Вспоминаю прошлую часть SS, она долго меня кормила потому что никто не понимал как к туда плагины свои прикрутить)))

        WA на все это настрать, они комьюнити разрабов игнорят, не клиент? бабла не занес? иди нахрен!

        А теперь по существу: лицензия позволяет развивать форк и отпочковаться чтобы не ждать пока пройдет пару лет и до твоего PR дойдут кривые руки WA.

      • +1
        info@ravencode.ru info@ravencode.ru 4 января 2018 16:45 #

        По поводу просмотра изменений, тут git все-таки вносит ясность - https://github.com/webasyst/webasyst-framework/commit/b4fd7ba44d227c0375d41853dd4f62d52aed733a вот последний PR. из интересного: в настройках плагинов стали доступны новые типы полей:

        self::HELP,
        self::TITLE,
        self::CUSTOM,
        self::INTERVAL,

        об этом тоже нигде ни слова не сказано)

      • 0
        Евгений Леман Евгений Леман 29 сентября 2017 23:40 #

        мм.. и чО?

      • -2
        Евгений Леман Евгений Леман 30 сентября 2017 05:31 #
        Я бы

        Ну предпочтения у каждого свои, только поясните посыл темы? Александр, ответьте честно, а вы в ченжлогах своих разработок прямо каждый измененный пиксель описываете? Особенно подобные изменения, на которые всем срать.

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

        Чтобы добавить комментарий, зарегистрируйтесь или войдите