Skip to content

Conversation

Zen-Raven
Copy link
Contributor

Submission Checklist

Warning

Submission Checklist
Failure to complete this checklist in its entirety will result in your Pull Request being dismissed. If you have any questions, please feel free to create an issue.

Note

Draft Pull Requests
If you are unclear about any of the rules regarding the creation of character sheets, or need assistance from the Roll20 team, please feel free to create a Draft PR and request feedback. We'd much rather provide assistance than reject a PR.

[Draw Steel] Sheet Worker Updates

Please format your pull request in the following way: [Sheet Name] Change Type: Description. For example: [D&D5e] New Feature: Adding dragons to dungeons.

  • The pull request title clearly contains the name of the sheet I am editing.
  • The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).

Pull Request Content

  • The pull request makes changes to files in only one sub-folder.
  • The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)
  • The pull request does not, without express prior permission from affected parties, include any material that could be considered to infringe on a Publisher's intellectual property rights, such as logos, images, rules text or other rules content.

Changes / Description

Sheet Worker Code

  • Refactored ability and feature sharing sheet worker code
  • Cleaned up power roll sheet worker code.
  • Reorganized sheet worker code for readability

Various Updates

  • Moved Draw Steel logo to top of sheet so it will be displayed for both the character and NPC sheets
  • Changed styling on NPC ability edit checkboxes to be pencil icons.
  • Changed player ability cards use a dropdowns for characteristics instead of a text box. Added dark mode coloring to new element.
  • Removed unneeded placeholder properties from spans.
  • Altered the Power Roll template to include character name.

-Refactored ability and feature sharing sheet worker code
- Reorganized code for readability
- Moved Draw Steel logo to top of sheet so it will be displayed for both the character and NPC sheets
- Changed styling on NPC ability edit checkboxes to be pencils.
- Cleaned up power roll sheet worker code.
- Changed player ability cards use a dropdown for characteristics instead of text. Added dark mode coloring to new element.
- removed unneeded placeholder properties from spans.
- Altered the Power Roll template to include character name.
@roll20deploy
Copy link
Contributor

Character Sheet Info Roll20 Internal Use only.

@kfroll20
Copy link
Contributor

Hello @Zen-Raven,

Your branch currently has merge conflicts that need to be addressed before we can merge this.

Thank you!

- Added comments to some sections for clarity
- Removed edit ability from some of the standard abilities as they will not need to be edited.
- Cleaned up sheet worker code for new standard abilities.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants