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

Item Offer Bind #2423

Merged
merged 3 commits into from
Dec 29, 2024
Merged

Conversation

canvaswalker
Copy link

Описание PR

Добавлен бинд на передачу предмета

(p.s После того как я вернулся в игру я заметил, что люди юзают итемоффер от Артура, собственно теперь на него появился удобный бинд, чтобы каждый раз не открывать контекстное меню)

Медиа

_

Проверки

  • PR полностью завершён и мне не нужна помощь чтобы его закончить.
  • Я ознакомился с наставлениями по работе с репозиторием и следовал им при создании PR'а.
  • Я внимательно просмотрел все свои изменения и багов в них не нашёл.
  • Я запускал локальный сервер со своими изменениями и всё протестировал.
  • Я добавил скриншот/видео демонстрации PR в игре, или этот PR этого не требует.

Изменения

🆑 canvaswalker

  • add: Добавлен бинд в настройки на передачу предмета! (по умолчанию Shift + R)

@github-actions github-actions bot added Changes: UI Изменение затронуло файлы ".xaml" Changes: Localization Изменение затронуло файлы ".ftl" Changes: C# Изменение затронуло файлы ".cs" labels Dec 29, 2024
itemReceiver.Giver = user;
itemReceiver.Item = item;
_alerts.ShowAlert(target, ItemOfferAlert);
_popupSystem.PopupEntity($"{Name(user)} протягивает {Name(item.Value)} {Name(target)}", user);
Copy link
Collaborator

Choose a reason for hiding this comment

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

Сделай через локаль

@Kirus59 Kirus59 self-assigned this Dec 29, 2024
.Register<ItemOfferSystem>();
}

private bool HandleItemOffer(in PointerInputCmdHandler.PointerInputCmdArgs args)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Лучше назвать как нибудь по типу HandleItemOfferKey

@@ -10,6 +10,7 @@ public static class ContentKeyFunctions
public static readonly BoundKeyFunction ActivateItemInWorld = "ActivateItemInWorld";
public static readonly BoundKeyFunction AltActivateItemInWorld = "AltActivateItemInWorld";
public static readonly BoundKeyFunction Drop = "Drop";
public static readonly BoundKeyFunction ItemOffer = "ItemOffer"; //SS220 ItemOffer bind
Copy link
Collaborator

Choose a reason for hiding this comment

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

Перенести в KeyFunctions220

@UrPrice
Copy link
Collaborator

UrPrice commented Dec 29, 2024

first time contributor xdd

@canvaswalker
Copy link
Author

я новеньки

@21Melkuu
Copy link

погодите это реально?

@stalengd stalengd merged commit 1f0d962 into SerbiaStrong-220:master Dec 29, 2024
14 checks passed
@canvaswalker canvaswalker deleted the hamsahamsahamsa branch December 29, 2024 17:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: C# Изменение затронуло файлы ".cs" Changes: Localization Изменение затронуло файлы ".ftl" Changes: UI Изменение затронуло файлы ".xaml" Status: Needs Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants