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.

Sign up for GitHub

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

Jump to bottom

Refactor: Remove CA State Web Template #2526

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from
Draft

Refactor: Remove CA State Web Template #2526

wants to merge 5 commits into from

Conversation

machikoyasuda
Copy link
Member

closes #2491

What this PR does

  • Remove CA State Web Template CSS, which will then remove the call to the CA State Web Template font
  • Remove CA State Web Template domain from CSP
  • Makes any CSS fixes necessary

Changelog

  • fix:text-left to text-start
  • fix: add css to fix skip nav
  • fix: button focuses
  • slight refactor: buttons in general. could be greatly refactored to mostly be written with css variables instead, reducing the number of new classes/modified classes.

@github-actions github-actions bot added deployment-dev [auto] Changes that will trigger a deploy if merged to dev front-end HTML/CSS/JavaScript and Django templates back-end Django views, sessions, middleware, models, migrations etc. labels Nov 19, 2024
@github-actions GitHub Actions
Copy link

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)		Lines missing
  benefits
  settings.py
Project Total  

This report was generated by python-coverage-comment-action

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
back-end Django views, sessions, middleware, models, migrations etc. deployment-dev [auto] Changes that will trigger a deploy if merged to dev front-end HTML/CSS/JavaScript and Django templates
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Web perf: Remove CA State Web Template
1 participant
@machikoyasuda