So far, things are looking pretty good. I don't know the game, so take whatever I say with a grain of salt... Generally speaking, for this to work with Roll20, you'll need to preface the name of each form element with "attr_" otherwise data won't save. As an example, under character, your Name field should look something like: <input type="text" name="attr_Name" placeholder="Name"> However, I would personally recommend the name be "attr_character_name", because then it will auto populate with the name of the character sheet journal, and the name will only need to be entered once. But that's a personal preference. Character Sheet Tab: Harm, Coin and Stash seem to be off balance. Meaning, there's a lot of white space weighted heavily on the right side of the sheet. Playbook Tab: Seems like the Special Abilities would be better suited as a repeating section rather than a static number of fields. With a repeating section, you create one set of fields and people can add however many of that "data set" they want. So, for your sheet, in practice you could do something like: <fieldset class="repeating_specialabilities">
<div class='sheet-2colrow' style='width:50%;'>
<div class='sheet-col'>
<input type="checkbox" name="attr_trained">
<input type="text" name="attr_specAbilityName" placeholder="Ability Name">
</div>
<div class='sheet-col'>
<input type="text" name="attr_specAbilityDesc" placeholder="Ability Description">
</div>
</div>
</fieldset> Then, an +Add button would appear on the character sheet and every time someone pressed that button, a new row would appear with those three fields (a checkbox and two text boxes). Dangerous Friends and Special Items would also benefit from being a Repeating Section, too. Understandard Items, you have three radio buttons, and currently all three of them can be selected. If only one of them should be selected at a time, then you will need to make them all the same name with different values. Something like: <input type="radio" name="attr_load" value="3">3 light
<input type="radio" name="attr_load" value="5">5 normal
<input type="radio" name="attr_load" value="6">6 heavy If all of them can be selected at the same time, then you would want to use checkboxes, because once you select a radio button set up the way they are currently, you can't unselect one. Which I suppose would also mean, if there is a non-encumbered value (less than 3), then you would want an option for that as well, otherwise, once the character is encumbered, the character would ALWAYS be encumbered. The last thing of note, the code seems to be backward to my mind. When I read code and try to match it with the design, I read top to bottom, and right to left. Meaning, I expect the elements at the top of the sheet to be near the top of the code, and for the tabs to be similar. So, I would expect the Character sheet Tab to be above the code for the Playbook tab, and the code for the Playbook tab to be above the code for the Crew tab. I had to do a control+F to find your code for the character name field, because it was nowhere near the top and scrolling through it I couldn't find it because all the Special Abilities were above it, etc. Not saying you have to conform to this methodology, but it seems to be what most sheet authors follow. I would also recommend reading over the Character Sheets Wiki Page , and especially the Building Character Sheets page . It has a lot of useful information. I hope this helps, and you've done a great job so far. Keep up the work and congrats on becoming a sheet author. It can get kind of addictive (just ask Coal Powered Puppet—we may need to have an intervention for him soon). :) Oh, and never hesitate to ask question in this forum—some of the most helpful people around frequent this forum and they are always happy to answer questions.
