From e5cab126336a566f2325b2f58bbd1f00de9d5b54 Mon Sep 17 00:00:00 2001
From: johnnyq <johnny@pittpc.com>
Date: Tue, 4 Jun 2024 15:11:13 -0400
Subject: [PATCH] Updated UI for Restrict Client Access

---
 admin_user_add_modal.php  | 38 ++++++++++++++++++++++----------------
 admin_user_edit_modal.php | 18 ++++++++++++------
 2 files changed, 34 insertions(+), 22 deletions(-)

diff --git a/admin_user_add_modal.php b/admin_user_add_modal.php
index daeeaf20..bd9fb0bc 100644
--- a/admin_user_add_modal.php
+++ b/admin_user_add_modal.php
@@ -109,25 +109,31 @@
                         </div>
 
                         <div class="tab-pane fade" id="pills-user-access">
-                            
-                            <h5>Restrict Client Access</h5>
-                            <small class="text-muted">Leave Blank for Full access to all clients, no affect on users with the admin role.</small>
 
-                            <?php
-
-                            $sql_client_select = mysqli_query($mysqli, "SELECT * FROM clients WHERE client_archived_at IS NULL ORDER BY client_name ASC");
-                            while ($row = mysqli_fetch_array($sql_client_select)) {
-                                $client_id = intval($row['client_id']);
-                                $client_name = nullable_htmlentities($row['client_name']);
-
-                            ?>
-
-                            <div class="form-group form-check">
-                                <input type="checkbox" class="form-check-input" name="clients[]" value="<?php echo $client_id; ?>">
-                                <label class="form-check-label ml-2"><?php echo $client_name; ?></label>
+                            <div class="alert alert-info">
+                                Leave Blank for Full access to all clients, no affect on users with the admin role.
                             </div>
 
-                            <?php } ?>
+                            <ul class="list-group">
+
+                                <?php
+
+                                $sql_client_select = mysqli_query($mysqli, "SELECT * FROM clients WHERE client_archived_at IS NULL ORDER BY client_name ASC");
+                                while ($row = mysqli_fetch_array($sql_client_select)) {
+                                    $client_id = intval($row['client_id']);
+                                    $client_name = nullable_htmlentities($row['client_name']);
+
+                                ?>
+                                <li class="list-group-item">
+                                    <div class="form-check">
+                                        <input type="checkbox" class="form-check-input" name="clients[]" value="<?php echo $client_id; ?>">
+                                        <label class="form-check-label ml-2"><?php echo $client_name; ?></label>
+                                    </div>
+                                </li>
+
+                                <?php } ?>
+
+                            </ul>
 
                         </div>
 
diff --git a/admin_user_edit_modal.php b/admin_user_edit_modal.php
index 7115c652..9d3f14fd 100644
--- a/admin_user_edit_modal.php
+++ b/admin_user_edit_modal.php
@@ -130,9 +130,11 @@
 
                         <div class="tab-pane fade" id="pills-user-access<?php echo $user_id; ?>">
 
-                            <h6>Restrict Client Access To<br><small class="text-muted">Leave Blank for Full access to all clients, no affect on users with the admin role.</small></h6>
+                            <div class="alert alert-info">
+                                Leave Blank for Full access to all clients, no affect on users with the admin role.
+                            </div>
 
-                            <hr>
+                            <ul class="list-group">
 
                             <?php
 
@@ -143,13 +145,17 @@
 
                             ?>
 
-                            <div class="form-group form-check">
-                                <input type="checkbox" class="form-check-input" name="clients[]" value="<?php echo $client_id_select; ?>" <?php if (in_array($client_id_select, $client_access_array)) { echo "checked"; } ?>>
-                                <label class="form-check-label ml-2"><?php echo $client_name_select; ?></label>
-                            </div>
+                                <li class="list-group-item">
+                                    <div class="form-check">
+                                        <input type="checkbox" class="form-check-input" name="clients[]" value="<?php echo $client_id_select; ?>" <?php if (in_array($client_id_select, $client_access_array)) { echo "checked"; } ?>>
+                                        <label class="form-check-label ml-2"><?php echo $client_name_select; ?></label>
+                                    </div>
+                                </li>
 
                             <?php } ?>
 
+                            </ul>
+
                         </div>
 
                     </div>