Use ColorIndex macro instead of magic numbers
Simplify calls to trap_Rocket_SetInnerRML

SetInnerRML sets the text inside the current element
Therefore, surrounding things with <span> or <div> is unnecessary because the parent element can be styled normally using RCSS
Every extra element added in the SetInnerHTML call will add to the work that libRocket must do on the SetInnerHTML call