-
Notifications
You must be signed in to change notification settings - Fork 210
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Лицензия торговцам #165
Лицензия торговцам #165
Conversation
ПР пока чисто для обсуждения, принимать не надо - лицензия выглядит уныленько. |
Так в html переносы строк <br>, а \n игнорируется - все правильно. Файл лучше в кэш положить, указав его в |
Какие надо рандомизированные поля:
Механизм генерации:
|
В рамках этого ПРа сделаю параметры и нормальную генерацию валидной лицензии. Ошибки потом другим ПРом вносить будем, а то много всего. Ты главное печати нарисуй, пока я поля эти генерирую, да. А то получается мы ПР не закроем этот без них. |
Печати в процессе. |
Да, они будут сильно пиксельные, шоб не выбиваться из стиля. |
Вкинул спрайтики в дискорд и запинил их в контрибутерке. |
Бамп. |
че, сделали? |
Сколько лет оно будет лежать? |
Пока Эпикус не вернется, очевидно. |
пусть лежит, я так сказать, продолжу дело Эпикуса... |
@BlakeCrazy если ты жив и легитимный президент, то давай я вышлю тебе dmiшки с печатями и айтемом лицензии. |
Ну все, пизда билду. |
На швятое покусились |
Ушла эпоха |
Бранчу не выкидываю, коммиты потом оттуда вытянем. |
Точно билд загнётся теперь |
Кощунство |
На работу! |
11df5de
to
367c9bd
Compare
К О Г Д А ? |
var/html_path | ||
|
||
/obj/item/weapon/paper/fromhtml/New() | ||
info = russian_to_utf8(file2text(html_path), prepare_to_browser = TRUE) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Мне не нравится, что оно забивает на предыдущие реализации New() и то, что оно не пользует info_links (а оно показывается, если прожать по бумаге ручкой).
Когда (если) вмержат #3257, можно будет запилить через функционал апгрейднутой бумаги (без явного пользования штмля). Ну и оно явно еще не готово, пусть повисит драфтом. |
Этот ПР уже неактуален и его надо переделывать чуть более чем полностью. Особенно после обновления бумаг от Джаммера. Думаю пришло время его закрыть. |
🎉 |
Добавил новый класс бумаг, которые при создании берут свой контент из .html файлов.
Техническая часть:
Возникла проблема с отображением html в бумагах. В обычном браузере он ведет себя довольно странно - два переноса строки превращается в один
<br>
, а один перенос строки игнорируется. Последовательность "\r\n\r\n" Бьонд категорически принимать отказался, так что я зафорсил в папке с описанием документов LF-стиль для окончаний, чтобы в конце строк всегда были \n вместо \r\n.Ну не знаю. В принципе, работает. Можно было бы регексы воткнуть, но чет не очень хочется.
Дизайн:
Лицензия выглядит так:

Видимо, я еще добавлю рандомную генерацию для различных полей лицензии (имя, дата, срок, компания, категории товаров, список станций и кораблей, которые можно посещать).
Категории товаров надо бы продумать, не знаю какие стоит выделить.
issue #46