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

CSS refresh #6243

Closed
wants to merge 3 commits into from
Closed

CSS refresh #6243

wants to merge 3 commits into from

Conversation

mperham
Copy link
Collaborator

@mperham mperham commented Mar 25, 2024

Upgrade Bootstrap and prune/simplify CSS rules where possible.

  • Upgrade to Bootstrap 3.4
  • Migrate to Bootstrap 4.6 if it doesn't break RTL

@mperham
Copy link
Collaborator Author

mperham commented Mar 25, 2024

@mperham
Copy link
Collaborator Author

mperham commented Mar 25, 2024

Bootstrap 4.6 does not appear to have any major changes from 4.5. Migration from v3: https://getbootstrap.com/docs/4.6/migration/

@mperham
Copy link
Collaborator Author

mperham commented Apr 25, 2024

Aborting this. It's a quagmire of manual testing I don't want to get into. Nothing's actually broken today and old CSS has never been a source of security issues so only forward compatibility with future browsers is a concern; we can tackle those issues as necessary when they become apparent.

@mperham mperham closed this Apr 25, 2024
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.

None yet

1 participant