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

feat(styling): sheet styling refactor and light mode theme #200

Merged
merged 36 commits into from
Jan 24, 2025

Conversation

MangoFVTT
Copy link
Collaborator

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

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

Description
This is a giant PR that that completely refactors actor sheet CSS and fixes a host of issues with theming. In summary, this includes the following changes:

  • Completely rebuild character and adversary sheets with new HTML and CSS classes, trying to reduce as much as possible duplicate styling in the CSS files. Made improvements as well to layout so that the sheet is usable in edit mode without any features getting hidden (e.g. injury rolling).
  • Add all BW font files and change the application to use those. Remove the old font files. This also means changes to chat cards since they use these new fonts as well.
  • Fix theme swapping and implement colours for both light/dark mode in both default/Stormlight themes.
  • Fix a range of issues that came up with sheet rebuilding, and also changed various parts of sheet functionality to prepare for future features.

Related Issue
Closes #188, #47, #44.

How Has This Been Tested?
I've tested this and fixed a bunch of bugs already. There are more, for sure, since this is a major sheet refactor. I will continue to fix those bugs along the way as I proceed through styling.

Screenshots (if applicable)
image
image
image
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: 12.331.

Additional context
This is Part 1 of a major styling effort as we have discussed. I suspect this is the bulk of the work re: colouring and a lot of styling partials that can be re-used elsewhere, but there is plenty more work to come.

@MangoFVTT MangoFVTT added ui User interface related issue ux User experience related issue chat message character sheet labels Jan 16, 2025
@MangoFVTT MangoFVTT added this to the Release 0.3.0 milestone Jan 16, 2025
@MangoFVTT MangoFVTT requested review from zithith and stanavdb January 16, 2025 08:36
@MangoFVTT MangoFVTT self-assigned this Jan 16, 2025
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should probably not actually merge these into the repo directly. At least not without putting these under a separate license.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In that case we should prioritise swapping the theme over to the stormlight module and getting that running, otherwise we'll end up with a lot of unfinished stuff floating around

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seconded!... Thirded?
Whatever, I agree ;)

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agreed

Copy link
Collaborator

@zithith zithith left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One quick thought: do we want to convert the png assets to webp's?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seconded!... Thirded?
Whatever, I agree ;)

@MangoFVTT
Copy link
Collaborator Author

MangoFVTT commented Jan 23, 2025

One quick thought: do we want to convert the png assets to webp's?

@zithith I can do that quite easily, will find some time today

@MangoFVTT
Copy link
Collaborator Author

@stanavdb @zithith I've moved all Stormlight assets over to the handbook module in this PR: https://github.com/the-metalworks/stormlight-handbook/pull/13. Just the handbook for now, so that if we need to make changes to the theme I only do them in one place. Later on pre-release we can copy over all this stuff to the other modules so the theme can be registered from anywhere.

@zithith zithith merged commit 1ba38f3 into release-0.3.0 Jan 24, 2025
1 check passed
@zithith zithith deleted the feat/light-theme-colours-188 branch January 24, 2025 18:12
NoahJGersh pushed a commit to NoahJGersh/cosmere-rpg that referenced this pull request Jan 24, 2025
…works#200)

* Clean up part 1

* Add masks

* Re-add content

* Fix naming error

* Add drop shadow style

* Implement all fixed elements

* Mostly finish details tab

* Move all templates to preload

* Implement search bar

* Fix action list

* Fix equipment list

* Finish goals and equipment tab

* Fix conditions

* Fix adversary and dynamic boxes

* Update to new fonts

* Fix chat card dice totals

* Adjust some colours and fonts

* Organise assets

* Rename texture

* Add banners and texture

* Fix alignment issue with new font

* Fix chat card font issues

* Fix light mode

* Minor fixes

* Remove Didact Gothic

* Fix font variant issue

* Default theme light mode colours

* Update images and colours

* Add some shadows for visibility

* Add SC font to definitions

* Fix path click bug

* Remove robot

* Rename all variables to avoid plotweaver confusion

* Remove redundant list styling

* Convert assets to webp

* Move stormlight theme to handbook module
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
character sheet chat message ui User interface related issue ux User experience related issue
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Implement Light Theme sheet colours Style adversary sheet Style character sheet
3 participants