LaravelRUS / laravel.ru

Please use https://github.com/LaravelRUS/laravel.su instead
Other
62 stars 13 forks source link

Тэг <img /> ломает разметку в документации #84

Open vladshcherbin opened 7 years ago

vladshcherbin commented 7 years ago

Issue - https://github.com/LaravelRUS/docs/issues/187

Аналогично, http://laravel.su/docs/5.4/releases#laravel-5.4

temyk commented 6 years ago

Разметка будет исправлена? уже почти год прошёл. Хотелось бы дочитать документацию

vladshcherbin commented 6 years ago

@aprixodko в планах была новая версия сайта, но у её главного разработчика сейчас вряд ли есть время на неё, так что скорее всего нет.

temyk commented 6 years ago

@vladshcherbin этот баг хотя бы исправить. Это же меньше времени, чем новый сайт)

SerafimArts commented 6 years ago

@aprixodko исходники текущей морды в ветке develop: https://github.com/LaravelRUS/laravel.ru/tree/develop

vladshcherbin commented 6 years ago

@aprixodko так оно и есть, но все как-то были настроены на то, что баги пофиксятся в новой версии и старую оставили как есть, она уже 4 года без практически каких-либо обновлений.

Спуся столько лет, большинство разработчиков старой версии занято новыми проектами, используют новые технологии или вообще оставили програмиирование и не особо хотят в старом копаться.

Мы на самом деле всегда рады PR и если есть возможность пофиксить этот баг, мы только за :)

vladshcherbin commented 6 years ago

После танцев с бубном с настройкой, кажется я знаю, как это решить:

1) убрать https://github.com/LaravelRUS/laravel.ru/blob/develop/composer.json#L11 2) обновить только https://github.com/LaravelRUS/laravel.ru/blob/develop/composer.json#L21

В новой версии пакета (2) используется обновлённый parsedown (1), в котором этого бага нет.

Небольшая загвоздка - в репе нет composer.lock, поэтому с сервака нужно аккуратно удалить пакет 1 и обновить пакет 2, другие не трогать ибо там беда (висят dev-master версии пакетов, которые подтянут совсем не то). На серваке должен быть composer.lock и аккуратно обновить по идее можно.

@SerafimArts можешь попробовать провернуть?

vladshcherbin commented 6 years ago

Также если есть возможность закоммитить composer.lock с сервака, я могу попробовать обновить локально и закинуть новый .lock файл.