Skip to content

Comments

[QoL][QoC] Clean up CSS into modular structure #111

Merged
gbowne1 merged 31 commits intomasterfrom
gbowne1-csscleanup
Feb 20, 2026
Merged

[QoL][QoC] Clean up CSS into modular structure #111
gbowne1 merged 31 commits intomasterfrom
gbowne1-csscleanup

Conversation

@gbowne1
Copy link
Owner

@gbowne1 gbowne1 commented Feb 16, 2026

This PR:

  • cleans up the monolithic style.css into multiple files and a css structure

  • changes /src/css/style.css to app.css which gets script tag imported to index.html

This is;

  • easier to maintain
  • promote less CI issues with multiple contributors and collaborators
  • easier to add new styles
  • easier to read
  • easier to debug style and layout issues by making it easier to narrow down conflicts

@gbowne1 gbowne1 requested a review from Ved178 February 16, 2026 21:25
@gbowne1 gbowne1 self-assigned this Feb 16, 2026
@gbowne1 gbowne1 added bug Something isn't working enhancement New feature or request cleanup labels Feb 16, 2026
@gbowne1 gbowne1 linked an issue Feb 16, 2026 that may be closed by this pull request
@gbowne1
Copy link
Owner Author

gbowne1 commented Feb 16, 2026

Ignore the CI failure here. This is WIP towards less failures.

@gbowne1 gbowne1 changed the title Gbowne1 csscleanup [QoL][QoC] Clean up CSS into modular structure Feb 16, 2026
@gbowne1
Copy link
Owner Author

gbowne1 commented Feb 16, 2026

Inline styles from;

  • login.html
  • register.html
  • go-live.html
  • watch.html

need to be added to /pages/ then linked in each page.

We should also consider some global styles and a mobile specific stylesheet at some point in the future

I would also recommend adding a bootstrap-override.css to house anything going against bootstrap 5.3's default styling.

@gbowne1
Copy link
Owner Author

gbowne1 commented Feb 16, 2026

Sorry for the large pile of commits, just showing work in progress here.

Forgot to add the bootstrap overrides that were in the original style.css
@gbowne1 gbowne1 linked an issue Feb 16, 2026 that may be closed by this pull request
Copy link
Collaborator

@Ved178 Ved178 left a comment

Choose a reason for hiding this comment

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

Changes seem to work fine. Ready to merge.

@gbowne1
Copy link
Owner Author

gbowne1 commented Feb 17, 2026

This should be a lot easier to maintain with some minor tweaks.

Less likely two or more contributors will have to touch the same file.

Also, now we can easily fix issues and make a more robust css system.

@gbowne1 gbowne1 merged commit 5e61957 into master Feb 20, 2026
0 of 3 checks passed
@gbowne1 gbowne1 deleted the gbowne1-csscleanup branch February 20, 2026 17:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working cleanup enhancement New feature or request refactor

Projects

None yet

2 participants