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

Consistent wrapper headers display on actor sheets #299

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Deixx
Copy link
Contributor

@Deixx Deixx commented Jan 10, 2025

Examples of changes:
Text was not properly centered due to fixed width (Agent Notes tab)
Some headers were 100% width (Vehicle Main tab)
Some were aligned-left (Agent Combat tab)

@Deixx
Copy link
Contributor Author

Deixx commented Jan 10, 2025

Full disclosure: I rarely play with CSS, so despite the changes being quite small I'd appreciate thorough verification.
This PR is the outcome of my tinkering with the CSS on "live" website. The results looked good and hopefully the proposed changes reflect what I did there ;)

With those changes all the headers I could find are now centered, with width = text + padding. Maybe I've gone too far ;)

@Deixx Deixx force-pushed the actors-wrapper-headers branch from 273b6c0 to 467877a Compare January 10, 2025 03:03
Deixx added 2 commits January 10, 2025 04:08
Examples of changes:
Text was not properly centered due to fixed width (Agent Notes tab)
Some headers were 100% width (Vehicle Main tab)
Some were aligned-left (Agent Combat tab)
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.

1 participant