Skip to content

Commit 09d0b1f

Browse files
authored
refactor(tree): remove renderer usage (#18930)
We've mostly gotten rid of the `Renderer2` usages in favor of native APIs, but this is one of the few places left where we were still using it.
1 parent 2fce0c6 commit 09d0b1f

File tree

2 files changed

+9
-4
lines changed

2 files changed

+9
-4
lines changed

src/cdk/tree/padding.ts

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,11 @@ export class CdkTreeNodePadding<T> implements OnDestroy {
6969

7070
constructor(private _treeNode: CdkTreeNode<T>,
7171
private _tree: CdkTree<T>,
72-
private _renderer: Renderer2,
72+
/**
73+
* @deprecated _renderer parameter no longer being used. To be removed.
74+
* @breaking-change 11.0.0
75+
*/
76+
_renderer: Renderer2,
7377
private _element: ElementRef<HTMLElement>,
7478
@Optional() private _dir: Directionality) {
7579
this._setPadding();
@@ -104,8 +108,8 @@ export class CdkTreeNodePadding<T> implements OnDestroy {
104108
const element = this._element.nativeElement;
105109
const paddingProp = this._dir && this._dir.value === 'rtl' ? 'paddingRight' : 'paddingLeft';
106110
const resetProp = paddingProp === 'paddingLeft' ? 'paddingRight' : 'paddingLeft';
107-
this._renderer.setStyle(element, paddingProp, padding);
108-
this._renderer.setStyle(element, resetProp, null);
111+
element.style[paddingProp] = padding || '';
112+
element.style[resetProp] = '';
109113
this._currentPadding = padding;
110114
}
111115
}

tools/public_api_guard/cdk/tree.d.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -113,7 +113,8 @@ export declare class CdkTreeNodePadding<T> implements OnDestroy {
113113
indentUnits: string;
114114
get level(): number;
115115
set level(value: number);
116-
constructor(_treeNode: CdkTreeNode<T>, _tree: CdkTree<T>, _renderer: Renderer2, _element: ElementRef<HTMLElement>, _dir: Directionality);
116+
constructor(_treeNode: CdkTreeNode<T>, _tree: CdkTree<T>,
117+
_renderer: Renderer2, _element: ElementRef<HTMLElement>, _dir: Directionality);
117118
_paddingIndent(): string | null;
118119
_setPadding(forceChange?: boolean): void;
119120
ngOnDestroy(): void;

0 commit comments

Comments
 (0)