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

More Directional windows #1476

Merged

Conversation

NightmareStalker
Copy link

@NightmareStalker NightmareStalker commented Jul 28, 2024

Описание PR

С какого-то времени нельзя больше строить в одном тайле несколько направленных окон, также при маппинге текстуры окон накладываются друг на друга, что выглядит достаточно не очень. Добавляю рабочие(не пропускают существ и газы) варианты направленных окон одним объектом.
Также изменил текстуру урановых окон под общий вид остальных. Добавил им крафты.

Медиа

image
image

Проверки

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

Изменения

🆑

  • add: Добавлены разные виды направленных окон из существующих стекол, как объекты, так и в крафт
  • tweak: Изменены текстуры уранового и укрепленного уранового стекол под общий стиль

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

github-actions bot commented Jul 28, 2024

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

Resources/Textures/Structures/Windows/directional.rsi

State Old New Status
uranium_reinforced_window Modified
uranium_window Modified

Resources/Textures/SS220/Structures/Windows/directionals/O-shaped.rsi

State Old New Status
clock_window Added
frosted_window Added
plasma_reinforced_window Added
plasma_window Added
reinforced_window Added
tinted_window Added
uranium_reinforced_window Added
uranium_window Added
window Added

Resources/Textures/SS220/Structures/Windows/directionals/U-shaped.rsi

State Old New Status
clock_window Added
frosted_window Added
plasma_reinforced_window Added
plasma_window Added
reinforced_window Added
tinted_window Added
uranium_reinforced_window Added
uranium_window Added
window Added

Resources/Textures/SS220/Structures/Windows/directionals/angular.rsi

State Old New Status
clock_window Added
frosted_window Added
plasma_reinforced_window Added
plasma_window Added
reinforced_window Added
tinted_window Added
uranium_reinforced_window Added
uranium_window Added
window Added

Resources/Textures/SS220/Structures/Windows/directionals/cracks_directional_ang.rsi

State Old New Status
DamageOverlay_12 Added
DamageOverlay_4 Added
DamageOverlay_8 Added

Resources/Textures/SS220/Structures/Windows/directionals/cracks_directional_doub.rsi

State Old New Status
DamageOverlay_12 Added
DamageOverlay_4 Added
DamageOverlay_8 Added

Resources/Textures/SS220/Structures/Windows/directionals/cracks_directional_o.rsi

State Old New Status
DamageOverlay_12 Added
DamageOverlay_4 Added
DamageOverlay_8 Added

Resources/Textures/SS220/Structures/Windows/directionals/cracks_directional_u.rsi

State Old New Status
DamageOverlay_12 Added
DamageOverlay_4 Added
DamageOverlay_8 Added

Resources/Textures/SS220/Structures/Windows/directionals/double.rsi

State Old New Status
clock_window Added
frosted_window Added
plasma_reinforced_window Added
plasma_window Added
reinforced_window Added
tinted_window Added
uranium_reinforced_window Added
uranium_window Added
window Added

Edit: diff updated after bf1d5a2

@NightmareStalker
Copy link
Author

я не понимаю, что оно хочет...
image

@Kirus59
Copy link
Collaborator

Kirus59 commented Jul 28, 2024

Стоимость стекла (строения) ниже, чем стоимость материалов, получаемых при его разборке.
Т.е. например у тебя UraniumReinforcedWindowDirectionalO имеет StaticPrice = 70.
А при разборке получаешь 4 ед. SheetRUGlass1, у каждого из которых StaticPrice = 50, следовательно 4*50 = 200
image

Стоимость стекла (строения) должна быть выше либо равна стоимости материалов, получаемых при ее разборке

@EstKemran
Copy link

Со спрайтами все отлично. В мете лишние пробелы, но это скорее косметика

Copy link

@DexlerXD DexlerXD left a comment

Choose a reason for hiding this comment

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

вот бы поправить отступы в мете...

@NightmareStalker
Copy link
Author

NightmareStalker commented Jul 30, 2024

вот бы поправить отступы в мете...

где? перед clock и plasma window?...
UPD: а, заметил... чтоб я еще хоть раз в блокноте редачил мету...

Comment on lines 26 to 33
{
"name": "clock_window",
"directions": 4
},
{
"name": "plasma_window",
"directions": 4
},

Choose a reason for hiding this comment

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

вот это тоже

Comment on lines 26 to 33
{
"name": "clock_window",
"directions": 4
},
{
"name": "plasma_window",
"directions": 4
},

Choose a reason for hiding this comment

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

и тут тоже

Comment on lines 26 to 33
{
"name": "clock_window",
"directions": 1
},
{
"name": "plasma_window",
"directions": 1
},

Choose a reason for hiding this comment

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

и тут тоже

@NightmareStalker
Copy link
Author

я вот вообще не понял, что это и как это было, учитывая, что в вск оно выглядело нормально, а тут... эээ...
image
но, вроде, поправил...

Copy link

@DexlerXD DexlerXD left a comment

Choose a reason for hiding this comment

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

image
а че по названиям?

@NightmareStalker
Copy link
Author

image а че по названиям?

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

@DexlerXD
Copy link

image а че по названиям?

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

лучше да...

@NightmareStalker
Copy link
Author

image а че по названиям?

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

лучше да...

окей

@NightmareStalker
Copy link
Author

только заметил, что нет локализации, хотя помню, что делал... бляхамуха...

@github-actions github-actions bot added the Changes: Localization Изменение затронуло файлы ".ftl" label Aug 1, 2024
@DexlerXD DexlerXD merged commit c74b602 into SerbiaStrong-220:master Aug 1, 2024
15 checks passed
@NightmareStalker NightmareStalker deleted the moredirectionalwindows branch December 27, 2024 01:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Localization Изменение затронуло файлы ".ftl" Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" Changes: Sprites Изменение затронуло файлы ".rsu / .png" Status: Needs Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants