Cinematic Log

[Moderator]

20:36, 27th Jun 2014
PurgeandFire: (old) Review (minor fix, PM me when ready):
http://www.hiveworkshop.com/forums/2547383-post10.html

Approved!

 

[edo494]

campaigns can compile vJass with the "not currently working(for me at least)" campaign builder/compiler, or if you compile the map and import it in.

However this is not really good thing. I can just use DisplayTimedTextToPlayer(GetLocalPlayer(), 0, 0, 0.1, message) and after that the transmission.

I mean, this is not bad thing and I see the use for it, but you hook a potentionally non existant function. You are doing the hook, so you are the one responsible for providing the function msg for the user.

 

[GywGod133]

its look useful if GUI-friendly :/

 

[Chaosy]

The thing is, I can't add more GUI.

The hook is needed, and the first two lines in the "GUI function" is also needed. You can't replace it. The local variables is there so you can use the last two actions in GUI. Else I'd just use the alternative trigger, it's a little shorter.

@edo
I know that it is working, you can extract the map > save it > import it again but I said it was troublesome, not impossible Well it's existant if you copy the trigger

 

[edo494]

I understand that hook here is necessary unless you want people to call custom script(I assume this is aimed at GUIers, which it most likely is), but you could also provide the msg function not in custom script but with the hook alltogether. Because if they forget to copy the function....oops some weird magic error goes off

also you should state that this requires JNGP to work

 

[Chaosy]

I agree, the best option would indeed to merge them together I'll actually change the alternative trigger to make that. Will update in a minute.

 

[edo494]

a bit slicker function:

function msg takes force toForce, unit whichUnit, string unitName, sound soundHandle, string message, integer timeType, real timeVal, boolean wait returns nothing
    call DisplayTimedTextToPlayer(GetLocalPlayer(), 0, 0, 1, message + " - " + GetUnitName(whichUnit))
    //for the sake of one liners
    call ClearTextMessagesBJ( toForce )
endfunction

a bit more efficient solution, but whichever is fine

 

[Rheiko]

that Category doesn't mention vJass though

 

[Chaosy]

I wasn't able to select both vjass and gui.

 

[PurgeandFire]

It is good, but you should edit your description to include "Requires JassNewGenPack (or JassHelper)" and note that "This will clear all messages displayed on the screen each time a transmission occurs". Some cinematics will use "Game - Display text message", so they should be aware that this system will automatically clear the screen of them.