From c34d6826a8036ef61f0ee182398337a7011658c1 Mon Sep 17 00:00:00 2001
From: Galen Charlton <gmc@equinoxOLI.org>
Date: Fri, 28 Apr 2023 11:11:36 +0000
Subject: [PATCH] LP#1967328: (follow-up) tweak layout

The modal to add permission maps is now wider and uses
a bit less vertical space, allowing the user to see more
of the pending mappings at once.

Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>
---
 .../app/staff/admin/server/perm-group-map-dialog.component.html   | 8 +++-----
 .../src/app/staff/admin/server/perm-group-map-dialog.component.ts | 4 +---
 .../eg2/src/app/staff/admin/server/perm-group-tree.component.ts   | 2 +-
 3 files changed, 5 insertions(+), 9 deletions(-)

diff --git a/Open-ILS/src/eg2/src/app/staff/admin/server/perm-group-map-dialog.component.html b/Open-ILS/src/eg2/src/app/staff/admin/server/perm-group-map-dialog.component.html
index 00cbd13cdd..7a994a339a 100644
--- a/Open-ILS/src/eg2/src/app/staff/admin/server/perm-group-map-dialog.component.html
+++ b/Open-ILS/src/eg2/src/app/staff/admin/server/perm-group-map-dialog.component.html
@@ -25,13 +25,13 @@
     </div>
     <ng-container *ngFor="let map of newPermMaps.controls; let i = index">
       <ng-container [formGroup]="map">
+        <hr>
         <div class="row mt-1 pt-1">
           <div class="col-lg-12">
-            <hr>
             <h5 i18n>{{map.controls.label.value}}</h5>
           </div>
         </div>
-        <div class="row row-cols-4 mt-1 pt-1">
+        <div class="row row-cols-5 mt-1 pt-1">
           <div class="col">
             <label [attr.for]="'depth-'+map.controls.id.value"
               i18n>Depth
@@ -52,9 +52,7 @@
             <input type="checkbox" formControlName="grantable"
               id="grantable-{{map.controls.id.value}}">
           </div>
-        </div>
-        <div class="row mt-1 pt-1">
-          <div class="col-lg-12">
+          <div class="col">
             <button type="button" class="btn btn-danger"
               (click)="remove(i)"
               i18n>Remove
diff --git a/Open-ILS/src/eg2/src/app/staff/admin/server/perm-group-map-dialog.component.ts b/Open-ILS/src/eg2/src/app/staff/admin/server/perm-group-map-dialog.component.ts
index 3ac357ce15..16a87d223c 100644
--- a/Open-ILS/src/eg2/src/app/staff/admin/server/perm-group-map-dialog.component.ts
+++ b/Open-ILS/src/eg2/src/app/staff/admin/server/perm-group-map-dialog.component.ts
@@ -162,6 +162,4 @@ export class PermGroupMapDialogComponent
     ngOnDestroy(): void {
         this.onDestroy.next();
     }
-}   
-
-
+}
diff --git a/Open-ILS/src/eg2/src/app/staff/admin/server/perm-group-tree.component.ts b/Open-ILS/src/eg2/src/app/staff/admin/server/perm-group-tree.component.ts
index 1b45e2197f..56fe908d8c 100644
--- a/Open-ILS/src/eg2/src/app/staff/admin/server/perm-group-tree.component.ts
+++ b/Open-ILS/src/eg2/src/app/staff/admin/server/perm-group-tree.component.ts
@@ -361,7 +361,7 @@ export class PermGroupTreeComponent implements OnInit {
     }
 
     openAddDialog() {
-        this.addMappingDialog.open().subscribe(
+        this.addMappingDialog.open({size: 'lg'}).subscribe(
             modified => {
                 if (modified) {
                     this.createMapString.current().then(msg => this.toast.success(msg));
-- 
2.11.0