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

Changes to speed up website part 2 #148

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

Changes to speed up website part 2 #148

wants to merge 4 commits into from

Conversation

johnrieth-zz
Copy link

This is a new pull request because I couldn't figure out git. I am sorry about the confusion.

I have gotten rid of the stylesheet from fontdeck since fontdeck has shutdown. Doing so should make the site load faster. I also upgraded jquery to the most recent version. Javascript is defered which means the javascript executes once the page has been parsed.

I changed the color of links to a blue to make the site more color blind accessible. The footer text is now black so the text is easier to see. HTML lang attribute is set to English.

I updated the _config.yml file to use plugins instead of gems. I got a warning that gems is deprecated and plugins is current way to load gems.

@poohlaga
Copy link
Collaborator

moving @ofZach comment from #146

cc @tchoi8 can you help with an SVG of the SFPC logo in apercu ? I think this PR would be great to have and we just need to get the logo in...

@Inkering
Copy link
Contributor

Inkering commented Jun 22, 2020

Separated lang and contrast into #238 and #240 issues//PRs. I have a PR for both going since this seems inactive...

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.

4 participants