fix(cdk/tree): fix lints

Author: BobobUnicorn

src/cdk/tree/tree.spec.ts

@@ -1278,7 +1278,7 @@ describe('CdkTree', () => {
         expect(treeElement.hasAttribute('tabindex')).toBeFalse();
       });
 
-      fit('sets the tabindex to the first item by default', () => {
+      it('sets the tabindex to the first item by default', () => {
         expect(nodes.map(x => x.getAttribute('tabindex')).join(', ')).toEqual('0, -1, -1');
       });
 

src/cdk/tree/tree.ts

@@ -45,7 +45,6 @@ import {
   computed,
   Signal,
   signal,
-  effect,
   ViewChild,
   ViewContainerRef,
   ViewEncapsulation,
@@ -103,16 +102,16 @@ type RenderingData<T> =
       renderNodes: readonly T[];
     };
 
-interface TemplateParams<T, K> {
+interface RenderNode<T, K> {
   context: CdkTreeNodeOutletContext<T>;
   template: TemplateRef<CdkTreeNodeOutletContext<T>>;
   key: K;
 }
 
 class NodeOutletTemplateContext<T, K> {
-  $implicit: Signal<Array<TemplateParams<T, K>>>;
+  $implicit: Signal<RenderNode<T, K>[]>;
 
-  constructor(nodes: Signal<Array<TemplateParams<T, K>>>) {
+  constructor(nodes: Signal<RenderNode<T, K>[]>) {
     this.$implicit = nodes;
   }
 }
@@ -129,7 +128,7 @@ class NodeOutletTemplateContext<T, K> {
   ],
 })
 export class CdkTreeNodeRenderer<T, K> implements OnInit {
-  readonly node = input.required<TemplateParams<T, K>>();
+  readonly node = input.required<RenderNode<T, K>>();
 
   ngOnInit() {
     if (CdkTreeNode.mostRecentTreeNode) {
@@ -138,7 +137,11 @@ export class CdkTreeNodeRenderer<T, K> implements OnInit {
   }
 }
 
-/** @docs-private */
+/**
+ * Invisible component that holds a template to be rendered inside the CdkTreeNodeOutlet.
+ *
+ * @docs-private
+ */
 @Component({
   selector: 'cdk-tree-node-outlet-template',
   template: `
@@ -154,8 +157,12 @@ export class CdkTreeNodeRenderer<T, K> implements OnInit {
     </ng-template>
   `,
   standalone: true,
-  styles: [`:host {display: none !important}`],
+  encapsulation: ViewEncapsulation.None,
+  changeDetection: ChangeDetectionStrategy.OnPush,
   imports: [CdkTreeNodeRenderer],
+  host: {
+    'style': 'display: none;',
+  },
 })
 export class CdkTreeNodeOutletTemplate<T, K> {
   @ViewChild('outletTemplate', {static: true, read: TemplateRef}) _nodeOutletTemplate: TemplateRef<
@@ -377,7 +384,7 @@ export class CdkTree<T, K = T>
     return this._renderData()?.flattenedNodes ?? [];
   });
   private readonly _flatNodesObs = toObservable(this._flattenedNodes);
-  private readonly _renderNodes = computed((): Array<TemplateParams<T, K>> => {
+  private readonly _renderNodes = computed((): RenderNode<T, K>[] => {
     const nodes = this._renderData()?.renderNodes ?? [];
     const levelAccessor = this._getLevelAccessor();
     const trackBy = this._trackByFn();
@@ -1426,7 +1433,6 @@ export class CdkTreeNode<T, K = T> implements OnDestroy, OnInit, TreeKeyManagerI
 
   constructor() {
     CdkTreeNode.mostRecentTreeNode = this as CdkTreeNode<T, K>;
-    console.log(inject(ViewContainerRef));
   }
 
   ngOnInit(): void {

tools/public_api_guard/cdk/tree.md

@@ -13,14 +13,17 @@ import { DataSource } from '@angular/cdk/collections';
 import { ElementRef } from '@angular/core';
 import { EventEmitter } from '@angular/core';
 import * as i0 from '@angular/core';
+import * as i1 from '@angular/common';
 import { InjectionToken } from '@angular/core';
+import { InputSignal } from '@angular/core';
 import { IterableDiffer } from '@angular/core';
 import { IterableDiffers } from '@angular/core';
 import { Observable } from 'rxjs';
 import { OnDestroy } from '@angular/core';
 import { OnInit } from '@angular/core';
 import { QueryList } from '@angular/core';
 import { SelectionModel } from '@angular/cdk/collections';
+import { Signal } from '@angular/core';
 import { Subject } from 'rxjs';
 import { TemplateRef } from '@angular/core';
 import { TrackByFunction } from '@angular/core';
@@ -87,7 +90,7 @@ export class CdkTree<T, K = T> implements AfterContentChecked, AfterContentInit,
     expand(dataNode: T): void;
     expandAll(): void;
     expandDescendants(dataNode: T): void;
-    expansionKey?: (dataNode: T) => K;
+    readonly expansionKey: InputSignal<((dataNode: T) => K) | undefined>;
     _getChildrenAccessor(): ((dataNode: T) => T[] | Observable<T[]> | null | undefined) | undefined;
     _getDirectChildren(dataNode: T): Observable<T[]>;
     // (undocumented)
@@ -122,7 +125,9 @@ export class CdkTree<T, K = T> implements AfterContentChecked, AfterContentInit,
     _setNodeTypeIfUnset(nodeType: 'flat' | 'nested'): void;
     toggle(dataNode: T): void;
     toggleDescendants(dataNode: T): void;
-    trackBy: TrackByFunction<T>;
+    readonly trackBy: InputSignal<TrackByFunction<T> | undefined>;
+    // (undocumented)
+    readonly _trackByFn: Signal<TrackByFunction<T>>;
     // @deprecated
     treeControl?: TreeControl<T, K>;
     _unregisterNode(node: CdkTreeNode<T, K>): void;
@@ -131,7 +136,7 @@ export class CdkTree<T, K = T> implements AfterContentChecked, AfterContentInit,
         end: number;
     }>;
     // (undocumented)
-    static ɵcmp: i0.ɵɵComponentDeclaration<CdkTree<any, any>, "cdk-tree", ["cdkTree"], { "dataSource": { "alias": "dataSource"; "required": false; }; "treeControl": { "alias": "treeControl"; "required": false; }; "levelAccessor": { "alias": "levelAccessor"; "required": false; }; "childrenAccessor": { "alias": "childrenAccessor"; "required": false; }; "trackBy": { "alias": "trackBy"; "required": false; }; "expansionKey": { "alias": "expansionKey"; "required": false; }; }, {}, ["_nodeDefs"], never, true, never>;
+    static ɵcmp: i0.ɵɵComponentDeclaration<CdkTree<any, any>, "cdk-tree", ["cdkTree"], { "dataSource": { "alias": "dataSource"; "required": false; }; "treeControl": { "alias": "treeControl"; "required": false; }; "levelAccessor": { "alias": "levelAccessor"; "required": false; }; "childrenAccessor": { "alias": "childrenAccessor"; "required": false; }; "trackBy": { "alias": "trackBy"; "required": false; "isSignal": true; }; "expansionKey": { "alias": "expansionKey"; "required": false; "isSignal": true; }; }, {}, ["_nodeDefs"], never, true, never>;
     // (undocumented)
     static ɵfac: i0.ɵɵFactoryDeclaration<CdkTree<any, any>, never>;
 }
@@ -143,7 +148,7 @@ export class CdkTreeModule {
     // (undocumented)
     static ɵinj: i0.ɵɵInjectorDeclaration<CdkTreeModule>;
     // (undocumented)
-    static ɵmod: i0.ɵɵNgModuleDeclaration<CdkTreeModule, never, [typeof i1.CdkNestedTreeNode, typeof i2.CdkTreeNodeDef, typeof i3.CdkTreeNodePadding, typeof i4.CdkTreeNodeToggle, typeof i5.CdkTree, typeof i5.CdkTreeNode, typeof i6.CdkTreeNodeOutlet], [typeof i1.CdkNestedTreeNode, typeof i2.CdkTreeNodeDef, typeof i3.CdkTreeNodePadding, typeof i4.CdkTreeNodeToggle, typeof i5.CdkTree, typeof i5.CdkTreeNode, typeof i6.CdkTreeNodeOutlet]>;
+    static ɵmod: i0.ɵɵNgModuleDeclaration<CdkTreeModule, never, [typeof i1_2.CdkNestedTreeNode, typeof i2.CdkTreeNodeDef, typeof i3.CdkTreeNodePadding, typeof i4.CdkTreeNodeToggle, typeof i5.CdkTree, typeof i5.CdkTreeNode, typeof i6.CdkTreeNodeOutlet], [typeof i1_2.CdkNestedTreeNode, typeof i2.CdkTreeNodeDef, typeof i3.CdkTreeNodePadding, typeof i4.CdkTreeNodeToggle, typeof i5.CdkTree, typeof i5.CdkTreeNode, typeof i6.CdkTreeNodeOutlet]>;
 }
 
 // @public
@@ -247,6 +252,16 @@ export class CdkTreeNodeOutletContext<T> {
     level: number;
 }
 
+// @public
+export class CdkTreeNodeOutletTemplate<T, K> {
+    // (undocumented)
+    _nodeOutletTemplate: TemplateRef<NodeOutletTemplateContext<T, K>>;
+    // (undocumented)
+    static ɵcmp: i0.ɵɵComponentDeclaration<CdkTreeNodeOutletTemplate<any, any>, "cdk-tree-node-outlet-template", never, {}, {}, never, never, true, never>;
+    // (undocumented)
+    static ɵfac: i0.ɵɵFactoryDeclaration<CdkTreeNodeOutletTemplate<any, any>, never>;
+}
+
 // @public
 export class CdkTreeNodePadding<T, K = T> implements OnDestroy {
     constructor(...args: unknown[]);
@@ -274,6 +289,18 @@ export class CdkTreeNodePadding<T, K = T> implements OnDestroy {
     static ɵfac: i0.ɵɵFactoryDeclaration<CdkTreeNodePadding<any, any>, never>;
 }
 
+// @public
+export class CdkTreeNodeRenderer<T, K> implements OnInit {
+    // (undocumented)
+    ngOnInit(): void;
+    // (undocumented)
+    readonly node: InputSignal<RenderNode<T, K>>;
+    // (undocumented)
+    static ɵdir: i0.ɵɵDirectiveDeclaration<CdkTreeNodeRenderer<any, any>, "[cdkTreeNodeRenderer]", never, { "node": { "alias": "node"; "required": true; "isSignal": true; }; }, {}, never, never, true, [{ directive: typeof i1.NgTemplateOutlet; inputs: { "ngTemplateOutlet": "template"; "ngTemplateOutletContext": "context"; }; outputs: {}; }]>;
+    // (undocumented)
+    static ɵfac: i0.ɵɵFactoryDeclaration<CdkTreeNodeRenderer<any, any>, never>;
+}
+
 // @public
 export class CdkTreeNodeToggle<T, K = T> {
     constructor(...args: unknown[]);