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

full auto fire tg edition #13479

Merged
merged 11 commits into from
Sep 22, 2024
Merged

Conversation

simb11
Copy link
Contributor

@simb11 simb11 commented Aug 16, 2024

Описание изменений

то же самое что и тут #13467 , только портировано с другого билда, на котором автоматика сделана через компоненту
Добавляет некоторым пушкам возможность стрелять автоматически и очередями.

bandicam.2024-08-16.20-43-38-972.mp4

В качестве противовеса куче автоматического оружия у антагов, в карго был добавлен ящик с L13 (security SMG который) и патронами к нему.

Ребаланс L6 SAW:
В коробке с патронами для пулемёта их теперь 100 штук (было 50)
Сами пули пулемёта стали намного более слабыми.

Почему и что этот ПР улучшит

Автоматы наконец то станут автоматами.

Авторство

Порт отсюда: https://github.com/tgstation/tgstation

Чеинжлог

🆑 Simbaka

  • add[link]: Добавлен автоматический режим стрельбы и стрельба короткими очередями.
  • add: В карго можно заказать пистолет-пулемёт и патроны для него.
  • balance: L6 SAW обладает большим магазином, но меньшим уроном.

@TauKitty
Copy link
Contributor

Changelog status: ✔️

@TauKitty TauKitty requested a review from a team August 16, 2024 19:16
Copy link
Contributor

@WatlerJe WatlerJe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Те же требования что и к предыдущему ПРу.

@TauKitty TauKitty requested a review from a team August 19, 2024 09:23
Copy link

github-actions bot commented Sep 2, 2024

Данный ПР автоматически отмечен как застоявшийся по причине длительного отсутствия обновлений. Он будет закрыт через 7 дней, если никакой активности не будет проявлено. Если вы считаете, что ПР еще актуален, или что я (злобный робот) пристаю к вам зря - просто напишите любой комментарий. Спасибо за ваш вклад.

@Ro2tCrab
Copy link
Contributor

Ro2tCrab commented Sep 6, 2024

Где ревью?
Опять ПРы тиранят?

@volas
Copy link
Member

volas commented Sep 8, 2024

на неделе надо в тм поставить, сейчас не ставлю так как сам еще не смотрел код

Comment on lines +180 to +182
/client/MouseDrag(src_object,atom/over_object,src_location,over_location,src_control,over_control,params)
SEND_SIGNAL(src, COMSIG_CLIENT_MOUSEDRAG, src_object, over_object, src_location, over_location, src_control, over_control, params)
..()
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

Don't define this unless you need it, because it generates extra communication that is otherwise avoided. Most operations can be done through Click(), DblClick(), and MouseDrop(). The other procedures are simply available for completeness.

https://www.byond.com/docs/ref/#/client/proc/MouseDrag

А тут еще и вызов сигнала на каждый дрег. Но видимо необходимые жертвы.

code/modules/client/client_defines.dm Outdated Show resolved Hide resolved
@volas volas added the Test Merge Candidate ПР с этим лэйблом будет или уже находится с тест мерже label Sep 11, 2024
@volas
Copy link
Member

volas commented Sep 22, 2024

Я бы может еще посмотрел по поводу шейка экрана, не очень он надоедливый при автомате?

А так, смысла держать в ТМ не вижу, так что мержу.

@volas volas merged commit 5a82681 into TauCetiStation:master Sep 22, 2024
12 checks passed
TauKitty added a commit to TauCetiStation/ClassicChangelog that referenced this pull request Sep 22, 2024
@simb11 simb11 deleted the full-auto-tg-edition branch September 25, 2024 19:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Balance Feature Новая фича Map Edit Sprites Test Merge Candidate ПР с этим лэйблом будет или уже находится с тест мерже
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants