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

Modifications to the Specialization Sheet #1579

Merged
merged 6 commits into from
Jun 27, 2024

Conversation

sparradee
Copy link
Contributor

  • Modifies the size of the window; scales the talent blocks as the window resizes; updates the appearance of the talent connectors
  • Increases the default size of the pop-out editor

This is a small overhaul of the Specialization sheet, intended to improve legibility and add better scaling.

Updated Sheet:
image

Scaling:
image

Question: Is there no sass/scss source for the mandar theme?

- Modifies the size of the window; scales the talent blocks as the window resizes; updates the appearance of the talent connectors
- Increases the default size of the pop-out editor
@wrycu
Copy link
Collaborator

wrycu commented Jun 26, 2024

@sparradee sorry for the delay here, I don't get email notifications for new PRs unless I'm directly assigned. Can you please update the changelog with a summary of the changes you've implemented here? I reviewed them and they look good.

@sparradee
Copy link
Contributor Author

@sparradee sorry for the delay here, I don't get email notifications for new PRs unless I'm directly assigned. Can you please update the changelog with a summary of the changes you've implemented here? I reviewed them and they look good.

Done. I put it under a "1.902" header, but that's likely pretty presumptive of me...

@wrycu wrycu self-requested a review June 27, 2024 03:52
@wrycu wrycu merged commit fa00b09 into StarWarsFoundryVTT:main Jun 27, 2024
@wrycu
Copy link
Collaborator

wrycu commented Jun 27, 2024

Not at all. I like the changes :)

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.

2 participants