Skip to content
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

fix/feat: Cyrilic fixes by Gaxer & Support by Larentoun. SS220 EDIT EDITION #202

Closed
wants to merge 378 commits into from
Closed

fix/feat: Cyrilic fixes by Gaxer & Support by Larentoun. SS220 EDIT EDITION #202

wants to merge 378 commits into from

Conversation

AyIong
Copy link
Collaborator

@AyIong AyIong commented Sep 18, 2023

Что этот PR делает

Фиксы кириллицы от @Gaxeer и @larentoun собранные в кучу и промаркированные как // SS220 EDIT
К сожалению оффы не приняли то что делал сику так что... довольствуемся тем что есть 😞

Почему это хорошо для игры

Никаких больше знаков вопроса.

Изображения изменений

image
image
image
image

Тестирование

В процессе, многое думаю ингейм всплывёт

Changelog

🆑
fix: Фиксы кириллицы
/:cl:

JimKil3 and others added 30 commits August 18, 2023 14:32
…growing into infestation spiders (ParadiseSS13#21988)

* The fixes

* Steelslayer review
)

* Fixes health scanner sprite not looping

FUCKING HELL

Removes old file

* Fixes health scanner icon animation not looping

Prevents medics from declaring people dead because they permanently flatline after being scanned

Adds 'indefinately' to loop status for medical scanner icons
* Fix Forsaken Cat spawn from Gold Slimes.

* Typo

* ???

* eeehhhh... no funky text
* beartrap trigger on find

* fixed an oversight

* Update code/game/objects/items/weapons/legcuffs.dm

Co-authored-by: DGamerL <[email protected]>

* 2 second timer for arming a trap

* Update code/game/objects/items/weapons/legcuffs.dm

Co-authored-by: DGamerL <[email protected]>

* adjust breakout time

* apply extra damage if bear

* 15% to delimb on opening bag

* reverts delimb

* first set of review changes

* small adjustment

* spring_trap proc work

* spring_trap first pass

* spring_trap take two

* code formatting and simple mob damage

* Update code/game/objects/items/weapons/legcuffs.dm

Co-authored-by: Charlie <[email protected]>

* review fixes

* Apply suggestions from code review

Co-authored-by: Henri215 <[email protected]>

* a little code cleaing for review

* Update code/game/objects/items/weapons/legcuffs.dm

Co-authored-by: Henri215 <[email protected]>

* Update code/game/objects/items/weapons/legcuffs.dm

Co-authored-by: Henri215 <[email protected]>

---------

Co-authored-by: DGamerL <[email protected]>
Co-authored-by: Charlie <[email protected]>
Co-authored-by: Henri215 <[email protected]>
…SS13#21313)

* Vars and ideas

* Adds supercharge implant

* species specific tot gear!

* Adds vox spikethrower

* Adds Love injecting pen (skrell)

* sprite issue fix

* typographical error.

* makes pen not refillable, pacifies

* Adds fireproofing nanite injector (plasmaman)

* Lower KD on spikethrower per qwerty

* Lewcc review

* aaaa so much space indentation

* steelslayer review

* Steelslayer review

* Lewcc reviews

* Changes the spans

* Lewc review, Thanks to DG for doing my PR for me.

* Update code/modules/projectiles/guns/energy/special_eguns.dm

Co-authored-by: Luc <[email protected]>

* Henri review

* Forgot this one

* Whoopsie number 2

---------

Co-authored-by: DGamerL <[email protected]>
Co-authored-by: Luc <[email protected]>
* fix: jobbaned can't spam-spawn as cult harvesters
ss220-space/Paradise#1283

* fix: oops, what is this check?

* smoke needs create_smoke
…g now. (ParadiseSS13#21674)

* RATATATATATA

* We use TRUE, not 1

* making it cost 12

* making progress

* FINALLY

* Lewcc review

* Whoops this too

* Lewcc review

* Might work, might not, but conflict time

* Henri review
* Add bump

* nah

* Add examine text to note it can be wrenched safely.
…aradiseSS13#21763)

* Better PDA logging, fixes sanitisation issue

* Adds sanitisation and max length to every input

---------

Co-authored-by: Miraviel <[email protected]>
* Add railing caps
-subtype of railing corners

* Batch of changes:
- Fixed cap sprite
- Added 2 more instances of caps (left and right)
- Added caps to rods crafting menu
- Changed cost of corner from 3 to 2
- Added cost to cap, it cost 2 rods

* Woops:
- Apparently railings corners and such borrow code from normal railing so they always drop 3 rods.

* Change railings to list

* removed density/climbable
- they inhert it from railing
* Adds some fresh emotes

* Adds necessary keybinds

* oops

* to_chat moment

* but here's the Thing

* Update code/modules/mob/living/carbon/human/human_emote.dm

Co-authored-by: Henri215 <[email protected]>

* Add missing name

---------

Co-authored-by: Henri215 <[email protected]>
* Add blacklisted areas for tcomms cores

* Blacklists some areas from tcomm cores

* this is better

* Update code/game/machinery/tcomms/tcomms_core.dm

Co-authored-by: Henri215 <[email protected]>

---------

Co-authored-by: Henri215 <[email protected]>
* Fixes embed surgery on synths not working

* Oops

* owner -> victim
* tweak: Better stacks

* review changes

* del return

* that's it
* Adds a new space ruin.

* Update code/datums/ruins/space_ruins.dm

lewcc's review

Co-authored-by: Luc <[email protected]>

* id_tag fix

* thanks StrongDMM

* guess who can't place '"' properly

* additions and fixes (i hope)

* the one commit to pass all CI checks

* defined in config

* removed an illegal placement of a railing

---------

Co-authored-by: Luc <[email protected]>
…3#21878)

* ok one second I gotta do something else

* Okay we ball

* Okay we are stealthier now

* Does this work? I have no idea.

* So cloooose

* I don't know what the fuck I am doing

* IT WOOOOOOORKS

* Contra reviews

* oops

* Contra review

* Now it actually works
…diseSS13#21885)

* and i saaaaid JUMP (JUMP!)

* logic moment + chef

* lewcc + contra reviews
* i think this is workin

* sillly me

* also this guh
* Make chem dispenser transfer to beaker by default

* Fix chem masters' default product name when supplied invalid product number
* Add minimum length + warning

* Remove nullcheck

* The Minimalist Code Initiative
AyIong and others added 8 commits September 19, 2023 03:12
<!-- Пишите **НИЖЕ** заголовков и **ВЫШЕ** комментариев, иначе что то
может пойти не так. -->
<!-- Вы можете прочитать Contributing.MD, если хотите узнать больше. -->

## Что этот PR делает
Вернул полы которых не было в модуле.
Поменял dir у plague пола, дабы на офф картах он не был чёрным на белом
полу.
Потрогал ЦК, так как там были неуместные plague полы, плюс починил
рипли.
Удалил ненужные свето-полы.

<!-- Вкратце опишите изменения, которые вносите. -->
<!-- Опишите **все** изменения, так как противное может сказаться на
рассмотрении этого PR'а! -->
<!-- Если вы исправляете Issue, добавьте "Fixes #1234" (где 1234 - номер
Issue) где-нибудь в описании PR'а. Это автоматически закроет Issue после
принятия PR'а. -->

## Почему это хорошо для игры
На мете вернулось название станции.

<!-- Опишите, почему, по вашему, следует добавить эти изменения в игру.
-->

## Изображения изменений
Ха-ха-ха

<!-- Если вы не меняли карту или спрайты, можете опустить эту секцию.
Если хотите, можете вставить видео. -->

## Тестирование
Ха...
<!-- Как вы тестировали свой PR, если делали это вовсе? -->

## Changelog

:cl:
fix: Мета вернула своё название на полу.
/:cl:

<!-- Оба :cl:'а должны быть на месте, что-бы чейнджлог работал! Вы
можете написать свой ник справа от первого :cl:, если хотите. Иначе
будет использован ваш ник на ГитХабе. -->
<!-- Вы можете использовать несколько записей с одинаковым префиксом
(Они используются только для иконки в игре) и удалить ненужные. Помните,
что чейнджлог должен быть понятен обычным игроком. -->
<!-- Если чейнджлог не влияет на игроков(например, это рефактор), вы
можете исключить всю секцию. -->
<!-- Пишите **НИЖЕ** заголовков и **ВЫШЕ** комментариев, иначе что то
может пойти не так. -->
<!-- Вы можете прочитать Contributing.MD, если хотите узнать больше. -->

## Что этот PR делает
Разделил центральный холл на несколько зон, дабы не был обесточен по КД
и клоун не игрался с файралярмами.
Подсоединил АПЦшки которые забыл подсоединить в ходе рефактора.

<!-- Вкратце опишите изменения, которые вносите. -->
<!-- Опишите **все** изменения, так как противное может сказаться на
рассмотрении этого PR'а! -->
<!-- Если вы исправляете Issue, добавьте "Fixes #1234" (где 1234 - номер
Issue) где-нибудь в описании PR'а. Это автоматически закроет Issue после
принятия PR'а. -->

## Почему это хорошо для игры
Фиксы багов
<!-- Опишите, почему, по вашему, следует добавить эти изменения в игру.
-->

## Изображения изменений
MapDIFF
<!-- Если вы не меняли карту или спрайты, можете опустить эту секцию.
Если хотите, можете вставить видео. -->

## Тестирование
Загрузил локалку, прогнал дебаггер, полетал и посмотрел через некоторое
время.
Старый SDMM снова насрал кучей строчек, увы.

<!-- Как вы тестировали свой PR, если делали это вовсе? -->

## Changelog

:cl:
fix: Починил свет на Керберосе.
/:cl:

<!-- Оба :cl:'а должны быть на месте, что-бы чейнджлог работал! Вы
можете написать свой ник справа от первого :cl:, если хотите. Иначе
будет использован ваш ник на ГитХабе. -->
<!-- Вы можете использовать несколько записей с одинаковым префиксом
(Они используются только для иконки в игре) и удалить ненужные. Помните,
что чейнджлог должен быть понятен обычным игроком. -->
<!-- Если чейнджлог не влияет на игроков(например, это рефактор), вы
можете исключить всю секцию. -->
@AyIong AyIong changed the title fix: Cyrilic fixes by Gaxer fix: Cyrilic fixes by Gaxer & Support by Larentoun Sep 18, 2023
@AyIong AyIong changed the title fix: Cyrilic fixes by Gaxer & Support by Larentoun fix/feat: Cyrilic fixes by Gaxer & Support by Larentoun Sep 18, 2023
@AyIong AyIong changed the title fix/feat: Cyrilic fixes by Gaxer & Support by Larentoun fix/feat: Cyrilic fixes by Gaxer & Support by Larentoun. SS220 EDIT EDITION Sep 18, 2023
@AyIong AyIong marked this pull request as ready for review September 18, 2023 23:38
@AyIong AyIong added the :finnadie: НЕ МОДУЛЬНО Па-ху-ю. label Sep 19, 2023
@SS220Manager SS220Manager added the 📜 CL валиден Этот чейнджлог будет успешно опубликован label Sep 19, 2023
@Furrior
Copy link

Furrior commented Sep 19, 2023

Все чар проки, кроме length_char я придумал как сделать проще, так что их не трогаем.

@Furrior
Copy link

Furrior commented Sep 19, 2023

Списки со всяким еще, в теории, можно чуть модульнее сделать

@AyIong AyIong marked this pull request as draft September 19, 2023 12:25
@Furrior Furrior force-pushed the master branch 2 times, most recently from 94e3526 to 985130e Compare September 20, 2023 15:51
@github-actions github-actions bot added the Merge Conflict Ну блять... label Sep 20, 2023
@AyIong
Copy link
Collaborator Author

AyIong commented Sep 20, 2023

No comment

@AyIong AyIong closed this Sep 20, 2023
@AyIong
Copy link
Collaborator Author

AyIong commented Sep 20, 2023

Хотя... секундочку

@AyIong AyIong reopened this Sep 20, 2023
@AyIong
Copy link
Collaborator Author

AyIong commented Sep 20, 2023

Не важно

@AyIong AyIong closed this Sep 20, 2023
@AyIong AyIong deleted the cyrilic-fixes-by-gaxer branch September 27, 2023 18:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:finnadie: НЕ МОДУЛЬНО Па-ху-ю. Merge Conflict Ну блять... 📜 CL валиден Этот чейнджлог будет успешно опубликован
Projects
None yet
Development

Successfully merging this pull request may close these issues.