Fix ID collision bug when adding components after deletion and modal footer visibility

- Fixed ID assignment in ResourceComponentService, CustomComponentService, ResourceDelimiterService, and ResourceTypeService to use Max(ID)+1 instead of Count+1
- This prevents ID collisions when adding new items after deletions
- Fixed AddModal footer being cut off by moving it inside modern-modal-content container
- Updated modal CSS to properly handle scrolling body with fixed footer using flexbox layout

Author: Bryan Soltis

src/Components/Modals/AddModal.razor

@@ -272,11 +272,11 @@
                 </div>
             </div>
         </div>
-    </div>
 
-<div class="modern-modal-footer">
-    <button title="Add" @onclick="Save" class="modern-btn-success">Add</button>
-    <button title="Cancel" @onclick="Cancel" class="modern-btn-secondary">Cancel</button>
+    <div class="modern-modal-footer">
+        <button title="Add" @onclick="Save" class="modern-btn-success">Add</button>
+        <button title="Cancel" @onclick="Cancel" class="modern-btn-secondary">Cancel</button>
+    </div>
 </div>
 
 @code {

src/Services/CustomComponentService.cs

@@ -193,18 +193,16 @@ public async Task<ServiceResponse> PostItemAsync(CustomComponent item)
                     // Set the new id
                     if (item.Id == 0)
                     {
-                        item.Id = items.Count + 1;
+                        // Use max ID + 1 instead of count + 1 to avoid ID collisions after deletions
+                        item.Id = items.Count > 0 ? items.Max(x => x.Id) + 1 : 1;
                     }
 
                     int position = 1;
                     items = [.. items.OrderBy(x => x.SortOrder)];
 
                     if (item.SortOrder == 0)
                     {
-                        if (items.Count > 0)
-                        {
-                            item.Id = items.Max(t => t.Id) + 1;
-                        }
+                        item.SortOrder = items.Count + 1;
                     }
 
                     // Determine new item id

src/Services/ResourceComponentService.cs

@@ -124,7 +124,8 @@ public async Task<ServiceResponse> PostItemAsync(ResourceComponent item)
                         // Set the new id
                         if (item.Id == 0)
                         {
-                            item.Id = items.Count + 1;
+                            // Use max ID + 1 instead of count + 1 to avoid ID collisions after deletions
+                            item.Id = items.Count > 0 ? items.Max(x => x.Id) + 1 : 1;
                         }
 
                         int position = 1;

src/Services/ResourceDelimiterService.cs

@@ -150,7 +150,8 @@ public async Task<ServiceResponse> PostItemAsync(ResourceDelimiter item)
                     // Set the new id
                     if (item.Id == 0)
                     {
-                        item.Id = items.Count + 1;
+                        // Use max ID + 1 instead of count + 1 to avoid ID collisions after deletions
+                        item.Id = items.Count > 0 ? items.Max(x => x.Id) + 1 : 1;
                     }
                     item.Enabled = true;
                     int position = 1;

src/Services/ResourceTypeService.cs

@@ -139,7 +139,8 @@ public async Task<ServiceResponse> PostItemAsync(ResourceType item)
                     // Set the new id
                     if (item.Id == 0)
                     {
-                        item.Id = items.Count + 1;
+                        // Use max ID + 1 instead of count + 1 to avoid ID collisions after deletions
+                        item.Id = items.Count > 0 ? items.Max(x => x.Id) + 1 : 1;
                     }
 
                     // Determine new item id

src/wwwroot/css/modern-components.css

@@ -2966,7 +2966,7 @@ h3.blazored-modal-title,
     padding: 16px 24px;
     border-top: 1px solid var(--color-border);
     background-color: var(--color-bg-secondary);
-    margin-top: auto; /* Push footer to bottom when content is short */
+    flex-shrink: 0; /* Prevent footer from shrinking */
 }
 
 /* Modal Animations */
@@ -2996,7 +2996,6 @@ h3.blazored-modal-title,
     display: flex;
     flex-direction: column;
     max-height: 85vh;
-    overflow-y: auto; /* Enable scrolling on the modal content */
     line-height: 1.6; /* Consistent line height for readability */
     font-size: 14px; /* Consistent font size */
 }
@@ -3015,7 +3014,6 @@ h3.blazored-modal-title,
     flex: 1;
     overflow-y: auto;
     padding: 1.5rem;
-    max-height: calc(85vh - 120px); /* Account for header and footer */
     line-height: 1.6; /* Consistent line height for readability */
     font-size: 14px; /* Consistent font size */
 }