Holopad #262
Holopad #262
Conversation
|
|
|
В этот раз всё нормально, ждем повторный ревью Людука, если так же всё будет нормально - то будет готово к мержу |
| @@ -886,3 +886,18 @@ var/global/list/humanoid_icon_cache = list() | |||
| return out_icon | |||
| else | |||
| return humanoid_icon_cache[icon_id] | |||
|
|
|||
| /proc/build_composite_icon_omnidir(atom/A) | |||
There was a problem hiding this comment.
У нас точно нету тысячи таких полков до тебя?
|
А до код стайла докопаться надо? |
все таки я думал картинка чуть больше будет выглядеть лучше но нет(в техах текстурка вылазит за тень)
| var/icon/composite = icon('icons/effects/effects.dmi', "icon_state"="nothing") | ||
| for(var/O in A.underlays) | ||
| var/image/I = O | ||
| composite.Blend(new/icon(I.icon, I.icon_state), ICON_OVERLAY) |
There was a problem hiding this comment.
Но это же underlay-и.
| return | ||
| var/mob/living/L = loc | ||
| if(isliving(loc) && L.client) | ||
| to_chat(loc, "<span class='warning'>Something vibrates..</span>") |
There was a problem hiding this comment.
Ну имхо написать бы где вибрирует, подумай с этим.
There was a problem hiding this comment.
Если эту вибрацию вообще никто не должен слышать кроме юзера, то зачем здесь лок? В ином случае можно использовать visible_message 🤷♂️
There was a problem hiding this comment.
А если он лежит на земле? Никто не увидит что он вибрирует?
There was a problem hiding this comment.
Под где - я имел ввиду «в кармане, в сумке», и т.д. Кстати, я тут подумал, мне кажется голопад в сумке работать не будет. У него лок- сумка
There was a problem hiding this comment.
Он же к ливинг приведен
|
|
||
| /obj/item/device/holopad/proc/placeCall() | ||
| var/list/Targets = list() | ||
| for(var/obj/item/device/holopad/H) |
There was a problem hiding this comment.
Что? А куда делось in world?
| addtimer(CALLBACK(src, .proc/update_holo), 1) | ||
|
|
||
| if(isliving(loc)) | ||
| to_chat(loc, "Connection established") |
There was a problem hiding this comment.
Круто. Может лучше типо Голопад говорит, «установлено подключение»
|
|
||
| /obj/item/device/holopad/atom_init() | ||
| . = ..() | ||
| uniq_id = rand(00000, 999999) |
There was a problem hiding this comment.
Я бы ещё тогда проверил есть ли такие же голопады. А то оно не уникально.
| id = rand(1000, 9999) | ||
| name = "[initial(name)] [id]" | ||
|
|
||
| /obj/item/device/holopad/Destroy() |
There was a problem hiding this comment.
А зачем переписывать прок если он всего-лишь вызывает родителя.
| hologram.pixel_x = ((L.dir&4)?32:((L.dir&8)?-32:0)) | ||
| hologram.pixel_y = ((L.dir&1)?32:((L.dir&2)?-32:0)) | ||
| else if(isturf(loc)) | ||
| hologram.dir = 2 |
There was a problem hiding this comment.
Дефайны есть
| if(!abonent.hologram) | ||
| abonent.hologram = new() | ||
| abonent.hologram.name = "Hologram [sanitize(id)]" | ||
| abonent.hologram.layer = 5 |
| addtimer(CALLBACK(src, .proc/update_holo_pos), 2) | ||
|
|
||
|
|
||
| /obj/item/device/holopad/attack_self(mob/user as mob) |
There was a problem hiding this comment.
As mob зачем?
There was a problem hiding this comment.
У нас не используется как приведение : так и преобразование as mob, as turf, as obj и прочее такое, оно тут не нужно
|
|
||
| /obj/item/device/holopad/hear_talk(mob/living/M, text, datum/language/speaking) | ||
| if(call_state == CALL_IN_CALL) | ||
| abonent.receive(text,M==loc) |
There was a problem hiding this comment.
Мне нравиться как голограммы ещё и переводят с других языков - на нужный.
Добавляет в игру голопад, с возможностью связи с другими членами персонала по средствам аудио и видео связи.
🆑 TermService