Roll20/roll20-character-sheets

[Mongoose Traveller] Computers skill show/hide broken, Deception formula wrong

tresi opened this issue · 1 comments

tresi commented

In the Mongoose Traveller character sheet, there's a button to "Select which Skills to show". Tapping that reveals the skills list. Here is the bug: tapping Computers from this list fails to show/hide Computers in the character sheet list below. As such, Computers isn't usable as a built-in skill.

Relatedly, while I was looking at this, I noticed that the formula used for the Deception skill accidentally uses the value of the Computers skill instead. This looks like an easy fix, although I haven't tested it.

Mongoose_Traveller/MongooseTraveller.html, line 810:

-                            <th><button name="roll_Deception" type="roll" value='&{template:skill-general} {{skill= Deception }} {{character= @{character_name} }} {{roll= [[(2d6 + [[@{skilltotal-Computers}]] + ?{Modifier|0})]] }}' ><div class="sheet-button_header">Deception</div></button></th>
+                            <th><button name="roll_Deception" type="roll" value='&{template:skill-general} {{skill= Deception }} {{character= @{character_name} }} {{roll= [[(2d6 + [[@{skilltotal-Deception}]] + ?{Modifier|0})]] }}' ><div class="sheet-button_header">Deception</div></button></th>
tresi commented

Thanks!