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

Респрайт пиратского плаща #2460

Merged
merged 4 commits into from
Jan 8, 2025

Conversation

Tris0r
Copy link

@Tris0r Tris0r commented Jan 6, 2025

Описание PR

Респрайт пиратского плаща

Медиа
17-12-2024_044334

Проверки

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

Изменения
🆑 Cortez, okroshka59

  • tweak: Изменен спрайт пиратского плаща

@github-actions github-actions bot added Changes: Sprites Изменение затронуло файлы ".rsu / .png" Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" labels Jan 6, 2025
Copy link

github-actions bot commented Jan 6, 2025

RSI Diff Bot; head commit 94040d4 merging into 24b4f07
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/SS220/Clothing/Neck/Cloaks/piratecap.rsi

State Old New Status
equipped-NECK Added
icon Added
inhand-left Added
inhand-right Added

Edit: diff updated after 94040d4

Comment on lines 10 to 15
{
"name": "icon"
},
{
"name": "equipped-NECK",
"directions": 4
Copy link
Collaborator

Choose a reason for hiding this comment

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

Ну-ну-ну. git checkout master -- Resources/Textures/Clothing/Neck/Cloaks/piratecap.rsi/meta.json

Copy link
Author

Choose a reason for hiding this comment

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

Чо

Copy link
Collaborator

Choose a reason for hiding this comment

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

Откати

@@ -188,7 +188,7 @@
description: A rather fetching black pirate cloak, complete with skull motif.
components:
- type: Sprite
sprite: Clothing/Neck/Cloaks/piratecap.rsi
sprite: SS220/Clothing/Neck/Cloaks/piratecloak220.rsi #ss220 resprite
Copy link
Collaborator

Choose a reason for hiding this comment

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

Что мешает оставить название RSI как piratecap? А то что оно в нашей папке - правильно, но 220 в названии как будто лишнее

Copy link
Author

Choose a reason for hiding this comment

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

Ничего.

{
"version": 1,
"license": "CC-BY-SA-3.0",
"copyright": "made by brainfood1183 (github) for ss14, sprites in hand by PuroSlavKing (Github) and RudeyCoolLeet#3875, resprite by okroshka59 (Discord) for SS220",

Choose a reason for hiding this comment

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

Поменяй лицензию на нашу. В копирайте спрайтед бай okroshka59 (Discord) for SS220

Copy link
Author

Choose a reason for hiding this comment

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

так это же респрайт, зачем

Choose a reason for hiding this comment

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

так это же респрайт, зачем

Респрайтом он является лишь номинально, по существу, из общего у них только то, что оба спрайт плащ, оба чёрного цвета и у обоих есть череп (и то черепа разные)

Copy link
Author

Choose a reason for hiding this comment

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

А почему в топике тогда написано "респрайт" :despair:

Choose a reason for hiding this comment

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

А почему в топике тогда написано "респрайт" :despair:

Потому что по своей сути это респрайт (обновление или изменение визуала, чтобы улучшить графику или исправить ошибки).
Это полный респрайт, а не частичный. То есть, обновилась не часть, а весь спрайт

@Tris0r
Copy link
Author

Tris0r commented Jan 8, 2025

Стален лучший!

@stalengd stalengd merged commit 51b8e52 into SerbiaStrong-220:master Jan 8, 2025
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" Changes: Sprites Изменение затронуло файлы ".rsu / .png" Status: Needs Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants