update lock

Author: riccardoperra

packages/angular-table/package.json

@@ -59,7 +59,6 @@
     "@angular/platform-browser": "^17.3.9",
     "@angular/platform-browser-dynamic": "^17.3.9",
     "ng-packagr": "^17.3.0",
-    "reflect-metadata": "^0.2.2",
     "typescript": "5.4.5",
     "vitest": "^1.6.0"
   },

packages/angular-table/src/flex-render.ts

@@ -73,10 +73,11 @@ export class FlexRenderDirective<TProps extends NonNullable<unknown>>
   ) {}
 
   ngOnChanges(changes: SimpleChanges) {
+    console.log('changes', changes)
     if (changes['content']) {
       this.renderFlags |= FlexRenderFlags.ContentChanged
     }
-    if (!changes['content'] && changes['props']) {
+    if (changes['props']) {
       this.renderFlags |= FlexRenderFlags.PropsReferenceChanged
     }
     this.checkView()
@@ -98,10 +99,9 @@ export class FlexRenderDirective<TProps extends NonNullable<unknown>>
     }
 
     const contentToRender = this.#getContentValue(this.props)
+
     if (contentToRender.kind === 'null' || !this.renderView) {
-      // Whether the content is null or view has not been rendered, we set contentChanged
-      // flag in order to re-initialize everything via the `render` method;
-      this.renderFlags |= FlexRenderFlags.ContentChanged
+      this.renderFlags = FlexRenderFlags.Creation
     } else {
       this.renderView.setContent(contentToRender.content)
       const previousContentInfo = this.renderView.previousContent
@@ -116,7 +116,10 @@ export class FlexRenderDirective<TProps extends NonNullable<unknown>>
   }
 
   checkView() {
-    if (this.renderFlags & FlexRenderFlags.ContentChanged) {
+    if (
+      this.renderFlags &
+      (FlexRenderFlags.ContentChanged | FlexRenderFlags.Creation)
+    ) {
       this.render()
       return
     }

packages/angular-table/src/flex-render/flags.ts

@@ -1,23 +1,23 @@
 export enum FlexRenderFlags {
   /**
-   * Whether the view has been created.
+   * Whether the view has not been created yet or the current view will be cleared during the update phase.
    */
   Creation = 1 << 0,
   /**
-   * Whether the `content` property has changed.
+   * Whether the `content` property has changed or the view need a complete re-rendering.
    * When this flag is enabled, the view is recreated from scratch after clearing the previous one.
    */
   ContentChanged = 1 << 1,
   /**
-   * Whether the `props` property has changed.
-   * When this flag is enabled, the view context is updated based on the type of the content:
-   * - Component view: inputs will be updated and view will be marked as dirty
-   * - TemplateRef | primitive values: view will be marked as dirty
+   * Whether the `props` property reference has changed.
+   * When this flag is enabled, the view context is updated based on the type of the content.
+   *
+   * For Component view, inputs will be updated and view will be marked as dirty.
+   * For TemplateRef and primitive values, view will be marked as dirty
    */
   PropsReferenceChanged = 1 << 2,
   /**
    * Whether the current rendered view has to be dirty checked.
-   * When this flag is enabled, the view will be updated based on it's type.
    */
   DirtyCheck = 1 << 3,
 }