@egjs/react-flicking - npm Package Compare versions

Comparing version 4.10.2-beta.0 to 4.10.2-beta.1

declaration/consts.d.ts

 import { FlickingProps } from "./types";
 export declare const DEFAULT_PROPS: FlickingProps;

declaration/Flicking.d.ts

@@ -17,2 +17,3 @@ import * as React from "react";
     private _prevChildren;
+    private _renderingState;
     get reactPanels(): (StrictPanel | NonStrictPanel | HTMLDivElement)[];
@@ -19,0 +20,0 @@ get renderEmitter(): Component<{

declaration/index.d.ts

@@ -0,0 +0,0 @@ import Flicking from "./Flicking";

declaration/index.umd.d.ts

 import Flicking from "./Flicking";
 export default Flicking;

declaration/NonStrictPanel.d.ts

@@ -0,0 +0,0 @@ import * as React from "react";

declaration/ReactElementProvider.d.ts

@@ -0,0 +0,0 @@ import { ElementProvider } from "@egjs/flicking";

declaration/ReactRenderer.d.ts

@@ -0,0 +0,0 @@ import { ExternalRenderer, PanelOptions, RendererOptions } from "@egjs/flicking";

declaration/StrictPanel.d.ts

@@ -0,0 +0,0 @@ import * as React from "react";

declaration/types.d.ts

@@ -0,0 +0,0 @@ /// <reference types="react" />

declaration/ViewportSlot.d.ts

@@ -0,0 +0,0 @@ import React, { ReactNode } from "react";

dist/flicking.cjs.js

@@ -7,3 +7,3 @@ /*
 repository: https://github.com/naver/egjs-flicking/tree/master/packages/react-flicking
-version: 4.10.0
+version: 4.10.2-beta.1
 */
@@ -484,2 +484,10 @@ 'use strict';
 
+var RenderingState;
+
+(function (RenderingState) {
+  RenderingState[RenderingState["NONE"] = 0] = "NONE";
+  RenderingState[RenderingState["RENDERED"] = 1] = "RENDERED";
+  RenderingState[RenderingState["UPDATED"] = 2] = "UPDATED";
+})(RenderingState || (RenderingState = {}));
+
 var Flicking =
@@ -500,2 +508,3 @@ /*#__PURE__*/
     _this._prevChildren = children;
+    _this._renderingState = RenderingState.NONE;
     return _this;
@@ -585,4 +594,5 @@ }
     var prevChildren = this._prevChildren; // Ignore updates before init, they will be updated after "ready" event's force update
+    // Also, ignore update between two successive rendering calls
 
-    if (!vanillaFlicking || !vanillaFlicking.initialized) return;
+    if (!vanillaFlicking || !vanillaFlicking.initialized || this._renderingState === RenderingState.RENDERED) return;
 
@@ -609,2 +619,3 @@ var nextChildren = this._getChildren(props.children);
     VanillaFlicking.sync(flicking, diffResult, this.reactPanels);
+    this._renderingState = RenderingState.UPDATED;
     this._diffResult = null;
@@ -658,2 +669,3 @@ };
     var panels = !!props.virtual && ((_b = props.panelsPerView) !== null && _b !== void 0 ? _b : -1) > 0 ? this._getVirtualPanels() : this._getPanels();
+    this._renderingState = RenderingState.RENDERED;
     return React.createElement(Viewport, __assign({}, attributes, {
@@ -660,0 +672,0 @@ className: viewportClasses.join(" "),

dist/flicking.esm.js

@@ -7,3 +7,3 @@ /*
 repository: https://github.com/naver/egjs-flicking/tree/master/packages/react-flicking
-version: 4.10.0
+version: 4.10.2-beta.1
 */
@@ -484,2 +484,10 @@ import * as React from 'react';
 
+var RenderingState;
+
+(function (RenderingState) {
+  RenderingState[RenderingState["NONE"] = 0] = "NONE";
+  RenderingState[RenderingState["RENDERED"] = 1] = "RENDERED";
+  RenderingState[RenderingState["UPDATED"] = 2] = "UPDATED";
+})(RenderingState || (RenderingState = {}));
+
 var Flicking =
@@ -500,2 +508,3 @@ /*#__PURE__*/
     _this._prevChildren = children;
+    _this._renderingState = RenderingState.NONE;
     return _this;
@@ -585,4 +594,5 @@ }
     var prevChildren = this._prevChildren; // Ignore updates before init, they will be updated after "ready" event's force update
+    // Also, ignore update between two successive rendering calls
 
-    if (!vanillaFlicking || !vanillaFlicking.initialized) return;
+    if (!vanillaFlicking || !vanillaFlicking.initialized || this._renderingState === RenderingState.RENDERED) return;
 
@@ -609,2 +619,3 @@ var nextChildren = this._getChildren(props.children);
     sync(flicking, diffResult, this.reactPanels);
+    this._renderingState = RenderingState.UPDATED;
     this._diffResult = null;
@@ -658,2 +669,3 @@ };
     var panels = !!props.virtual && ((_b = props.panelsPerView) !== null && _b !== void 0 ? _b : -1) > 0 ? this._getVirtualPanels() : this._getPanels();
+    this._renderingState = RenderingState.RENDERED;
     return React.createElement(Viewport, __assign({}, attributes, {
@@ -660,0 +672,0 @@ className: viewportClasses.join(" "),

dist/flicking.umd.js

@@ -7,3 +7,3 @@ /*
 repository: https://github.com/naver/egjs-flicking/tree/master/packages/react-flicking
-version: 4.10.0
+version: 4.10.2-beta.1
 */
@@ -482,2 +482,10 @@ (function (global, factory) {
 
+    var RenderingState;
+
+    (function (RenderingState) {
+      RenderingState[RenderingState["NONE"] = 0] = "NONE";
+      RenderingState[RenderingState["RENDERED"] = 1] = "RENDERED";
+      RenderingState[RenderingState["UPDATED"] = 2] = "UPDATED";
+    })(RenderingState || (RenderingState = {}));
+
     var Flicking =
@@ -498,2 +506,3 @@ /*#__PURE__*/
         _this._prevChildren = children;
+        _this._renderingState = RenderingState.NONE;
         return _this;
@@ -583,4 +592,5 @@ }
         var prevChildren = this._prevChildren; // Ignore updates before init, they will be updated after "ready" event's force update
+        // Also, ignore update between two successive rendering calls
 
-        if (!vanillaFlicking || !vanillaFlicking.initialized) return;
+        if (!vanillaFlicking || !vanillaFlicking.initialized || this._renderingState === RenderingState.RENDERED) return;
 
@@ -607,2 +617,3 @@ var nextChildren = this._getChildren(props.children);
         VanillaFlicking.sync(flicking, diffResult, this.reactPanels);
+        this._renderingState = RenderingState.UPDATED;
         this._diffResult = null;
@@ -656,2 +667,3 @@ };
         var panels = !!props.virtual && ((_b = props.panelsPerView) !== null && _b !== void 0 ? _b : -1) > 0 ? this._getVirtualPanels() : this._getPanels();
+        this._renderingState = RenderingState.RENDERED;
         return React.createElement(Viewport, __assign({}, attributes, {
@@ -658,0 +670,0 @@ className: viewportClasses.join(" "),

package.json

 {
   "name": "@egjs/react-flicking",
-  "version": "4.10.2-beta.0",
+  "version": "4.10.2-beta.1",
   "description": "Everyday 30 million people experience. It's reliable, flexible and extendable carousel.",
@@ -32,3 +32,3 @@ "main": "dist/flicking.cjs.js",
     "@types/react": "^18.0.5",
-    "@types/react-dom": "18.0.0",
+    "@types/react-dom": "16.8.4",
     "@types/react-is": "^17.0.0",
@@ -35,0 +35,0 @@ "cpx": "^1.5.0",

src/react-flicking/ReactRenderer.ts

@@ -5,3 +5,2 @@ /*
  */
-import { flushSync } from "react-dom";
 import { ExternalRenderer, PanelOptions, RendererOptions, getFlickingAttached } from "@egjs/flicking";
@@ -43,3 +42,3 @@
       });
-        reactFlicking.forceUpdate();
+      reactFlicking.forceUpdate();
     });
@@ -59,3 +58,3 @@ }
       });
-        reactFlicking.forceUpdate();
+      reactFlicking.forceUpdate();
     });
@@ -62,0 +61,0 @@ }

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

222211

increased by4.88%

Number of package files

36

increased by20%

Lines of code

2887

increased by4.34%

No dependency changes