From b7c8cfff764a95cfb910b320758cdf3d62a02dc0 Mon Sep 17 00:00:00 2001
From: Marco Minerva <marco.minerva@gmail.com>
Date: Thu, 20 Feb 2025 12:19:20 +0100
Subject: [PATCH] Update button types in Ask.razor and improve CSS focus style

- Specified `Type` attributes for buttons in `Ask.razor` to clarify their intended behavior.
- Added `h1:focus` style in `app.css` to remove default outline for focused headings.
---
 SqlDatabaseVectorSearch/Components/Pages/Ask.razor | 6 +++---
 SqlDatabaseVectorSearch/wwwroot/css/app.css        | 8 ++++----
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/SqlDatabaseVectorSearch/Components/Pages/Ask.razor b/SqlDatabaseVectorSearch/Components/Pages/Ask.razor
index f9d4691..b1c282d 100644
--- a/SqlDatabaseVectorSearch/Components/Pages/Ask.razor
+++ b/SqlDatabaseVectorSearch/Components/Pages/Ask.razor
@@ -59,7 +59,7 @@
                                             </div>
                                             <div class="text-end bg-transparent">
                                                 <Tooltip Title="Copy to Clipboard" Color="TooltipColor.Dark" Placement="TooltipPlacement.Bottom">
-                                                    <Button Outline="false" @onclick="@(async () => await CopyToClipboard(message.Text))">
+                                                    <Button Type="ButtonType.Button" Outline="false" @onclick="@(async () => await CopyToClipboard(message.Text))">
                                                         <Icon @ref="clipboardIcon" Name="IconName.Clipboard" />
                                                     </Button>
                                                 </Tooltip>
@@ -84,10 +84,10 @@
             </span>
             <input type="text" @bind="@question" @bind:event="oninput" placeholder="Ask me anything..." class="form-control border-0" maxlength="2000" autofocus @onkeydown="HandleKeyDown" />
             <div class="input-group-text bg-transparent border-0">
-                <Button @ref="askButton" Color="ButtonColor.Primary" Disabled="@(isAsking || string.IsNullOrWhiteSpace(question))" @onclick="AskQuestion">
+                <Button Type="ButtonType.Submit" @ref="askButton" Color="ButtonColor.Primary" Disabled="@(isAsking || string.IsNullOrWhiteSpace(question))" @onclick="AskQuestion">
                     <Icon Name="IconName.Send" />
                 </Button>
-                <Button @ref="resetButton" Class="ms-2" Color="ButtonColor.Secondary" Disabled="@isAsking" @onclick="Reset">
+                <Button Type="ButtonType.Reset" @ref="resetButton" Class="ms-2" Color="ButtonColor.Secondary" Disabled="@isAsking" @onclick="Reset">
                     <Icon CustomIconName="bi bi-x-lg" />
                 </Button>
             </div>
diff --git a/SqlDatabaseVectorSearch/wwwroot/css/app.css b/SqlDatabaseVectorSearch/wwwroot/css/app.css
index a1da72e..947645a 100644
--- a/SqlDatabaseVectorSearch/wwwroot/css/app.css
+++ b/SqlDatabaseVectorSearch/wwwroot/css/app.css
@@ -11,10 +11,6 @@
     --bb-sidebar2-nav-item-group-background-color: rgba(var(--bb-sidebar2-nav-item-text-active-color-rgb), 0.08);
 }
 
-h1:focus {
-    outline: none;
-}
-
 .bb-sidebar2 nav .nav-item a {
     color: var(--bb-sidebar2-nav-item-text-color) !important;
 }
@@ -29,6 +25,10 @@ h1:focus {
         color: var(--bb-sidebar2-nav-item-text-active-color) !important;        
     }
 
+h1:focus {
+    outline: none;
+}
+
 .valid.modified:not([type=checkbox]) {
     outline: 1px solid #26b050;
 }