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

Sweeps and spins. #4458

Merged

Conversation

LudwigVonChesterfield
Copy link
Contributor

@LudwigVonChesterfield LudwigVonChesterfield commented Jan 6, 2020

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

basically, #3755

Теперь кроме как шваброй можно махать-пихать различными вещами, а так же:

Щитами можно махать, и если кто-то упрётся в твой машущийся щит своим махком, его застанит, опрокинет.

Если имея что-то, что может пихать ты ударишь по руке с щитом - ты можешь опрокинуть щитоносца.

Крутить вокруг себя можно так же на колёсико мышки.

По конкретным предметам:

  • Топором можно крутиться и махать, но он довольно медленный. Он упирается об щит, и стены.

  • Копьём можно пихать, и даже атаковать через один тайл(Или притягивать и атаковать).

  • Энерго-милишные оружия можно махать и крутить, и они не упираются ни об что кроме *столов, мусоро, полочек.

  • Двойную саблю можно крутить.

2019-08-06_11-32-16

  • Тростью можно притягивать и отталкивать от людей, но не бить через тайл.

  • Если щитом пихнуть куклу об что-то, что мешает ей ходить, она упадёт

  • При махании по самому себе попасть невозможно.

Откатывает #3974

Есть возможность эти все финты проворачивать с помощью мышки, если вы выбрали тайл рядом с вами, и начали его перетаскивать на тайл не рядом с вами - вы отпихнёте.

Если перетащили тайл не рядом с вами, на тот который рядом - притянете.

Если все "выделенные" вами тайлы будут рядом с вами - вы сделаете свайп по ним.

Если вы проведёте мышкой через себя из одной стороны в противоположную - вы начнётся крутиться.

Подсказки к всему этому в игре:

image

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

Добавит чуть больше механа нашей боёвочке.

🆑 Luduk

  • rscadd[link]: Крутить, махать, пихать можно не только шваброй.
  • rscadd: Пихание щитом кого-то об стену - опрокинет их.
  • tweak: Швабра теперь засовывается в тележку не альт-кликом а перетаскиванием её на тележку.

@TauKitty
Copy link
Contributor

TauKitty commented Jan 6, 2020

Changelog status: ✔️

@TauKitty TauKitty added Feature Новая фича Tweak labels Jan 6, 2020
Comment on lines +207 to +209
var/obj/item/I = get_active_hand()
if(I && next_move <= world.time && !incapacitated() && (SEND_SIGNAL(I, COMSIG_ITEM_MIDDLECLICKWITH, A, src) & COMSIG_ITEM_CANCEL_CLICKWITH))
return
Copy link
Contributor

Choose a reason for hiding this comment

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

Мне кажется этот иф (и далее так же) длинный и некрасивый и тяжело читается.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Я не знаю как сделать его красивше :(. Вставлять уровни вложенности в иф не очень хочется, как и выносить проверки в какие-то отдельные переменные до ифа.

Comment on lines +477 to +478
SCB.can_sweep = TRUE
SCB.can_spin = TRUE
Copy link
Contributor

Choose a reason for hiding this comment

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

Что-то я сомневаюсь что топориком ботаника можно свипать и спинать.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Формализируй пожалуйста чем могжно свипать и спинать, потому-что я вот решил что топориком ботаника можно.

Если вдруг это пройдёт, можно абстрагировать от конкретных предметов, и смотреть по w_class можно ли чем-то свипать или нет, а пока...

Copy link
Contributor

Choose a reason for hiding this comment

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

Спинать можно чем-то продолговатым, свипать чем-то длинным и широким.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Я в отдельном ПР-е пройдусь и сделаю вещам длину, ширину, и трали-вали для вот этого всего. Пока-что топориком можно будет вращать просто потому-что это смешно.

@SpaiR
Copy link
Member

SpaiR commented Mar 1, 2020

Думаю можно больше мучать.

@SpaiR SpaiR closed this Mar 1, 2020
@SpaiR
Copy link
Member

SpaiR commented Apr 18, 2020

Простите, был не прав. Проблемы признаю, перед автором извиняюсь.

@SpaiR SpaiR reopened this Apr 18, 2020
@stale
Copy link

stale bot commented May 19, 2020

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

@stale stale bot added the Stalled PR label May 19, 2020
- Removed all the "default_" thingy procs, in favour of using a proc, that calls the callback if neccesary.
- Removed the hit_on_harm_blahblah vars, in favour of consistency.
@stale stale bot removed the Stalled PR label May 19, 2020
@LudwigVonChesterfield
Copy link
Contributor Author

В недалеком будущем(ТМ) после этого ПР-а хочу внести такие изменения в "боёвку":

  • Выпилить рандом из переломов, отрубаний конечностей.
  • Сделать больше различий для существующих "видов" оружия - тупое, колющее, лезвие:

    Лезвия которым машут будет отрезать конечности гораздо проще, в то время как лезвие которым тычут будет работать как колющее. Но не будут дробить кости. Взмахи лезвиями не будут станить от ударов по стене.
    Колющие будут наносить урон внутренним органам и чаще вызывать внутренние кровотечения. Но не будут дробить кости
    Тупые не будут вызывать кровотечения, но будуть дробить кости. Замахиваясь тупым - урон от тупого оружия будет сильно увеличен. Но тупые взмахи будут станить при ударе по стене.

@LudwigVonChesterfield LudwigVonChesterfield added the Test Merge Candidate ПР с этим лэйблом будет или уже находится с тест мерже label May 20, 2020
@LudwigVonChesterfield
Copy link
Contributor Author

Оверлей предмета ниже стола.

@Motzord
Copy link
Contributor

Motzord commented May 20, 2020

Влияет ли как-то прицел, на удары во время свайпа/спина?

Было бы очень круто крутится даблсвором на 360 снося всем головы

@LudwigVonChesterfield
Copy link
Contributor Author

Удары во время свайпа/спина ничем не отличаются от обычных ударов, просто не имеют КД. Интент, прицел, и любые другие модификаторы - учитываются.

@Motzord
Copy link
Contributor

Motzord commented May 20, 2020

Не хватает фичи, чтобы при прицеливании по ногам отрывало/дамажило обе ноги. Дабл есворд так не умеет почему-то, хотя свайп пролетает сквозь культяпки всех кукол

@LudwigVonChesterfield LudwigVonChesterfield merged commit 542d7a9 into TauCetiStation:master May 23, 2020
TauKitty added a commit to TauCetiStation/ClassicChangelog that referenced this pull request May 23, 2020
KIBORG04 pushed a commit to KIBORG04/TauBORG that referenced this pull request May 25, 2020
* Placing all click signals.

* Swiping component. Replacing old mop behaviour.

* Making a bunch of stuff swipable.

* Replacing a proc with a ton' of named arguments with a builder datum

* Rebranding. hit_on_harm removal.

- Removed all the "default_" thingy procs, in favour of using a proc, that calls the callback if neccesary.
- Removed the hit_on_harm_blahblah vars, in favour of consistency.

* async sweep calling

* slight improvements

* a tip, even one unnecesary var removed

* defines and stuff

* swipe effect layer

* Motzord issue fix

* mop cleaning everything
Zap-zapper pushed a commit to Zap-zapper/TauCetiClassic that referenced this pull request Aug 6, 2020
* Placing all click signals.

* Swiping component. Replacing old mop behaviour.

* Making a bunch of stuff swipable.

* Replacing a proc with a ton' of named arguments with a builder datum

* Rebranding. hit_on_harm removal.

- Removed all the "default_" thingy procs, in favour of using a proc, that calls the callback if neccesary.
- Removed the hit_on_harm_blahblah vars, in favour of consistency.

* async sweep calling

* slight improvements

* a tip, even one unnecesary var removed

* defines and stuff

* swipe effect layer

* Motzord issue fix

* mop cleaning everything
@LudwigVonChesterfield LudwigVonChesterfield deleted the spin_and_slash branch November 25, 2021 22:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Новая фича Test Merge Candidate ПР с этим лэйблом будет или уже находится с тест мерже Tweak
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants