chore: fix some string concatenation scope

6pac · Jul 15, 2023 · 5126920 · 5126920
1 parent 0864159
commit 5126920
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/plugins/slick.rowmovemanager.js b/plugins/slick.rowmovemanager.js
@@ -103,7 +103,7 @@
           dd.clonedSlickRow.style.marginLeft = Number(options.rowMoveShadowMarginLeft || 0) + 'px';
           dd.clonedSlickRow.style.marginTop = Number(options.rowMoveShadowMarginTop || 0) + 'px';
           dd.clonedSlickRow.style.opacity = '' + (options.rowMoveShadowOpacity || 0.95);
-          dd.clonedSlickRow.style.transform = 'scale(' + (options.rowMoveShadowScale || 0.75);
+          dd.clonedSlickRow.style.transform = 'scale(' + (options.rowMoveShadowScale || 0.75) + ')';
           _canvas.appendChild(dd.clonedSlickRow);
         }
       }

diff --git a/slick.grid.js b/slick.grid.js
@@ -1285,7 +1285,7 @@ if (typeof Slick === "undefined") {
         const headerTarget = hasFrozenColumns() ? ((i <= options.frozenColumn) ? _headerL : _headerR) : _headerL;
         const headerRowTarget = hasFrozenColumns() ? ((i <= options.frozenColumn) ? _headerRowL : _headerRowR) : _headerRowL;
 
-        const header = utils.createDomElement('div', { id: '' + uid + m.id, dataset: { id: m.id }, className: 'ui-state-default slick-header-column', title: m.toolTip || '' }, headerTarget);
+        const header = utils.createDomElement('div', { id: '' + (uid + m.id), dataset: { id: m.id }, className: 'ui-state-default slick-header-column', title: m.toolTip || '' }, headerTarget);
         utils.createDomElement('span', { className: 'slick-column-name', innerHTML: sanitizeHtmlString(m.name) }, header);
         utils.width(header, m.width - headerColumnWidthDiff);
 
@@ -1318,7 +1318,7 @@ if (typeof Slick === "undefined") {
 
         if (m.sortable) {
           header.classList.add("slick-header-sortable");
-          utils.createDomElement('div', { className: 'slick-sort-indicator ' + options.numberedMultiColumnSort && !options.sortColNumberInSeparateSpan ? ' slick-sort-indicator-numbered' : '' }, header);
+          utils.createDomElement('div', { className: 'slick-sort-indicator ' + ((options.numberedMultiColumnSort && !options.sortColNumberInSeparateSpan) ? ' slick-sort-indicator-numbered' : '') }, header);
           if (options.numberedMultiColumnSort && options.sortColNumberInSeparateSpan) {
             utils.createDomElement('div', { className: 'slick-sort-indicator-numbered' }, header);
            }
@@ -3476,7 +3476,7 @@ if (typeof Slick === "undefined") {
         }
       }
 
-      stringArray.push('<div class="' + cellCss + (addlCssClasses ? ' ' + addlCssClasses : '') + '" ' + toolTip + customAttrStr + '>');
+      stringArray.push('<div class="' + cellCss + (addlCssClasses ? ' ' + addlCssClasses : '') + '" ' + (toolTip + customAttrStr) + '>');
 
       // if there is a corresponding row (if not, this is the Add New row or this data hasn't been loaded yet)
       if (item) {