Skip to content

Commit

Permalink
fix(common): replace innerHTML: '×' with textContent: '×' (#…
Browse files Browse the repository at this point in the history
…1156)

Co-authored-by: ghiscoding <Ghislain.Beaulac@se.com>
  • Loading branch information
ghiscoding and ghiscoding-SE committed Oct 27, 2023
1 parent 47cc546 commit e8b2cfb
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion packages/common/src/extensions/extensionCommonUtils.ts
Expand Up @@ -12,7 +12,7 @@ export function addCloseButtomElement(this: SlickColumnPicker | SlickGridMenu, m
const closePickerButtonElm = createDomElement('button', {
type: 'button', className: 'close',
ariaLabel: 'Close',
innerHTML: '&times;',
textContent: '×',
dataset: { dismiss: context instanceof SlickColumnPicker ? 'slick-column-picker' : 'slick-grid-menu' }
});
menuElm.appendChild(closePickerButtonElm);
Expand Down
2 changes: 1 addition & 1 deletion packages/common/src/extensions/menuFromCellBaseClass.ts
Expand Up @@ -155,7 +155,7 @@ export class MenuFromCellBaseClass<M extends CellMenu | ContextMenu> extends Men
menuElm.style.width = findWidthOrDefault(this.addonOptions?.width);
}

const closeButtonElm = createDomElement('button', { ariaLabel: 'Close', className: 'close', type: 'button', innerHTML: '&times;', dataset: { dismiss: this._menuCssPrefix } });
const closeButtonElm = createDomElement('button', { ariaLabel: 'Close', className: 'close', type: 'button', textContent: '×', dataset: { dismiss: this._menuCssPrefix } });

// -- Option List section
if (!(this.addonOptions as CellMenu | ContextMenu).hideOptionSection && isColumnOptionAllowed && optionItems.length > 0) {
Expand Down

0 comments on commit e8b2cfb

Please sign in to comment.