ayo/lion
1
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

feat(icon): allow custom iconManager

Browse source
This commit is contained in:
Jose C. Perea 2020-09-27 17:40:23 +02:00 committed by Joren Broekema
parent 84775744d8
commit 3147126295
2 changed files with 11 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
.changeset/forty-buttons-occur.md Normal file
View file

@ -0,0 +1,5 @@
---
'@lion/icon': minor
---
Allow setting a custom icon manager, which is important if you as a subclasser are managing your singletons.

7
packages/icon/src/LionIcon.js
View file

@ -161,6 +161,11 @@ export class LionIcon extends LitElement {
} }
} }
// eslint-disable-next-line class-methods-use-this
get _iconManager() {
return icons;
}
/** /**
* @param {string} prevIconId * @param {string} prevIconId
*/ */
@ -172,7 +177,7 @@ export class LionIcon extends LitElement {
} }
} else { } else {
const iconIdBeforeResolve = this.iconId; const iconIdBeforeResolve = this.iconId;
const svg = await icons.resolveIconForId(iconIdBeforeResolve); const svg = await this._iconManager.resolveIconForId(iconIdBeforeResolve);
// update SVG if it did not change in the meantime to avoid race conditions // update SVG if it did not change in the meantime to avoid race conditions
if (this.iconId === iconIdBeforeResolve) { if (this.iconId === iconIdBeforeResolve) {