Repair & medical object attributes for eden editor

[SzwedzikPL]

http://i.imgur.com/LXfNjDB.jpg //units
http://i.imgur.com/ad0qBLq.jpg //vehicles
http://i.imgur.com/j6S7Tmo.jpg //props

[jokoho48]

👍

[jonpas]

Should we use a tag in those settings to set them apart from vanilla ones and make it clear it's an ACE setting? Eg. [ACE] Is Medic

[jonpas]

Probably meant to be Whether and object.

[Daishiknyte]

Jonpas is correct. Any added commands, menus, etc., should clearly show the mod they're associated with.

[nicolasbadano]

👍 Nice. I agree that some indication that those settings are ace would be great. Maybe:
ACE: Is Medic:

[kymckay]

I feel like having categories for ace attributes would make more sense than using the special states category. Also eliminates the need to repetitively label each attribute with ace in the name.

[commy2]

^ this

[commy2]

lower case is

[SzwedzikPL]

I've added ace category and used common_Options stringtable

http://i.imgur.com/Z1tI6b3.jpg //units
http://i.imgur.com/KRwpL0Y.jpg //vehicles
http://i.imgur.com/CNoeQJe.jpg //props

[gienkov]

👍
10.02.2016 01:18 "SzwedzikPL" notifications@github.com napisał(a):

I've added ace category and used common_Options stringtable

http://i.imgur.com/Z1tI6b3.jpg //units
http://i.imgur.com/KRwpL0Y.jpg //vehicles
http://i.imgur.com/CNoeQJe.jpg //props

—
Reply to this email directly or view it on GitHub
#3288 (comment).

[nicolasbadano]

Looks good @SzwedzikPL

[nicolasbadano]

👍 For merging this for 3.5.0

[PabstMirror]

@SzwedzikPL
These do not work in MP, only the server has the correct settings.

        class AttributesBase {
            class Default {
                expression = "_this setVariable ['%%s',_value,true];";
            };

The default uses the broadcast flag whereas these do not.
We might want to just use inheritance instead of redefining expression.
As far as I can tell only the server executes the code.

Also see notes in #3339