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.

Sign up for GitHub

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

Jump to bottom

Turtlenecks for command #2276

Merged
merged 5 commits into from
Jun 18, 2024
Merged

Turtlenecks for command #2276

merged 5 commits into from
Jun 18, 2024

Conversation

Deenkaide
Copy link
Contributor

@Deenkaide Deenkaide commented Jun 17, 2024 •
edited
Loading

Описание PR

ПР на добавление недостающих водолазок глав и респрайт имеющихся.
Водолазки ГСБ и НРа перенесены в папку Корвакса, на их местах респрайты (новые костюмы)

Медиа
Водолазки:
Кэп
ГП
ГСБ
КМ
ГВ
НР
СИ

Костюмы ГСБ и НРа
РНД
ГСБАльт

Проверки

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

Изменения
🆑 Deenkaide

  • add: Водолазки для ГП и Капитана
  • add: Чёрный костюм для ГСБ
  • add: Фиолетовый костюм для НРа
  • tweak: Перенесены и исправлены водолазки ГСБ и НРа
  • tweak: Водолазки СИ и ГВ перерисованы
  • tweak: Водолазка КМа исправлена в руке

@Deenkaide Deenkaide requested a review from Morb0 as a code owner June 17, 2024 14:03
@github-actions github-actions bot added Status: Needs Review This PR requires new reviews before it can be merged Changes: No C# Code has not been changed Changes: Sprites Should be reviewed or fixed by people who are knowledgeable with spriting or visual design labels Jun 17, 2024
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Jun 17, 2024 •
edited
Loading

RSI Diff Bot; head commit ba1c616 merging into 5bab884
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/Clothing/Uniforms/Jumpskirt/ce_turtle.rsi

State Old New Status
equipped-INNERCLOTHING Modified
icon Modified
inhand-left Modified
inhand-right Modified

Resources/Textures/Clothing/Uniforms/Jumpskirt/cmo_turtle.rsi

State Old New Status
equipped-INNERCLOTHING Modified
icon Modified
inhand-left Modified
inhand-right Modified

Resources/Textures/Clothing/Uniforms/Jumpskirt/hos_alt.rsi

State Old New Status
equipped-INNERCLOTHING Modified
icon Modified
inhand-left Modified
inhand-right Modified

Resources/Textures/Clothing/Uniforms/Jumpskirt/qmturtleskirt.rsi

State Old New Status
inhand-left Modified
inhand-right Modified

Resources/Textures/Clothing/Uniforms/Jumpskirt/rnd.rsi

State Old New Status
equipped-INNERCLOTHING Modified
icon Modified
inhand-left Modified
inhand-right Modified

Resources/Textures/Clothing/Uniforms/Jumpsuit/ce_turtle.rsi

State Old New Status
equipped-INNERCLOTHING-reptilian Modified
equipped-INNERCLOTHING Modified
icon Modified
inhand-left Modified
inhand-right Modified

Resources/Textures/Clothing/Uniforms/Jumpsuit/cmo_turtle.rsi

State Old New Status
equipped-INNERCLOTHING Modified
icon Modified
inhand-left Modified
inhand-right Modified
equipped-INNERCLOTHING-reptilian Added

Resources/Textures/Clothing/Uniforms/Jumpsuit/hos_alt.rsi

State Old New Status
equipped-INNERCLOTHING-reptilian Modified
equipped-INNERCLOTHING Modified
icon Modified
inhand-left Modified
inhand-right Modified

Resources/Textures/Clothing/Uniforms/Jumpsuit/qmturtle.rsi

State Old New Status
equipped-INNERCLOTHING-reptilian Modified
inhand-left Modified
inhand-right Modified

Resources/Textures/Clothing/Uniforms/Jumpsuit/rnd.rsi

State Old New Status
equipped-INNERCLOTHING-reptilian Modified
equipped-INNERCLOTHING Modified
icon Modified
inhand-left Modified
inhand-right Modified

Resources/Textures/Corvax/Clothing/Uniforms/Jumpskirt/cap_turtleneck.rsi

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

Resources/Textures/Corvax/Clothing/Uniforms/Jumpskirt/hop_turtleneck.rsi

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

Resources/Textures/Corvax/Clothing/Uniforms/Jumpskirt/hos_black.rsi

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

Resources/Textures/Corvax/Clothing/Uniforms/Jumpskirt/rd_formal.rsi

State Old New Status
equipped-INNERCLOTHING-monkey Added
equipped-INNERCLOTHING Added
icon Added
inhand-left Added
inhand-right Added

Resources/Textures/Corvax/Clothing/Uniforms/Jumpsuit/cap_turtleneck.rsi

State Old New Status
equipped-INNERCLOTHING-reptilian Added
equipped-INNERCLOTHING Added
icon Added
inhand-left Added
inhand-right Added

Resources/Textures/Corvax/Clothing/Uniforms/Jumpsuit/hop_turtleneck.rsi

State Old New Status
equipped-INNERCLOTHING-reptilian Added
equipped-INNERCLOTHING Added
icon Added
inhand-left Added
inhand-right Added

Resources/Textures/Corvax/Clothing/Uniforms/Jumpsuit/hos_black.rsi

State Old New Status
equipped-INNERCLOTHING-reptilian Added
equipped-INNERCLOTHING Added
icon Added
inhand-left Added
inhand-right Added

Resources/Textures/Corvax/Clothing/Uniforms/Jumpsuit/rd_formal.rsi

State Old New Status
equipped-INNERCLOTHING-reptilian Added
equipped-INNERCLOTHING Added
icon Added
inhand-left Added
inhand-right Added

Edit: diff updated after ba1c616

@lzk228
Copy link
Collaborator

lzk228 commented Jun 17, 2024

по поводу водолазки смо, я бы взял с #2190 , там низ более тёмный
но пусть решает @SonicHDC

@Deenkaide
Copy link
Contributor Author

Когда-нибудь я научусь нормально пользоваться гитом...
ПР не закончен ещё, но хочу услышать критику, если есть такая)

@Deenkaide
Новые водолазки
a3b179c 
Нужно будет прописать копирайт и перевод
@Deenkaide
Copy link
Contributor Author

Добавил новые предметы в папку Корвакса.
Нужно понять какой копирайт прописывать в моём случае...
А также найти где переводить их

@lzk228
Copy link
Collaborator

lzk228 commented Jun 17, 2024

я трясусь, чтобы не сказать легендарную фразу, но я так люблю водолазки в этой ебучей игре

@Deenkaide
Copy link
Contributor Author

я трясусь, чтобы не сказать легендарную фразу, но я так люблю водолазки в этой ебучей игре

Оффы такое не примут, скорее всего, а шакалить до их уровня... у меня руки не поднимутся)

lzk228
lzk228 requested changes Jun 17, 2024
View reviewed changes
Copy link
Collaborator

@lzk228 lzk228 left a comment

Choose a reason for hiding this comment

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

  • для хоса и рд оставь респрайты, а именно отдельные костюмы вынеси в корвакс

  • ещё было бы хорошо сделать их в лодауты, можешь попробовать, если не сможешь, то я сделаю

  • по поводу копирайта можешь просто написать Created by Deenkaide (github)

Resources/Prototypes/Corvax/Entities/Clothing/Uniforms/jumpskirts.yml Outdated Show resolved Hide resolved
@Deenkaide
Copy link
Contributor Author

  • для хоса и рд оставь респрайты, а именно отдельные костюмы вынеси в корвакс

  • ещё было бы хорошо сделать их в лодауты, можешь попробовать, если не сможешь, то я сделаю

  • по поводу копирайта можешь просто написать Created by Deenkaide (github)

  1. так и сделал
  2. я с лодаутами работал и планировал добавить их туда.
    Плюс надо в принтер засунуть, наверное
  3. Мне уже отписали по этому поводу. Писать, что я создал мне совесть не позволит, ибо во многом просто перекрасы КМа и ВУ. Но я где возможно восстановлю прошлый копирайт и добавлю себя, а вот с новыми вещами надо подумать

@lzk228
Copy link
Collaborator

lzk228 commented Jun 17, 2024

  1. так и сделал

вот эти костюмчики в корвакс
image
чтобы водолазки на месте остались, а костюмы отдельно
image

2. Плюс надо в принтер засунуть, наверное

да

3. Мне уже отписали по этому поводу. Писать, что я создал мне совесть не позволит, ибо во многом просто перекрасы КМа и ВУ. Но я где возможно восстановлю прошлый копирайт и добавлю себя, а вот с новыми вещами надо подумать

тогда напиши Resprited from блаблабла by NAME

@Deenkaide
Copy link
Contributor Author

  1. так и сделал

вот эти костюмчики в корвакс

Понял. Я просто увидел у оффов текстурки этих "водолазок" и названия папок (описания предметов я не читал) и подумал, что в нашем репозитории их просто назвали водолазками, тем более респрайты у нас. Поэтому эти костюмчик на их место запихнул.

Сейчас специально проверил и оказалось, что у оффов это тоже водолазки...
Поменяю тогда уж, но мне не нравится нейминг папок у оффов... запутанность создают. Потому что понять, что hos_alt и rnd - это водолазки сложно без чтения кода в другой папке и документе.

Надо будет запустить к ним в репозиторий руки))

@Deenkaide
Copy link
Contributor Author

Вроде местами всё поменял, надеюсь ничего не пропустил

@lzk228 lzk228 requested a review from SonicHDC June 18, 2024 09:15
@Deenkaide
Copy link
Contributor Author

Чего... мне Аутпост тест завалили?

@SonicHDC
Copy link
Collaborator

Шутки за 300

@SonicHDC SonicHDC merged commit c36ccb0 into space-syndicate:master Jun 18, 2024
15 of 16 checks passed
@Deenkaide
Copy link
Contributor Author

Аээээ, а лодауты и переводы)

@lzk228 lzk228 mentioned this pull request Jun 18, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lzk228 lzk228 lzk228 approved these changes

@Morb0 Morb0 Awaiting requested review from Morb0 Morb0 is a code owner

@SonicHDC SonicHDC Awaiting requested review from SonicHDC

Assignees
No one assigned
Labels
Changes: No C# Code has not been changed Changes: Sprites Should be reviewed or fixed by people who are knowledgeable with spriting or visual design Status: Needs Review This PR requires new reviews before it can be merged
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Deenkaide @lzk228 @SonicHDC