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

Main #215

Merged
merged 6 commits into from
Feb 8, 2025
Merged

Main #215

merged 6 commits into from
Feb 8, 2025

Conversation

aaronhans
Copy link
Contributor

The main things causing poor performance on prior versions of the site are:

  • The bulk of rendering of pages being done on the client after retrieving the multi MB full dataset file.
  • Large client side dependencies for bootstrap and font awesome

Changes to address this are:

  • Do the home page, major paged sorts and the profile page all with static site generation so all that HTML is created at build time
  • Remove the client side font awesome dependency with a new 11ty plugin that inlines the svgs based on the classnames during build. Getting access to this required a switch from jekyll to 11ty
  • Using css purging to trim bootstrap dependencies down to only what is being used.

These changes took the lighthouse performance score on the homepage from 49 to 98. Since future ScanGov plans include auditing performance of sites in the dataset it will be nice to set an example with our own infrastructure.

The updates required a lot of changes. We shouldn't merge until we are confident we have parity. This version of the site can be browsed at https://d1ernd7wc7yhlq.cloudfront.net/

Looks like the last version of the site used the built in Github jekyll build for github pages. This now has a git action to run the 11ty build but the github pages configuration has to be modified to run against the gh-pages branch which is populated during the git action triggered build.

@aaronhans aaronhans merged commit 62957c1 into ScanGov:main Feb 8, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant