-
Notifications
You must be signed in to change notification settings - Fork 16
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
Feature new slider with react-slick #230
base: master
Are you sure you want to change the base?
Conversation
src/components/Layout.jsx
Outdated
export default TemplateWrapper; | ||
export const Head = () => ( | ||
<> | ||
<link href='https://fonts.gstatic.com' rel='preconnect' crossorigin/> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it seems that this has no effect. Reading this article https://www.igvita.com/2015/08/17/eliminating-roundtrips-with-preconnect/
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
From Lighthouse test: Warnings: A
was found for "http://cdnjs.cloudflare.com" but was not used by the browser. Only use
preconnect for important origins that the page will certainly request.
So maybe better to remove this?
to slick font
if it affect page loading
charSet instead of charset
Testing new components with react-slick, probably a more performant and reliable slider.