feat: ui cleanup

4 files changed, 68 insertions, 32 deletions

diff --git a/src/marlin.web/Views/Agents/Create.cshtml b/src/marlin.web/Views/Agents/Create.cshtml
index 4f0f31a..e112eeb 100644
--- a/src/marlin.web/Views/Agents/Create.cshtml
+++ b/src/marlin.web/Views/Agents/Create.cshtml
@@ -1,8 +1,13 @@
 @using marlin.core.models
 
 <form method="post" asp-action="Create">
-    Name: <input type="text" name="@nameof(Agent.Name)" /><br/>
-    Max Concurrent Jobs: <input type="number" name="@nameof(Agent.MaxConcurrentJobs)" value="1"/><br/>
+    <dl>
+        <dt>Name</dt>
+        <dd><input type="text" name="@nameof(Agent.Name)" placeholder="myagent-1"/></dd>
+        
+        <dt>Max Concurrent Jobs</dt>
+        <dd><input type="number" name="@nameof(Agent.MaxConcurrentJobs)" value="1"/></dd>
+    </dl>
     <button type="submit">Create</button>
 </form>
 
diff --git a/src/marlin.web/Views/VCS/Create.cshtml b/src/marlin.web/Views/VCS/Create.cshtml
index e04592d..5f76192 100644
--- a/src/marlin.web/Views/VCS/Create.cshtml
+++ b/src/marlin.web/Views/VCS/Create.cshtml
@@ -4,7 +4,6 @@
 <form method="post" asp-action="Create">
     URL: <input type="text" name="@nameof(VersionControlSource.UpstreamUrl)" /><br/>
     Filter: <input type="text" name="@nameof(VersionControlSource.BranchFilter)" /><br/>
-    Main Branch: <input type="text" name="@nameof(VersionControlSource.MainBranch)" /><br/>
     Polling Interval: <input type="number" name="@nameof(VersionControlSource.MinPollingInterval)" /><br/>
     <button type="submit">Create</button>
 </form>
diff --git a/src/marlin.web/Views/VCS/Details.cshtml b/src/marlin.web/Views/VCS/Details.cshtml
index a9ef0ea..44fa91e 100644
--- a/src/marlin.web/Views/VCS/Details.cshtml
+++ b/src/marlin.web/Views/VCS/Details.cshtml
@@ -1,55 +1,73 @@
 @using marlin.core
 @using marlin.core.models
-@using marlin.web.Controllers
-@model (VersionControlSource Source, List<VCSController.SecretOption> SecretOptions)
+@model VersionControlSource
 
-<form method="post" asp-route-id="@Model.Source.Id" asp-action="Edit">
-    URL: <input type="text" name="@nameof(VersionControlSource.UpstreamUrl)" value="@Model.Source.UpstreamUrl"/><br/>
-    Filter: <input type="text" name="@nameof(VersionControlSource.BranchFilter)" value="@Model.Source.BranchFilter"/><br/>
-    Main Branch: <input type="text" name="@nameof(VersionControlSource.MainBranch)" value="@Model.Source.MainBranch"/><br/>
-    Polling Interval: <input type="number" name="@nameof(VersionControlSource.MinPollingInterval)" value="@Model.Source.MinPollingInterval"/><br/>
-    Access Secret:
-    <select name="@nameof(VersionControlSource.AccessSecretId)">
-        <option value="">(none)</option>
-        @foreach (var secret in Model.SecretOptions)
-        {
-            <option value="@secret.Id" selected="@(secret.Id == Model.Source.AccessSecretId)">@secret.Label</option>
-        }
-    </select><br/>
-    <button type="submit">Save</button>
-</form>
+<div class="row">
+    <div class="col">
+        <h3>Details</h3>
+        <hr/>
+        <form method="post" asp-route-id="@Model.Id" asp-action="Edit">
+            <dl>
+                <dt>URL</dt>
+                <dd><input type="text" name="@nameof(VersionControlSource.UpstreamUrl)" value="@Model.UpstreamUrl"/></dd>
 
+                <dt>Filter</dt>
+                <dd><input type="text" name="@nameof(VersionControlSource.BranchFilter)" value="@Model.BranchFilter"/></dd>
+
+                <dt>Polling Interval</dt>
+                <dd><input type="number" name="@nameof(VersionControlSource.MinPollingInterval)" value="@Model.MinPollingInterval"/></dd>
+            </dl>
+
+            <button type="submit">Save</button>
+            <a asp-action="ListSecret" asp-route-sourceId="@Model.Id">Secrets</a>
+            <a asp-action="List">Back</a>
+        </form>
+    </div>
+</div>
+<br/>
+
+<h3>Branches</h3>
+<hr/>
 <table>
     <thead>
     <tr>
-        <th></th>
         <th>branch</th>
         <th>commit</th>
-        <th>status</th>
+        <th>actions</th>
     </tr>
     </thead>
     <tbody>
-    @foreach (var branch in Model.Source.Branches)
+    @foreach (var branch in Model.Branches.OrderByDescending(b => b.Name == Model.MainBranch))
     {
-        <tr>
-            <td>@(branch.Name == Model.Source.MainBranch ? "@" : "")</td>
-            <td>@branch.Name</td>
+        <tr class="@(branch.Name == Model.MainBranch ? "highlighted" : "")">
+            <td>
+                @branch.Name
+                @if (branch.Name == Model.MainBranch)
+                {
+                    <span>(primary)</span>
+                }
+            </td>
             <td>@branch.Commit.Hash</td>
             <td>
                 @if (branch.Commit.BlobsFetched)
                 {
                     <a asp-action="Blobs" asp-route-commitId="@branch.CommitId" asp-route-sourceId="@branch.SourceId">blobs</a>
                 }
-                
+
                 @if (branch.Commit.WorkflowScanned)
                 {
                     <a asp-action="Workflow" asp-route-commitId="@branch.CommitId" asp-route-sourceId="@branch.SourceId">workflow</a>
                 }
+                
+                @if (branch.Name != Model.MainBranch)
+                {
+                    <form method="post" asp-action="SetMainBranch" asp-route-sourceId="@Model.Id" style="display:inline">
+                        <input type="hidden" name="branchName" value="@branch.Name"/>
+                        <button type="submit" class="btn btn-link p-0">make primary</button>
+                    </form>
+                }
             </td>
         </tr>
     }
     </tbody>
-</table>
-
-<a asp-action="ListSecret" asp-route-sourceId="@Model.Source.Id">Secrets</a>
-<a asp-action="List">Back</a>
+</table>
\ No newline at end of file
diff --git a/src/marlin.web/Views/VCS/ListSecret.cshtml b/src/marlin.web/Views/VCS/ListSecret.cshtml
index 25db7fe..84e3061 100644
--- a/src/marlin.web/Views/VCS/ListSecret.cshtml
+++ b/src/marlin.web/Views/VCS/ListSecret.cshtml
@@ -1,5 +1,19 @@
 @model (VersionControlSource Source, List<ISecret> Secrets)
 
+<form method="post" asp-action="SetAccessSecret" asp-route-sourceId="@Model.Source.Id">
+    <label>
+        Access Secret
+        <select name="secretId">
+            <option value="">(none)</option>
+            @foreach (var secret in Model.Secrets)
+            {
+                <option value="@secret.Id" selected="@(secret.Id == Model.Source.AccessSecretId)">@secret.Label</option>
+            }
+        </select>
+    </label>
+    <button type="submit">Save</button>
+</form>
+
 <a asp-action="CreateSecret" asp-route-sourceId="@Model.Source.Id">New</a>
 
 <table>
@@ -42,4 +56,4 @@
     </tbody>
 </table>
 
-<a asp-action="Details", asp-route-id="@Model.Source.Id">back</a>
+<a asp-action="Details" asp-route-id="@Model.Source.Id">back</a>