Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Textarea not configured to inherit font-family by default #129

Merged
merged 1 commit into from
Oct 30, 2024

Conversation

RostCS
Copy link
Contributor

@RostCS RostCS commented Oct 29, 2024

Type
What type of pull request is this? (e.g., Bug fix, Feature, Refactor, etc.)

  • Bug fix
  • Feature
  • Refactor
  • Other (please describe):

Description
My first commit to the repository. Simply just fixing a minor CSS error where the font in the character sheet "Purposes and Obstacles" field is defaulting to Calibri, rather than than one of the previously used fonts

Related Issue
Closes #127

How Has This Been Tested?
Create a new "Character" actor, navigate to the goals tab, verify the font underneath the "Purpose" and "Obstacle" header.

Screenshots (if applicable)
Prior:
image

Updated:
image

Checklist:

  • I have commented on my code, particularly in hard-to-understand areas.
  • My changes do not introduce any new warnings or errors.
  • My PR does not contain any copyrighted works that I do not have permission to use.
  • I have tested my changes on Foundry VTT version: v12.331.

Additional context
Let me know if I have done any of this process wrong per the contribution rules of this repository. Thanks!

@stanavdb
Copy link
Collaborator

Can you change the base of this to release-0.1.2? We're keeping main inline with current release

@RostCS RostCS changed the base branch from main to release-0.1.2 October 29, 2024 16:31
@RostCS
Copy link
Contributor Author

RostCS commented Oct 29, 2024

Sounds good, should be done

@stanavdb stanavdb merged commit c12f689 into the-metalworks:release-0.1.2 Oct 30, 2024
1 check passed
@stanavdb
Copy link
Collaborator

stanavdb commented Nov 1, 2024

Just for the sake of tracking who closed what issues, can you leave a comment on #127? Can't assign you, unless you leave a comment

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Incorrect font in "purpose and obstacle" field
2 participants