Feature/arbol contable edicion #131
Conversation
| <nb-icon nbTreeGridRowToggle | ||
| *ngIf="isDir(); else fileIcon" | ||
| [attr.aria-label]="expanded ? 'collapse' : 'expand'" | ||
| [icon]="expanded ? 'folder-open' : 'folder'" | ||
| aria-hidden="true" pack="fas"> | ||
| </nb-icon> | ||
|
|
||
| <ng-template #fileIcon> | ||
| <nb-icon *ngIf="isDoc()" | ||
| [icon]="'file-invoice-dollar'" aria-hidden="true" pack="fas" status="primary"> | ||
| </nb-icon> | ||
| </ng-template> | ||
| |
There was a problem hiding this comment.
¿Por qué hay un template y a la vez un html?
En vez de
template: `...`,Mover lo que está en el template al html y usar
templateUrl: './arbol-cuentas-contables.component.html',para centralizar el html en un solo lado...
Ejemplo:
There was a problem hiding this comment.
Como está es un componente reutilizable, se utiliza en otros lugares dentro del proyecto.
el cambio realizado fue sobre el componente ya existente en el proyecto como se puede ver.
En este caso el componente es para el manejo de los iconos según el estado y tipo de nodo.
Para "centralizar" el HTML habría que hacer una refactorizacion en otros componentes también.
Mi recomendación es trabajar la refactorizacion como otro issue dónde se contemple extraer los componentes genéricos reutilizables en un lugar específico dentro de la estructura del proyecto. No esta mal desacoplar los HTML en varios componentes concretos, esto mejora la lectura,
mantenibilidad y reusabilidad del código
Edicion nodo padre restringida.
Ajustes en look and feel arbol contable
#141