[DNM] 516 relay screen_loc

[NinjaPikachuska]

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

Breaking change: Previously, render_source replaced an icon with the render_target by anchoring to the center of the icon. This was stupid, because it created the need for hacky workarounds and it produced a ton of annoying bugs, but it was also unintuitive. Now, when render_source replaces an icon it anchors to the lower left, just as if you were simply using a different icon. (Lummox JR)

реле теперь крепляется в нужной точке

при каждом моб логине по всем реле проходится и выставляется позиция в зависимости от версии бьенда игрока, чтобы оставить 515 совместимость и исключить случаи перезахода пользователя с разными версиями
это не очень правильно, но с релизом 516 можно будет удалить это

но сейчас это только для тмов в связке с #13712

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

игра будет работать на 516

Авторство

Чеинжлог

[TauKitty]

Changelog status: 🔘 No changelog

[volas]

тут проблема с камерами, у всех /atom/movable/screen/map_view сбрасывается плейн на уровень игры, т.к. byond использует переменную одновременно и для координат, и для экрана.

TauCetiClassic/code/_onclick/hud/rendering/plane_master.dm

Lines 31 to 34 in b17704f

	if(map_view)
	assigned_map = map_view
	// don't use fixed size, it can break map_view scaling
	screen_loc = "[map_view]:1,1"

думаю костыль с проверкой версии клиента можно было бы вытащить в /atom/movable/screen/plane_master/atom_init()

[NinjaPikachuska]

атом_инит не совсем подходит, потому что плейны (за исключением камер) всего раз создаются и хранятся, а игрок может зайти с 515 и перезайти с 516

закоммитил фикс, реле теперь должно нормально на камерах сидеть