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

Выправил все шаттеры #12988

Closed
wants to merge 6 commits into from

Conversation

AzzyDreemurr13
Copy link
Contributor

@AzzyDreemurr13 AzzyDreemurr13 commented Mar 23, 2024

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

Проверил все карты и на всех исправил шаттеры
Затронуты карты: Исход, Астероид (стандарт), ЦК(Велосити), Прометей, Гамма, Строечка, Фалкон, Дельта

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

fixes #12982

Авторство

Сидел все полчаса в StrongDMM и проверял все шаттеры на каждой карте - Я

Чеинжлог

🆑 Azzy.Dreemurr

  • bugfix: На всех картах Шаттеры были выправлены в вертикальный вид

@TauKitty
Copy link
Contributor

Changelog status: ✔️

@TauKitty TauKitty requested a review from a team March 23, 2024 20:27
@Deahaka
Copy link
Contributor

Deahaka commented Mar 23, 2024

Я прочитал и удивился, проверил Дельту и удивился ещё, посмотрел на коддифф и продолжил быть в удивлении, посмотрел в дифф мапбота по первым двум картам и немного разудивился, посмотрел на центкомм карту ниже и не понял вообще ничего.... Можно скриншоты "было > стало" и обвести в кружочек что меняли на что?
Хотя бы ДВА

@AzzyDreemurr13
Copy link
Contributor Author

Дехака, Я Шаттеры повернул вертикально на всех картах.

@Deahaka
Copy link
Contributor

Deahaka commented Mar 23, 2024

Может у меня конечно не актуальная версия карты...
1000_F_465668832_jVngGSvtVp3HFBIZ8iF9caqKejfpa9s5

@Deahaka , моя ошибка, вот фиксы для дельты
@AzzyDreemurr13
Copy link
Contributor Author

@Deahaka , всё залил для Дельты тоже фиксы (описание сейчас исправлю)

Исправил синди шаттеры на цк
@volas
Copy link
Member

volas commented Mar 25, 2024

Почему-то у тебя горизонтальные либо не пофикшены, либо те которые были правильные - переделаны на не правильно.
Было правильно, в соответствии с проекцией (велосити):
Снимок экрана от 2024-03-25 09-33-03
Стало не правильно, против проекции:
Снимок экрана от 2024-03-25 09-32-53

Мне кажется, тут всё еще какой-то мискомуникейшен. Я в дискорде писал, проще сам спрайт перевернуть, так как его перевернули неправильно в https://github.com/TauCetiStation/TauCetiClassic/pull/12607/checks (открыть икон диффбот). Еще раз - их не переворачивали на карте, чтоб они стали не правильно. Им неправильно повернули спрайт.

@volas
Copy link
Member

volas commented Mar 25, 2024

но неправильность dir-а в спрайте не отменяет возможность того, что где-то на карте тоже неправильно повернуты (и минус на минус даёт плюс)

@AzzyDreemurr13
Copy link
Contributor Author

AzzyDreemurr13 commented Mar 28, 2024

Я наверное в воскресенье закрою ПР если не пойму что от меня хотел Волас, ибо сижу и невдупляю, что не так и почему не нравится, что я все (с шаттерами на велосити согласен, верну) повернул в одну сторону.

@volas
Copy link
Member

volas commented Mar 28, 2024

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

повернул в одну сторону

Еще раз попробую - у них две стороны, и обе должны использоваться. Одна для горизонтальных, другая для вертикальных. Повернутые в сторону юзера, в соответствии со своей проекцией. И проекция там точно такая же, как и у дверей и стен, тупо смотри на двери и поворачивай точно так же.

@AzzyDreemurr13
Copy link
Contributor Author

Пока повещаю табличку, после работы продолжу

Copy link

Данный ПР автоматически отмечен как застоявшийся по причине длительного отсутствия обновлений. Он будет закрыт через 7 дней, если никакой активности не будет проявлено. Если вы считаете, что ПР еще актуален, или что я (злобный робот) пристаю к вам зря - просто напишите любой комментарий. Спасибо за ваш вклад.

Copy link

ПР закрыт из-за длительного отсуствия активности. Для переоткрытия ПРа, пожалуйста, обратитесь к кому-либо из мейнтейнеров. Вы можете призвать их в комментарии слапнув @TauCetiStation/maintainers.

@github-actions github-actions bot closed this Apr 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Шторки повернуты не в ту сторону
4 participants