Skip to content
This repository has been archived by the owner on Jan 17, 2020. It is now read-only.

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

Open
vladshcherbin opened this issue Sep 5, 2017 · 7 comments
Labels

Comments

@vladshcherbin
Copy link
Contributor

Issue - LaravelRUS/docs#187

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

@temyk
Copy link

temyk commented Aug 2, 2018

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

@vladshcherbin
Copy link
Contributor Author

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

@temyk
Copy link

temyk commented Aug 2, 2018

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

@SerafimArts
Copy link
Member

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

@vladshcherbin
Copy link
Contributor Author

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

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

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

@vladshcherbin
Copy link
Contributor Author

vladshcherbin commented Oct 30, 2018

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

  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
Copy link
Contributor Author

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

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

3 participants