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

3 Глобуса #13646

Merged
merged 6 commits into from
Nov 27, 2024
Merged

Conversation

DarthSidiousPalpatine
Copy link
Contributor

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

Добавил 3 глобуса. Вот они слева направо: Земля, Яргон-4, Венера, Марс (уже был).
image

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

Начинаю парад атомизации своих непропущенных ПРов.

Авторство

AndreyGysev

Чеинжлог

🆑 AndreyGysev

  • rscadd: Добавил 3 новых глобуса.

Добавил 3 глобуса
@TauKitty
Copy link
Contributor

Changelog status: ✔️

@TauKitty TauKitty added Feature Новая фича Sprites labels Nov 25, 2024
@TauKitty TauKitty requested a review from a team November 25, 2024 05:22
@Ayaksblack
Copy link
Contributor

Божечки

@DarthSidiousPalpatine
Copy link
Contributor Author

Божечки

Что такое?

@Ayaksblack
Copy link
Contributor

Царь вернулса

@DarthSidiousPalpatine
Copy link
Contributor Author

Царь вернулса

Царь говнокода и кодерспрайта 😸

Copy link
Contributor

@WatlerJe WatlerJe left a comment

Choose a reason for hiding this comment

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

Для нарративного дизайна было бы неплохо еще добавить глобусы родных планет для игровых рас.

@DarthSidiousPalpatine
Copy link
Contributor Author

Для нарративного дизайна было бы неплохо еще добавить глобусы родных планет для игровых рас.

Ну Яргон-4 это планета скреллов. Оки, осталось добавить Могес для унатхов и таярову планетку.
Обновлю тогда ПР? Или отдельным? Отдельным наверн, не хочу портить ПР. Отдельный будет... Туда мб ещё "карту найденного гештальта дион в секторе 726-Б или тип того. Да, отдельный ПР будет.

@TauKitty TauKitty requested a review from a team November 26, 2024 13:30
Обновил всё что нужно.
icon = 'icons/obj/items.dmi'
icon_state = "globe"

/obj/item/globe/mars
Copy link
Member

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.

всм

Copy link
Member

Choose a reason for hiding this comment

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

image

Ты жесткий тип там задал и смотришь на жетский тип

Copy link
Contributor Author

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.

ща верну как было крч)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Киборг, ты что, хочешь чтобы я в этом ПРе всё таки это сделал? Просто ПР про 3 глобуса. Не хочу в нём фиксить полку. Можно мерж 3 глобуса и я заливаю ещё 3 глобуса для "нарративного дизайна"? :3

Copy link
Member

Choose a reason for hiding this comment

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

Тогда мне будет нужен еще один пр, где ты глобусы пихаешь в один тайп, а у полки пишешь is_type_in_list

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Киборг, а вопрос такой, у меня по коду потом надо выдернуть оффсеты по типу. И как я их выдерну если у меня тип в списке будет не совпадать с типом предмета?
image

Copy link
Contributor Author

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.

Аааа, сорян, я уже нашёл что там ниже есть прок get_type_in_list, удобно)

@DarthSidiousPalpatine
Copy link
Contributor Author

Так, всё, ПР обновлён, исправлен конфликт иконок с бюстами, ну и добавлены смещения для полки.

@KIBORG04 KIBORG04 merged commit 68df28e into TauCetiStation:master Nov 27, 2024
12 checks passed
TauKitty added a commit to TauCetiStation/ClassicChangelog that referenced this pull request Nov 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Новая фича Sprites
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants