refactor: Removed support for static strings

Simplified some of the logic in the `update`
method of the directive.

Fixed some instances where static parts were
still present.

Author: rkaraivanov

src/components/common/part-map.ts

@@ -9,20 +9,19 @@ import {
 } from 'lit/directive.js';
 
 export interface PartMapInfo {
-  readonly [name: string]: string | boolean | number;
+  readonly [name: string]: boolean | null | undefined;
 }
 
 class PartMapDirective extends Directive {
   private _previousParts?: Set<string>;
-  private _staticParts?: Set<string>;
 
   constructor(partInfo: PartInfo) {
     super(partInfo);
 
     if (
       partInfo.type !== PartType.ATTRIBUTE ||
       partInfo.name !== 'part' ||
-      (partInfo.strings?.length as number) > 2
+      (partInfo.strings?.length as number) > 0
     ) {
       throw new Error(
         '`partMap() can only be used in the `part` attribute and must be the only part in the attribute.'
@@ -40,47 +39,33 @@ class PartMapDirective extends Directive {
     part: AttributePart,
     [partMapInfo]: DirectiveParameters<this>
   ) {
+    const partList = part.element.part;
+
     if (this._previousParts === undefined) {
       this._previousParts = new Set();
-      if (part.strings !== undefined) {
-        this._staticParts = new Set(
-          part.strings
-            .join(' ')
-            .split(/\s/)
-            .filter((s) => s !== '')
-        );
-      }
+
       for (const name in partMapInfo) {
-        if (partMapInfo[name] && !this._staticParts?.has(name)) {
+        if (partMapInfo[name]) {
+          partList.add(name);
           this._previousParts.add(name);
         }
       }
 
       return this.render(partMapInfo);
     }
 
-    const partList = part.element.part;
-
     for (const name of this._previousParts) {
-      if (!(name in partMapInfo)) {
+      if (!(name in partMapInfo) || !partMapInfo[name]) {
         partList.remove(name);
-        this._previousParts!.delete(name);
+        this._previousParts.delete(name);
       }
     }
 
     for (const name in partMapInfo) {
       const value = !!partMapInfo[name];
-      if (
-        value !== this._previousParts.has(name) &&
-        !this._staticParts?.has(name)
-      ) {
-        if (value) {
-          partList.add(name);
-          this._previousParts.add(name);
-        } else {
-          partList.remove(name);
-          this._previousParts.delete(name);
-        }
+      if (value && !this._previousParts.has(name)) {
+        partList.add(name);
+        this._previousParts.add(name);
       }
     }
 

src/components/progress/base.ts

@@ -170,7 +170,7 @@ export abstract class IgcProgressBaseComponent extends LitElement {
 
     return this.labelFormat
       ? html`<span part=${partMap(parts)}>${this.renderLabelFormat()}</span>`
-      : html`<span part="${partMap(parts)} counter"></span>`;
+      : html`<span part=${partMap({ ...parts, counter: true })}></span>`;
   }
 
   protected renderLabelFormat() {

src/components/progress/linear-progress.ts

@@ -66,7 +66,7 @@ export default class IgcLinearProgressComponent extends IgcProgressBaseComponent
       <div part="base" style=${styleMap(this._styleInfo)}>
         <div part="track">
           <div part=${partMap(parts)}></div>
-          <div part="${partMap(parts)} secondary"></div>
+          <div part=${partMap({ ...parts, secondary: true })}></div>
         </div>
         ${this.renderDefaultSlot()}
       </div>