Merge pull request #2670 from shubhamkumar9199/fix/view-gl-account-html

WEB-319 Fix View GL Account Component: HTML Structure & SCSS Formatting

src/app/accounting/chart-of-accounts/view-gl-account/view-gl-account.component.html

@@ -1,4 +1,4 @@
-<div class="layout-row align-end gap-2px responsive-column container m-b-20">
+<div class="container m-b-20 layout-row layout-lt-md-column align-end gap-2percent">
   <span *mifosxHasPermission="'CREATE_GLACCOUNT'">
     <button
       mat-raised-button
@@ -11,90 +11,79 @@
       {{ 'labels.buttons.Subledger Account' | translate }}
     </button>
   </span>
+
   <span *mifosxHasPermission="'UPDATE_GLACCOUNT'">
     <button mat-raised-button color="primary" [routerLink]="['edit']">
       <fa-icon icon="edit" class="m-r-10"></fa-icon>
       {{ 'labels.buttons.Edit' | translate }}
     </button>
   </span>
+
   <span *mifosxHasPermission="'UPDATE_GLACCOUNT'">
     <button mat-raised-button color="warn" *ngIf="!glAccount.disabled" (click)="changeGlAccountState()">
       <fa-icon icon="lock" class="m-r-10"></fa-icon>
       {{ 'labels.buttons.Disable' | translate }}
     </button>
   </span>
+
   <span *mifosxHasPermission="'UPDATE_GLACCOUNT'">
     <button mat-raised-button color="accent" *ngIf="glAccount.disabled" (click)="changeGlAccountState()">
       <fa-icon icon="lock-open" class="m-r-10"></fa-icon>
       {{ 'labels.buttons.Enable' | translate }}
     </button>
   </span>
+
   <button mat-raised-button color="warn" (click)="deleteGlAccount()" *mifosxHasPermission="'DELETE_GLACCOUNT'">
     <fa-icon icon="trash" class="m-r-10"></fa-icon>
     {{ 'labels.buttons.Delete' | translate }}
   </button>
 </div>
 
+<!-- GL Account Details -->
 <div class="container">
   <mat-card>
     <mat-card-content>
-      <div class="layout-row-wrap">
-        <div class="flex-50 header">
-          {{ 'labels.inputs.Account Type' | translate }}
+      <div class="content-rows">
+        <div class="content-row">
+          <div class="label">{{ 'labels.inputs.Account Type' | translate }}</div>
+          <div class="value">{{ 'labels.inputs.accounting.' + glAccount.type?.value | translate }}</div>
         </div>
 
-        <div class="flex-50">
-          {{ 'labels.inputs.accounting.' + glAccount.type?.value | translate }}
+        <div class="content-row">
+          <div class="label">{{ 'labels.inputs.GL Code' | translate }}</div>
+          <div class="value">{{ glAccount.glCode }}</div>
         </div>
 
-        <div class="flex-50 header">
-          {{ 'labels.inputs.GL Code' | translate }}
+        <div class="content-row" *ngIf="glAccount.parent">
+          <div class="label">{{ 'labels.inputs.Parent Account Name' | translate }}</div>
+          <div class="value">
+            <a
+              class="tab-link"
+              [routerLink]="['/accounting/chart-of-accounts/gl-accounts/view/' + glAccount.parent.id]"
+            >
+              <mifosx-gl-account-display [glAccount]="glAccount.parent"></mifosx-gl-account-display>
+            </a>
+          </div>
         </div>
 
-        <div class="flex-50">
-          {{ glAccount.glCode }}
+        <div class="content-row" *ngIf="glAccount.tagId.id">
+          <div class="label">{{ 'labels.inputs.Tag' | translate }}</div>
+          <div class="value">{{ glAccount.tagId.name }}</div>
         </div>
 
-        <div class="flex-50 header" *ngIf="glAccount.parent">
-          {{ 'labels.inputs.Parent Account Name' | translate }}
+        <div class="content-row">
+          <div class="label">{{ 'labels.inputs.Account Usage' | translate }}</div>
+          <div class="value">{{ glAccount.usage?.value }}</div>
         </div>
 
-        <div class="flex-50" *ngIf="glAccount.parent">
-          <a class="tab-link" [routerLink]="['/accounting/chart-of-accounts/gl-accounts/view/' + glAccount.parent.id]">
-            <mifosx-gl-account-display [glAccount]="glAccount.parent"></mifosx-gl-account-display>
-          </a>
+        <div class="content-row">
+          <div class="label">{{ 'labels.inputs.Manual Entries Allowed' | translate }}</div>
+          <div class="value">{{ glAccount.manualEntriesAllowed | yesNo }}</div>
         </div>
 
-        <div class="flex-50 header" *ngIf="glAccount.tagId.id">
-          {{ 'labels.inputs.Tag' | translate }}
-        </div>
-
-        <div class="flex-50" *ngIf="glAccount.tagId.id">
-          {{ glAccount.tagId.name }}
-        </div>
-
-        <div class="flex-50 header">
-          {{ 'labels.inputs.Account Usage' | translate }}
-        </div>
-
-        <div class="flex-50">
-          {{ glAccount.usage?.value }}
-        </div>
-
-        <div class="flex-50 header">
-          {{ 'labels.inputs.Manual Entries Allowed' | translate }}
-        </div>
-
-        <div class="flex-50">
-          {{ glAccount.manualEntriesAllowed | yesNo }}
-        </div>
-
-        <div class="flex-50 header" *ngIf="glAccount.description">
-          {{ 'labels.inputs.Description' | translate }}
-        </div>
-
-        <div class="flex-50" *ngIf="glAccount.description">
-          {{ glAccount.description }}
+        <div class="content-row" *ngIf="glAccount.description">
+          <div class="label">{{ 'labels.inputs.Description' | translate }}</div>
+          <div class="value">{{ glAccount.description }}</div>
         </div>
       </div>
     </mat-card-content>

src/app/accounting/chart-of-accounts/view-gl-account/view-gl-account.component.scss

@@ -1,14 +1,102 @@
 .container {
   max-width: 37rem;
+  margin: 0 auto;
+  padding: 1rem;
+
+  mat-card {
+    margin-bottom: 1rem;
+    padding: 1.5rem;
+    box-shadow: 0 2px 4px rgb(0 0 0 / 10%);
+    border: 1px solid var(--border-color, #ddd);
+    border-radius: 8px;
+    background-color: var(--card-background, #fff);
+    transition:
+      background-color 0.3s ease,
+      border-color 0.3s ease;
+  }
+
+  .layout-row-wrap {
+    display: grid;
+    grid-template-columns: 50% 50%;
+    gap: 1rem;
+
+    .flex-50 {
+      padding: 0.5rem;
+
+      &.mat-body-strong {
+        color: var(--label-color, #555);
+        font-weight: 600;
+      }
+    }
+  }
 
   .content {
     div {
       margin: 1rem 0;
       word-wrap: break-word;
 
-      &.header {
-        font-weight: 500;
+      span {
+        display: block;
+        padding: 0.25rem 0;
+        color: var(--text-color, inherit);
       }
     }
   }
+
+  .content-rows {
+    display: flex;
+    flex-direction: column;
+    gap: 1rem;
+
+    .content-row {
+      display: flex;
+      flex-direction: row;
+      border-bottom: 1px solid var(--border-color-light, #eee);
+      padding-bottom: 0.5rem;
+
+      .label {
+        flex: 0 0 40%;
+        font-weight: 600;
+        color: var(--label-color, #555);
+        padding: 0.5rem 1rem 0.5rem 0;
+      }
+
+      .value {
+        flex: 0 0 60%;
+        padding: 0.5rem 0;
+        color: var(--text-color, inherit);
+
+        span {
+          display: block;
+          padding: 0.25rem 0;
+        }
+      }
+    }
+  }
+
+  .back-button-container {
+    margin-top: 2rem;
+  }
+
+  button {
+    transition: all 0.2s ease;
+
+    &:hover {
+      transform: translateY(-2px);
+    }
+  }
+}
+
+:host-context(.dark-theme) {
+  --border-color: #444;
+  --border-color-light: #3a3a3a;
+  --card-background: #2d2d2d;
+  --label-color: #b8b8b8;
+  --text-color: #e0e0e0;
+
+  .container {
+    mat-card {
+      box-shadow: 0 2px 4px rgb(0 0 0 / 30%);
+    }
+  }
 }