Skip to content
This repository has been archived by the owner on Jun 28, 2021. It is now read-only.

Local support #523

Merged
merged 21 commits into from
Dec 25, 2016
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
21 commits
Select commit Hold shift + click to select a range
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions src/config.js
Original file line number Diff line number Diff line change
Expand Up @@ -90,6 +90,7 @@ module.exports = Object.assign({
/* SEO: https://developers.google.com/structured-data/slsb-overview#markup_examples */
/* SEO: https://developers.google.com/structured-data/site-name#markup_requirements */
script: [
{ src: "https://cdn.polyfill.io/v2/polyfill.min.js?features=Intl.~locale.en"},
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yikes. What does this do?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are we not already using babel-polyfill?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

react-intl depends on built-in intl api, which is missing in old browser and safari. This polyfill add missing intl for those browsers.

https://github.com/yahoo/react-intl/wiki#i18n-in-javascript

{
type: 'application/ld+json',
innerHTML: `{
Expand Down
2 changes: 1 addition & 1 deletion src/styles/partials/_noscript-message.scss
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
.noscript-message {
.noscript-warning {
position: fixed;
top: 0;
left: 0;
Expand Down