-
Notifications
You must be signed in to change notification settings - Fork 3
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/membership pages #22
Conversation
post.hbs
Outdated
|
||
{{!-- Facebook SDK for comments --}} | ||
<div id="fb-root"></div> | ||
<script async defer crossorigin="anonymous" src="https://connect.facebook.net/cs_CZ/sdk.js#xfbml=1&version=v3.3"></script> |
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.
Z těchto předgenerovaných kódů od třetích stran jsem pořád zmatená ― proč tam musí být obojí? Nestačil by jen async
v našem případě?
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.
Skript je sice v zápatí, takže ty atributy se nevyužijí úplně naplno, ale přesto aspoň trochu – pokud vím. async
odblokuje renderování, defer
odloží vykonání skriptu po načtení a sníží prioritu načítání. Dokonce bych řekl, že je bezpečné takovýhle skript dát do head
.
Myslím, že ve skriptech by se dalo někdy uklidit, v tom duchu je založená issue #4. Tak či onak bych to nechal, přijde mi to správně.
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.
Už se mi podařilo si najít odpověď:
The defer attribute may be specified even if the async attribute is specified, to cause legacy Web browsers that only support defer (and not async) to fall back to the defer behavior instead of the synchronous blocking behavior that is the default.
Určitě bych to teď neřešila. Tohle byla taková řečnická otázka, ke které bych se ráda v budoucnu vrátila, klidně až v rámci issue.
Gscan is temporarily disabled from `npm test` since it raises false alarm on usage of `author` in `default.hbs`. Reported to Gscan as TryGhost/gscan#365.
bf90fd1
to
423bb48
Compare
Pro otestování funkcionality:
Více viditelného obsahu na telefonech – před a po: