@syncfusion/ej2-popups - npm Package Compare versions

Comparing version 20.3.60 to 20.4.38

.eslintrc.json

@@ -9,3 +9,4 @@ {
         "plugin:@typescript-eslint/recommended",
-        "plugin:jsdoc/recommended"
+        "plugin:jsdoc/recommended",
+        "plugin:security/recommended"
     ],
@@ -22,2 +23,3 @@ "parser": "@typescript-eslint/parser",
         "@typescript-eslint/tslint",
+        "eslint-plugin-security",
         "jsdoc"
@@ -27,2 +29,15 @@ ],
         "use-isnan": "error",
+        "security/detect-unsafe-regex":"error",
+        "security/detect-buffer-noassert":"error",
+        "security/detect-child-process":"error",
+        "security/detect-disable-mustache-escape":"error",
+        "security/detect-eval-with-expression":"error",
+        "security/detect-no-csrf-before-method-override":"error",
+        "security/detect-non-literal-fs-filename":"error",
+        "security/detect-non-literal-regexp":"error",
+        "security/detect-non-literal-require":"error",
+        "security/detect-object-injection":"error",
+        "security/detect-possible-timing-attacks":"error",
+        "security/detect-pseudoRandomBytes":"error",
+        "security/detect-new-buffer":"error",
         "@typescript-eslint/no-inferrable-types": "off",
@@ -29,0 +44,0 @@ "@typescript-eslint/ban-types": ["warn", {

CHANGELOG.md

@@ -5,20 +5,2 @@ # Changelog
 
-### Dialog
-
-#### Bug Fixes
-
-- `#I420756` - Now, the z-index value calculation will function properly when you have multiple dialog elements on the same page.
-
-## 20.3.59 (2022-11-29)
-
-### Dialog
-
-#### Bug Fixes
-
-- `#I415699` - Now, the `DialogUtility` position property works properly when X coordinates are passed as strings and Y coordinates are passed as numbers.
-
-- `#I420058` - Now, the  Dialog Close icon Button top and left position values are equal.
-
-## 20.3.58 (2022-11-22)
-
 ### Tooltip
@@ -28,36 +10,5 @@
 
-- `#I415386` - The Tooltip not shown for dynamically added target while pressing Tab key has been resolved.
+- `#I415013` - Resolved detached elements memory leak issue in the Tooltip component.
+- `#I415386` - The issue "Tooltip did not appear for the second time when using the tab key to focus on the remove icon of the uploader component" has been resolved.
 
-## 20.3.56 (2022-11-08)
-
-### Tooltip
-
-#### Bug Fixes
-
-- `#F177524` - The script error "When setting `opensOn` as `Click` with a dynamic content in the React Tooltip component" has been resolved.
-
-## 20.3.50 (2022-10-18)
-
-### Dialog
-
-#### Bug Fixes
-
-- `#I400881` - Now, When you set a secondary font color in the bootstrap 5 theme using the theme studio, the changes will be reflected for the Dialog Cancel button.
-
-## 20.3.49 (2022-10-11)
-
-### Tooltip
-
-#### Bug Fixes
-
-- `#I397894` - The issue with "when using custom template variant the anchor that triggers the tooltip when hover is unable to receive keyboard focus" has been resolved.
-
-## 20.3.48 (2022-10-05)
-
-### Dialog
-
-#### Bug Fixes
-
-- `#I387601` - Now, when you call the destroy method to destroy the Dialog control, it properly destroys the close icons instance.
-
 ## 20.2.49 (2022-09-13)
@@ -71,2 +22,4 @@
 
+- `#I400881` - Now, When you set a secondary font color in the bootstrap 5 theme using the theme studio, the changes will be reflected for the Dialog Cancel button.
+
 ## 20.2.45 (2022-08-23)
@@ -73,0 +26,0 @@

dist/global/index.d.ts

 /*!
 *  filename: index.d.ts
-*  version : 20.3.60
+*  version : 20.4.38
 *  Copyright Syncfusion Inc. 2001 - 2020. All rights reserved.
@@ -5,0 +5,0 @@ *  Use of this code is subject to the terms of our license.

package.json

 {
   "_from": "@syncfusion/ej2-popups@*",
-  "_id": "@syncfusion/ej2-popups@20.3.59",
+  "_id": "@syncfusion/ej2-popups@19.24.4",
   "_inBundle": false,
-  "_integrity": "sha512-OdUPtx2AhgzS2iBKGGxyCBlytLoqhTdpTb0hVvHkvuIciftls0hQB5SRr5RMrrJgk5vUdNqlI0x1hVyGVKVbSA==",
+  "_integrity": "sha512-0H42+536FPDtA/bzH79G/L+n59SUtrrqr6QbbBUvxofgQdAZCgh0Q0eV7Wds8o3x/s8qCNOu3s4QbtpWn+9bNg==",
   "_location": "/@syncfusion/ej2-popups",
@@ -45,4 +45,4 @@ "_phantomChildren": {},
   ],
-  "_resolved": "https://nexus.syncfusion.com/repository/ej2-hotfix-new/@syncfusion/ej2-popups/-/ej2-popups-20.3.59.tgz",
-  "_shasum": "db78c2e7271a594447ade1bbfe8bbfc8afd1ba78",
+  "_resolved": "https://nexus.syncfusion.com/repository/ej2-release/@syncfusion/ej2-popups/-/ej2-popups-19.24.4.tgz",
+  "_shasum": "cef705f3ec7b667a3cb69fb3a0d068a4c9622870",
   "_spec": "@syncfusion/ej2-popups@*",
@@ -58,4 +58,4 @@ "_where": "/jenkins/workspace/ease-automation_release_19.1.0.1/packages/included",
   "dependencies": {
-    "@syncfusion/ej2-base": "~20.3.56",
-    "@syncfusion/ej2-buttons": "~20.3.58"
+    "@syncfusion/ej2-base": "~20.4.38",
+    "@syncfusion/ej2-buttons": "~20.4.38"
   },
@@ -95,4 +95,4 @@ "deprecated": false,
   "typings": "index.d.ts",
-  "version": "20.3.60",
+  "version": "20.4.38",
   "sideEffects": false
 }

src/common/collision.js

@@ -142,3 +142,2 @@ /**
     }
-    // eslint-disable-next-line
     var tEdge = { TL: null,
@@ -145,0 +144,0 @@ TR: null,

src/common/position.js

@@ -49,3 +49,2 @@ /**
 export function calculatePosition(currentElement, positionX, positionY, parentElement, targetValues) {
-    //eslint-disable-next-line
     popupRect = undefined;
@@ -134,2 +133,3 @@ popupRect = targetValues;
     }
+    element = null;
     return pos;
@@ -136,0 +136,0 @@ }

src/common/resize.js

@@ -26,3 +26,2 @@ /**
 var containerElement;
-/* eslint-disable */
 var resizeStart = null;
@@ -177,3 +176,3 @@ var resize = null;
     if (this.targetEle && targetElement && targetElement.querySelector('.' + DIALOG_RESIZABLE)) {
-        containerElement = this.target === 'body' || 'document.body' || document.body ? null : this.targetEle;
+        containerElement = this.target === ('body' || 'document.body' || document.body) ? null : this.targetEle;
         maxWidth = this.targetEle.clientWidth;
@@ -237,2 +236,6 @@ maxHeight = this.targetEle.clientHeight;
     calculateValues();
+    var dialogResizeElement = targetElement.classList.contains('e-dialog');
+    if ((e.target.classList.contains(RESIZE_HANDLER) || e.target.classList.contains('e-dialog-border-resize')) && dialogResizeElement) {
+        e.target.classList.add(FOCUSED_HANDLER);
+    }
     var coordinates = e.touches ? e.changedTouches[0] : e;
@@ -251,3 +254,3 @@ originalMouseX = coordinates.pageX;
     EventHandler.add(target, touchMoveEvent, onMouseMove, this);
-    EventHandler.add(document, touchEndEvent, onMouseUp);
+    EventHandler.add(document, touchEndEvent, onMouseUp, this);
 }
@@ -254,0 +257,0 @@ /* istanbul ignore next */

src/dialog/dialog-model.d.ts

@@ -38,3 +38,2 @@ import { Component, Property, Event, Collection, L10n, EmitType, Complex, compile, createElement } from '@syncfusion/ej2-base';import { addClass, removeClass, detach, attributes, prepend, setStyleAttribute } from '@syncfusion/ej2-base';import { NotifyPropertyChanges, INotifyPropertyChanged, ChildProperty, isBlazor } from '@syncfusion/ej2-base';import { isNullOrUndefined, formatUnit, append, EventHandler, Draggable, extend } from '@syncfusion/ej2-base';import { BlazorDragEventArgs, SanitizeHtmlHelper, Browser } from '@syncfusion/ej2-base';import { Button, ButtonModel } from '@syncfusion/ej2-buttons';import { Popup, PositionData, getZindexPartial } from '../popup/popup';import { PositionDataModel } from '../popup/popup-model';import { createResize, removeResize, setMinHeight, setMaxWidth, setMaxHeight } from '../common/resize';
      */
-    /* eslint-disable */
     click?: EmitType<Object>;
@@ -298,3 +297,2 @@
      */
-    /* eslint-disable */
     created?: EmitType<Object>;
@@ -309,3 +307,2 @@
      */
-    /* eslint-disable */
     open?: EmitType<Object>;
@@ -338,3 +335,2 @@
      */
-    /* eslint-disable */
     close?: EmitType<Object>;
@@ -359,3 +355,2 @@
      */
-    /* eslint-disable */
     dragStart?: EmitType<Object>;
@@ -370,3 +365,2 @@
      */
-    /* eslint-disable */
     dragStop?: EmitType<Object>;
@@ -381,3 +375,2 @@
      */
-    /* eslint-disable */
     drag?: EmitType<Object>;
@@ -391,3 +384,2 @@
      */
-    /* eslint-disable */
     overlayClick?: EmitType<Object>;
@@ -401,3 +393,2 @@
      */
-    /* eslint-disable */
     resizeStart?: EmitType<Object>;
@@ -411,3 +402,2 @@
      */
-    /* eslint-disable */
     resizing?: EmitType<Object>;
@@ -421,3 +411,2 @@
      */
-    /* eslint-disable */
     resizeStop?: EmitType<Object>;
@@ -424,0 +413,0 @@

src/dialog/dialog.d.ts

@@ -774,3 +774,3 @@ import { Component, EmitType } from '@syncfusion/ej2-base';
      *
-     * @returns {DialogDimension}
+     * @returns {DialogDimension}- returns the dialog element Dimension.
      * @public
@@ -777,0 +777,0 @@ */

src/popup/popup-model.d.ts

@@ -144,3 +144,2 @@ import { setStyleAttribute, addClass, removeClass, ChildProperty, Complex } from '@syncfusion/ej2-base';import { isNullOrUndefined, formatUnit } from '@syncfusion/ej2-base';import { Browser } from '@syncfusion/ej2-base';import { calculatePosition, OffsetPosition, calculateRelativeBasedPosition } from '../common/position';import { Animation, AnimationModel, Property, Event, EmitType, Component } from '@syncfusion/ej2-base';import { NotifyPropertyChanges, INotifyPropertyChanged } from '@syncfusion/ej2-base';import { EventHandler } from '@syncfusion/ej2-base';import { flip, fit, isCollide , CollisionCoordinates } from '../common/collision';
      */
-    /* eslint-disable */
     open?: EmitType<Object>;
@@ -153,3 +152,2 @@
      */
-    /* eslint-disable */
     close?: EmitType<Object>;
@@ -162,5 +160,4 @@
      */
-    /* eslint-disable */
     targetExitViewport?: EmitType<Object>;
 
 }

src/popup/popup.js

@@ -642,4 +642,5 @@ var __extends = (this && this.__extends) || (function () {
                 this.element.offsetParent.tagName === 'BODY') {
-                this.element.style.top = window.scrollY > parseInt(this.element.style.top) ? formatUnit(window.scrollY - parseInt(this.element.style.top))
-                    : formatUnit(parseInt(this.element.style.top) - window.scrollY);
+                this.element.style.top = window.scrollY > parseInt(this.element.style.top, 10) ?
+                    formatUnit(window.scrollY - parseInt(this.element.style.top, 10))
+                    : formatUnit(parseInt(this.element.style.top, 10) - window.scrollY);
                 this.element.style.position = 'fixed';
@@ -646,0 +647,0 @@ this.fixedParent = true;

src/spinner/spinner.js

@@ -6,3 +6,2 @@ import { isNullOrUndefined, classList, createElement } from '@syncfusion/ej2-base';
 var DEFT_FLUENT_WIDTH = 30;
-var DEFT_TAIL_WIDTH = 24;
 var DEFT_BOOT_WIDTH = 30;
@@ -128,3 +127,3 @@ var DEFT_BOOT4_WIDTH = 36;
     var uniqueID = random_generator();
-    globalTimeOut[uniqueID] = { timeOut: 0, type: 'Material', radius: radius };
+    globalTimeOut["" + uniqueID] = { timeOut: 0, type: 'Material', radius: radius };
     create_material_element(container, uniqueID, makeElement, CLS_MATERIALSPIN);
@@ -142,3 +141,3 @@ mat_calculate_attributes(radius, container, 'Material', CLS_MATERIALSPIN);
     var uniqueID = random_generator();
-    globalTimeOut[uniqueID] = { timeOut: 0, type: 'Bootstrap4', radius: radius };
+    globalTimeOut["" + uniqueID] = { timeOut: 0, type: 'Bootstrap4', radius: radius };
     create_material_element(container, uniqueID, makeElement, CLS_BOOT4SPIN);
@@ -156,3 +155,3 @@ mat_calculate_attributes(radius, container, 'Bootstrap4', CLS_BOOT4SPIN);
     var uniqueID = random_generator();
-    globalTimeOut[uniqueID] = { timeOut: 0, type: 'Bootstrap5', radius: radius };
+    globalTimeOut["" + uniqueID] = { timeOut: 0, type: 'Bootstrap5', radius: radius };
     create_material_element(container, uniqueID, makeElement, CLS_BOOT5SPIN);
@@ -171,4 +170,4 @@ mat_calculate_attributes(radius, container, 'Bootstrap5', CLS_BOOT5SPIN);
     var timeOutVar = 0;
-    globalTimeOut[uniqueID].timeOut = 0;
-    globalObject[uniqueID] = globalVariables(uniqueID, radius, 0, 0);
+    globalTimeOut["" + uniqueID].timeOut = 0;
+    globalObject["" + uniqueID] = globalVariables(uniqueID, radius, 0, 0);
     // eslint-disable-next-line
@@ -187,3 +186,3 @@ var spinnerInfo = { uniqueID: uniqueID, container: container, globalInfo: globalObject, timeOutVar: timeOutVar };
     var uniqueID = random_generator();
-    globalTimeOut[uniqueID] = { timeOut: 0, type: 'Fabric', radius: radius };
+    globalTimeOut["" + uniqueID] = { timeOut: 0, type: 'Fabric', radius: radius };
     create_fabric_element(container, uniqueID, CLS_FABRICSPIN, makeElement);
@@ -201,9 +200,16 @@ fb_calculate_attributes(radius, container, CLS_FABRICSPIN);
     var uniqueID = random_generator();
-    globalTimeOut[uniqueID] = { timeOut: 0, type: 'Fluent', radius: radius };
+    globalTimeOut["" + uniqueID] = { timeOut: 0, type: 'Fluent', radius: radius };
     create_fabric_element(container, uniqueID, CLS_FLUENTSPIN, makeElement);
     fb_calculate_attributes(radius, container, CLS_FLUENTSPIN);
 }
+/**
+ *
+ * @param {HTMLElement} container - specifies the element
+ * @param {number} radius - specifies the radius
+ * @param {createElementParams} makeElement - specifies the element
+ * @returns {void}
+ */
 function createTailwindSpinner(container, radius, makeElement) {
     var uniqueID = random_generator();
-    globalTimeOut[uniqueID] = { timeOut: 0, type: 'Tailwind', radius: radius };
+    globalTimeOut["" + uniqueID] = { timeOut: 0, type: 'Tailwind', radius: radius };
     create_fabric_element(container, uniqueID, CLS_TAILWINDSPIN, makeElement);
@@ -221,3 +227,3 @@ fb_calculate_attributes(radius, container, CLS_TAILWINDSPIN);
     var uniqueID = random_generator();
-    globalTimeOut[uniqueID] = { timeOut: 0, type: 'HighContrast', radius: radius };
+    globalTimeOut["" + uniqueID] = { timeOut: 0, type: 'HighContrast', radius: radius };
     create_fabric_element(container, uniqueID, CLS_HIGHCONTRASTSPIN, makeElement);
@@ -286,3 +292,3 @@ fb_calculate_attributes(radius, container, CLS_HIGHCONTRASTSPIN);
     var uniqueID = random_generator();
-    globalTimeOut[uniqueID] = { timeOut: 0, type: 'Bootstrap', radius: radius };
+    globalTimeOut["" + uniqueID] = { timeOut: 0, type: 'Bootstrap', radius: radius };
     create_bootstrap_element(innerContainer, uniqueID, makeElement);
@@ -410,3 +416,3 @@ boot_calculate_attributes(innerContainer, radius);
         function boot_animate(radius) {
-            if (globalTimeOut[id].isAnimate) {
+            if (globalTimeOut["" + id].isAnimate) {
                 ++count;
@@ -578,3 +584,2 @@ circle.setAttribute('r', radius + '');
  */
-// eslint-disable-next-line
 function createCircle(start, end, easing, duration, count, max, spinnerInfo) {
@@ -767,3 +772,2 @@ var id = ++spinnerInfo.globalInfo[spinnerInfo.uniqueID].previousId;
 }
-// eslint-disable-next-line
 /**
@@ -804,4 +808,4 @@ * Function to show the Spinner.
             var id = svgEle.getAttribute('id');
-            globalTimeOut[id].isAnimate = !isHide;
-            switch (globalTimeOut[id].type) {
+            globalTimeOut["" + id].isAnimate = !isHide;
+            switch (globalTimeOut["" + id].type) {
                 case 'Material':
@@ -904,5 +908,5 @@ // eslint-disable-next-line
         var id = svgElement.getAttribute('id');
-        clearTimeout(globalTimeOut[id].timeOut);
+        clearTimeout(globalTimeOut["" + id].timeOut);
     }
     setTheme(theme, container, radius, makeEle);
 }

src/tooltip/tooltip.js

@@ -171,2 +171,3 @@ var __extends = (this && this.__extends) || (function () {
         this.trigger('afterOpen', this.tooltipEventArgs);
+        this.tooltipEventArgs = null;
     };
@@ -179,2 +180,3 @@ Tooltip.prototype.closePopupHandler = function () {
         this.trigger('afterClose', this.tooltipEventArgs);
+        this.tooltipEventArgs = null;
     };
@@ -452,2 +454,3 @@ Tooltip.prototype.calculateTooltipOffset = function (position) {
         }
+        // eslint-disable-next-line security/detect-non-literal-fs-filename -- Safe as no value holds user input
         this.showTooltip(target, this.animation.open, e);
@@ -752,2 +755,3 @@ };
         });
+        this.tooltipEventArgs = null;
     };
@@ -1068,3 +1072,2 @@ Tooltip.prototype.popupHide = function (hideAnimation, target) {
         }
-        target.removeAttribute("tabindex");
     };
@@ -1071,0 +1074,0 @@ Tooltip.prototype.findTarget = function () {

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

3544864

increased by0.1%

Worsened metrics

Lines of code

49046

Dependency changes

• + Added@syncfusion/ej2-base@20.4.51(transitive)

• + Added@syncfusion/ej2-buttons@20.4.53(transitive)

• + Added@syncfusion/ej2-icons@20.4.48(transitive)

• - Removed@syncfusion/ej2-base@20.3.56(transitive)

• - Removed@syncfusion/ej2-buttons@20.3.58(transitive)

• - Removed@syncfusion/ej2-icons@20.3.56(transitive)

• Updated@syncfusion/ej2-base@~20.4.38

• Updated@syncfusion/ej2-buttons@~20.4.38