Character Sheet Mod (coming soon)

Very interesting. ;-)

Looking forward to hearing your opinion on it when...be posting soon

<a href="https://github.com/earmarkaudiology-png/1e-AD-D-Character-Sheet-Mod/tree/main" rel="nofollow">https://github.com/earmarkaudiology-png/1e-AD-D-Character-Sheet-Mod/tree/main</a>

You need to make sure the adding of armour in the Armour Details section or in the Equipment section is somehow indexed so that one does not end up with duplicates and blanks like this from players who were not exactly computer literate: I mentioned this issue in October - or that's when I took the screen shots above. But it's still happening unfortunately, even on sheets a number of my players who are face to face and won't do anything on computer. I suspect there's some process that is doing it, but I have no idea which one. Sheet updates perhaps? I do not know. -- Tim

The mod I made does not tie armor to the armor section. It’s a separate drop down. I’ll have double check it.&nbsp;

A duplicate armor issue has plagued Tim's sheets but it doesn't have any connection to Rotto's mods.&nbsp; fyi: Adding something directly to Armor Details will also add it to Equipment&gt;Armor and you can also add to Armor Details from Equipment|Armor "+Add" These armor items are then "synced". (ie changes to either effect the other) @Tim Feel free to send me a direct message or create a new thread Tim since this really isn't relevant to Rotto's mod.

ROTTO said: The mod I made does not tie armor to the armor section. It’s a separate drop down. I’ll have double check it.&nbsp; Sorry Rotto, I assumed you were doing some major work on the sheet. :)

vÍnce said: A duplicate armor issue has plagued Tim's sheets but it doesn't have any connection to Rotto's mods.&nbsp; fyi: Adding something directly to Armor Details will also add it to Equipment&gt;Armor and you can also add to Armor Details from Equipment|Armor "+Add" These armor items are then "synced". (ie changes to either effect the other) @Tim Feel free to send me a direct message or create a new thread Tim since this really isn't relevant to Rotto's mod. Didn't realize that, sorry. It's still an issue with us for some reason.

@ROTTO&nbsp; I got to play around with the modded sheet a little.&nbsp; I DO LIKE the auto-fill options. Looks like you've put a lot of effort into this Rotto!&nbsp; I'm curious to look into the code in more depth. Later this week, life/work willing.&nbsp; With just a quick view; it seems you may be using an older version of the sheetworkers? I recently converted nearly all functions and roll20's built-in API commands to async versions (ie orcsAsync: <a href="https://github.com/onyxring/orcsAsync" rel="nofollow">https://github.com/onyxring/orcsAsync</a> )&nbsp; Hard to tell since some of the codes minified. I was seeing a continuous console.log error: "Character Sheet Error: Trying to do getSectionIDs when no character is active in sandbox." Adding armor to Armor Details and clicking "in Use" triggers the item to be added as Equipment&gt;Armor and syncs as intended, but the name field was blank on Equipment&gt;Armor. I transmogrified an existing 1e character and the name fields for any existing attacks, weapons, equipment, were all blank.&nbsp; The attribute data still existed because I could make a roll and the chat output included the proper attribute name. I'm&nbsp; sure it has something to do with the preset drop-down lists. Spells were completely gone though... Where's my spell tabs and sorting options?&nbsp; You're killing me Smalls&nbsp; &nbsp;;-P

Didn't care for the sorting option for the spells so I changed the whole layout.&nbsp;The transmogrified character problem is related to how the preset dropdown-linked fields are being populated versus how older attribute data was stored. Since the rolls still resolve with the correct names, the underlying attribute data seems intact, so this looks more like a field-mapping/display issue than missing data. I had to update the trading post data to get them to match and those worked ok. I'll look into this...

I also changed the class name from mage to magic user. When you change that on the sheet the spells should show up better. Not sure how to backtrack for old sheets with new dropdowns. Maybe you can fix that but I'm thinking this won't work well with old filled out character sheets.&nbsp;

<a href="https://github.com/earmarkaudiology-png/1e-AD-D-Character-Sheet-Mod/blob/main/html" rel="nofollow">https://github.com/earmarkaudiology-png/1e-AD-D-Character-Sheet-Mod/blob/main/html</a> Made some changes.&nbsp;&nbsp;

Tim...just realized the issue with armor leaving blanks in the inventory is still happening. Gotta see if I can fix that or Vince but to me it's not a big deal.

The repeating sections will leave one blank item as a placeholder if no repeating items exist.&nbsp; It's a feature. ;-)

Vince...was the current and max spells line a manual entry? I made use the source book data to auto adjust to level and memorized spells. The filtering made it supper buggy for spells you had. I'm clearly not proficient enough to figure that out.

There isn't any automation and/or any CSS hide/show based on current and max. Depending on what you want to do with them, you might want to verify the actual attribute names though.&nbsp; If I recall they use a different naming pattern than say current and max on equipment.&nbsp; I had to pick choose my battles while revising the old sheet.&nbsp; (I had a major roll in it's development but I was not the original sheet author) example; &lt;span&gt;&lt;input type="text" class="width-fill" name="attr_spells_lvl_0_current" title="@{spells_lvl_0_current}" value="0"&gt;&lt;/span&gt; &lt;span&gt;&lt;input type="text" class="width-fill" name="attr_spells_lvl_0_max" title="@{spells_lvl_0|max}" value="0"&gt;&lt;/span&gt;

ROTTO said: Tim...just realized the issue with armor leaving blanks in the inventory is still happening. Gotta see if I can fix that or Vince but to me it's not a big deal. The blanks aren't too bad an issue, but as a DB guy it bothers me. LOL -- Tim

So the way it "should" function is that when a sheet is opened there's a check to see if there are any repeating rows. If there is already a row or more, do nothing, else include a single row as a placeholder. The idea (good/bad) being, if you open a repeating section you can start filling in row data (all available fields are ready for entry) without having to use the "+Add"&nbsp; first.&nbsp; Of course if you do not fill in that first row and start with a new row, you will have one blank row unless you delete it...