@twreporter/react-article-components 1.6.0-rc.0 → 1.6.0

package/CHANGELOG.md

@@ -3,6 +3,30 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+# [1.6.0](https://github.com/twreporter/twreporter-npm-packages/compare/@twreporter/react-article-components@1.6.0-rc.2...@twreporter/react-article-components@1.6.0) (2022-07-21)
+
+**Note:** Version bump only for package @twreporter/react-article-components
+
+
+
+
+
+# [1.6.0-rc.2](https://github.com/twreporter/twreporter-npm-packages/compare/@twreporter/react-article-components@1.6.0-rc.1...@twreporter/react-article-components@1.6.0-rc.2) (2022-07-20)
+
+**Note:** Version bump only for package @twreporter/react-article-components
+
+
+
+
+
+# [1.6.0-rc.1](https://github.com/twreporter/twreporter-npm-packages/compare/@twreporter/react-article-components@1.6.0-rc.0...@twreporter/react-article-components@1.6.0-rc.1) (2022-07-20)
+
+**Note:** Version bump only for package @twreporter/react-article-components
+
+
+
+
+
 # [1.6.0-rc.0](https://github.com/twreporter/twreporter-npm-packages/compare/@twreporter/react-article-components@1.5.0...@twreporter/react-article-components@1.6.0-rc.0) (2022-07-08)
 
 

package/lib/components/body/embedded-code.js

@@ -7,13 +7,17 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports["default"] = exports.Caption = exports.Block = void 0;
 
-var _react = _interopRequireWildcard(require("react"));
+var _body = _interopRequireDefault(require("../../constants/prop-types/body"));
 
 var _propTypes = _interopRequireDefault(require("prop-types"));
 
+var _react = _interopRequireDefault(require("react"));
+
 var _styledComponents = _interopRequireDefault(require("styled-components"));
 
-var _reactWaypoint = require("react-waypoint");
+var _theme = _interopRequireDefault(require("../../constants/theme"));
+
+var _color = _interopRequireDefault(require("../../constants/color"));
 
 var _forEach = _interopRequireDefault(require("lodash/forEach"));
 
@@ -21,20 +25,8 @@ var _get = _interopRequireDefault(require("lodash/get"));
 
 var _merge = _interopRequireDefault(require("lodash/merge"));
 
-var _theme = _interopRequireDefault(require("../../constants/theme"));
-
-var _color = _interopRequireDefault(require("../../constants/color"));
-
-var _body = _interopRequireDefault(require("../../constants/prop-types/body"));
-
 function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { "default": obj }; }
 
-function _getRequireWildcardCache(nodeInterop) { if (typeof WeakMap !== "function") return null; var cacheBabelInterop = new WeakMap(); var cacheNodeInterop = new WeakMap(); return (_getRequireWildcardCache = function _getRequireWildcardCache(nodeInterop) { return nodeInterop ? cacheNodeInterop : cacheBabelInterop; })(nodeInterop); }
-
-function _interopRequireWildcard(obj, nodeInterop) { if (!nodeInterop && obj && obj.__esModule) { return obj; } if (obj === null || _typeof(obj) !== "object" && typeof obj !== "function") { return { "default": obj }; } var cache = _getRequireWildcardCache(nodeInterop); if (cache && cache.has(obj)) { return cache.get(obj); } var newObj = {}; var hasPropertyDescriptor = Object.defineProperty && Object.getOwnPropertyDescriptor; for (var key in obj) { if (key !== "default" && Object.prototype.hasOwnProperty.call(obj, key)) { var desc = hasPropertyDescriptor ? Object.getOwnPropertyDescriptor(obj, key) : null; if (desc && (desc.get || desc.set)) { Object.defineProperty(newObj, key, desc); } else { newObj[key] = obj[key]; } } } newObj["default"] = obj; if (cache) { cache.set(obj, newObj); } return newObj; }
-
-function _extends() { _extends = Object.assign || function (target) { for (var i = 1; i < arguments.length; i++) { var source = arguments[i]; for (var key in source) { if (Object.prototype.hasOwnProperty.call(source, key)) { target[key] = source[key]; } } } return target; }; return _extends.apply(this, arguments); }
-
 function _classCallCheck(instance, Constructor) { if (!(instance instanceof Constructor)) { throw new TypeError("Cannot call a class as a function"); } }
 
 function _defineProperties(target, props) { for (var i = 0; i < props.length; i++) { var descriptor = props[i]; descriptor.enumerable = descriptor.enumerable || false; descriptor.configurable = true; if ("value" in descriptor) descriptor.writable = true; Object.defineProperty(target, descriptor.key, descriptor); } }
@@ -100,8 +92,6 @@ function dispatchWindowLoadEvent() {
   window.dispatchEvent(loadEvent);
 }
 
-var infogramEmbed = 'infogram';
-
 var EmbeddedCode = /*#__PURE__*/function (_React$PureComponent) {
   _inherits(EmbeddedCode, _React$PureComponent);
 
@@ -113,19 +103,18 @@ var EmbeddedCode = /*#__PURE__*/function (_React$PureComponent) {
     _classCallCheck(this, EmbeddedCode);
 
     _this = _super.call(this, props);
+    _this._embedded = /*#__PURE__*/_react["default"].createRef();
+    return _this;
+  }
 
-    _defineProperty(_assertThisInitialized(_this), "state", {
-      isLoaded: false
-    });
-
-    _defineProperty(_assertThisInitialized(_this), "executeScript", function () {
-      _this.setState({
-        isLoaded: true
-      });
+  _createClass(EmbeddedCode, [{
+    key: "componentDidMount",
+    value: function componentDidMount() {
+      var _this2 = this;
 
-      var node = _this._embedded.current;
+      var node = this._embedded.current;
 
-      var scripts = _.get(_this.props, ['data', 'content', 0, 'scripts']);
+      var scripts = _.get(this.props, ['data', 'content', 0, 'scripts']);
 
       if (node && Array.isArray(scripts)) {
         var scriptsCount = scripts.length;
@@ -140,7 +129,7 @@ var EmbeddedCode = /*#__PURE__*/function (_React$PureComponent) {
             try {
               scriptEle.setAttribute(name, value);
             } catch (err) {
-              console.error('Failed to set an attribute to the embbeded script.\n', "embedded element id: ".concat(_.get(_this.props, 'data.id', ''), "\n"), "attribute name: ".concat(name, "\n"), "attribute value: ".concat(value, "\n"), 'error:\n', err);
+              console.error('Failed to set an attribute to the embbeded script.\n', "embedded element id: ".concat(_.get(_this2.props, 'data.id', ''), "\n"), "attribute name: ".concat(name, "\n"), "attribute value: ".concat(value, "\n"), 'error:\n', err);
             }
           });
 
@@ -166,71 +155,31 @@ var EmbeddedCode = /*#__PURE__*/function (_React$PureComponent) {
 
         node.appendChild(scriptsFragment);
       }
-    });
-
-    _defineProperty(_assertThisInitialized(_this), "loadEmbed", function () {
-      if (!_this.state.isLoaded) {
-        _this.executeScript();
-      }
-    });
-
-    _this._embedded = /*#__PURE__*/_react["default"].createRef();
-
-    var _$get = _.get(_this.props, ['data', 'content', 0], {}),
-        caption = _$get.caption,
-        embeddedCodeWithoutScript = _$get.embeddedCodeWithoutScript;
-
-    _this._caption = caption;
-    _this._embeddedCodeWithoutScript = embeddedCodeWithoutScript;
-    return _this;
-  }
-
-  _createClass(EmbeddedCode, [{
-    key: "componentDidMount",
-    value: function componentDidMount() {
-      var _this$_embeddedCodeWi;
-
-      // Delay loading infogram in loadEmbed()
-      if (!((_this$_embeddedCodeWi = this._embeddedCodeWithoutScript) !== null && _this$_embeddedCodeWi !== void 0 && _this$_embeddedCodeWi.includes(infogramEmbed))) {
-        this.executeScript();
-      }
-    }
-  }, {
-    key: "componentWillUnmount",
-    value: function componentWillUnmount() {
-      this._embedded = null;
-      this._caption = null;
-      this._embeddedCodeWithoutScript = null;
     }
   }, {
     key: "render",
     value: function render() {
-      var _this$_embeddedCodeWi2;
-
       var className = this.props.className;
 
-      var embed = /*#__PURE__*/_react["default"].createElement("div", {
+      var _$get = _.get(this.props, ['data', 'content', 0], {}),
+          caption = _$get.caption,
+          embeddedCodeWithoutScript = _$get.embeddedCodeWithoutScript;
+
+      return /*#__PURE__*/_react["default"].createElement("div", {
         className: className
       }, /*#__PURE__*/_react["default"].createElement(Block, {
         ref: this._embedded,
         dangerouslySetInnerHTML: {
-          __html: this._embeddedCodeWithoutScript
+          __html: embeddedCodeWithoutScript
         }
-      }), this._caption ? /*#__PURE__*/_react["default"].createElement(Caption, null, this._caption) : null);
-
-      if ((_this$_embeddedCodeWi2 = this._embeddedCodeWithoutScript) !== null && _this$_embeddedCodeWi2 !== void 0 && _this$_embeddedCodeWi2.includes(infogramEmbed)) {
-        return this.state.isLoaded ? embed : null;
-      }
-
-      return embed;
+      }), caption ? /*#__PURE__*/_react["default"].createElement(Caption, null, caption) : null);
     }
   }]);
 
   return EmbeddedCode;
-}(_react["default"].PureComponent); // Serious layout shifts show up when loading bunch of infograms due to lack of heights,
-// so here we apply waypoint wrapper to load infogram dynamically to avoid layout shifts for anchors.
-// https://twreporter-org.atlassian.net/browse/TWREPORTER-60
+}(_react["default"].PureComponent);
 
+exports["default"] = EmbeddedCode;
 
 _defineProperty(EmbeddedCode, "propTypes", {
   className: _propTypes["default"].string,
@@ -239,22 +188,4 @@ _defineProperty(EmbeddedCode, "propTypes", {
 
 _defineProperty(EmbeddedCode, "defaultProps", {
   className: ''
-});
-
-var WayPointWrapper = function WayPointWrapper(props) {
-  var embedRef = (0, _react.useRef)(null);
-
-  var onEnter = function onEnter() {
-    embedRef.current.loadEmbed();
-  };
-
-  return /*#__PURE__*/_react["default"].createElement(_reactWaypoint.Waypoint, {
-    onEnter: onEnter,
-    fireOnRapidScroll: false
-  }, /*#__PURE__*/_react["default"].createElement("div", null, /*#__PURE__*/_react["default"].createElement(EmbeddedCode, _extends({}, props, {
-    ref: embedRef
-  }))));
-};
-
-var _default = WayPointWrapper;
-exports["default"] = _default;
+});

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@twreporter/react-article-components",
-  "version": "1.6.0-rc.0",
+  "version": "1.6.0",
   "description": "The Reporter react article components, which are used in article page",
   "main": "lib/components/article-page.js",
   "repository": "https://github.com/twreporter/twreporter-npm-packages.git",
@@ -25,9 +25,9 @@
   },
   "dependencies": {
     "@twreporter/core": "^1.4.1",
-    "@twreporter/react-components": "^8.10.0",
+    "@twreporter/react-components": "^8.11.0",
     "@twreporter/redux": "^7.2.2",
-    "@twreporter/universal-header": "^2.2.10",
+    "@twreporter/universal-header": "^2.2.11",
     "howler": "^2.2.3",
     "lodash": "^4.17.11",
     "memoize-one": "^5.0.5",
@@ -40,5 +40,5 @@
   "files": [
     "lib"
   ],
-  "gitHead": "5ff16a85b38a7668e29e94a2d2e9d2ee8261bc52"
+  "gitHead": "22127d74e0d0433e87b427aed82394972cade963"
 }