From 4ded7526ebc6e13ff64452a901b83d41aba10b64 Mon Sep 17 00:00:00 2001
From: Axel Peytavin <peytavin@stanford.edu>
Date: Thu, 8 Feb 2024 15:02:08 -0800
Subject: [PATCH 1/3] Refactor persona buttons and form layout

---
 natural_frontend/templates/queryForm.html | 95 ++++++++++++++++-------
 1 file changed, 67 insertions(+), 28 deletions(-)
diff --git a/natural_frontend/templates/queryForm.html b/natural_frontend/templates/queryForm.html
index 02ff079..adb5f50 100644
--- a/natural_frontend/templates/queryForm.html
+++ b/natural_frontend/templates/queryForm.html
@@ -95,9 +95,9 @@
         max-width: 800px;
         margin: 0 auto;
       }
-      .persona-buttons button {
+      .persona-buttons-container button {
         margin: 5px;
-        padding: 10px;
+        padding: 10px 20px;
         border-radius: 5px;
         border: none;
         background-color: #f0f0f0; /* A neutral color for non-selected buttons */
@@ -105,13 +105,13 @@
         transition: background-color 0.3s;
         background-image: none;
       }
-      .persona-buttons button.active {
+      .persona-buttons-container button.active {
         background-color: #a1eafb; /* A highlighted color for the active/selected button */
       }
-      .persona-buttons {
+      .persona-buttons-container {
         margin-bottom: 20px;
       }
-      .persona-buttons h2 {
+      .persona-buttons-container h2 {
         margin-bottom: 5px;
       }
       .generate-btn {
@@ -165,9 +165,33 @@
         flex-direction: row;
         align-items: center;
         justify-content: center;
+      }
+
+      .persona-buttons-container {
+        display: flex;
+        flex-direction: row;
+        align-items: center;
+        flex-wrap: wrap;
+        justify-content: center;
         margin-bottom: 20px;
       }
 
+      .form-container {
+        width: 600px;
+        display: flex;
+        flex-direction: column;
+        align-items: center;
+      }
+
+      .form-container h2 {
+        margin-bottom: 20px;
+      }
+
+      .persona-buttons-container button.active {
+        background-color: #a1eafb; /* Highlight color for the active/selected button */
+        color: #333; /* Text color for the active/selected button */
+      }
+
       @keyframes spin {
         0% {
           transform: rotate(0deg);
@@ -188,7 +212,7 @@
     </style>
   </head>
   <body>
-    <form action="/gen_frontend" method="post">
+    <form action="/gen_{{ frontend_endpoint }}" method="post" class="form-container">
       <div class="logo-container">
         <img
           src="/static/natural_frontend_logo.png"
@@ -197,33 +221,22 @@
         />
         <h1>Natural Frontend</h1>
       </div>
-      <p>Pick your persona for your API experience, we'll handle the rest!</p>
       <div>
-        <label for="persona">Who are you? </label>
+        <h2>Who are you?</h2>
+        <div class="persona-buttons-container">
+          {% for persona in potential_personas %}
+          <button type="button" onclick="selectPersona('{{persona.persona}}', '{{ persona.description }}', event)">
+            {{ persona.persona }}
+          </button>
+          {% endfor %}
+          <button type="button" onclick="selectPersona('Other', '', event)">Other</button>
+        </div>
         <div class="input-icon">
           <i class="fa fa-user icon"></i>
-          <input
-            type="text"
-            id="persona"
-            name="persona"
-            class="input-with-icon"
-            placeholder="{{ potential_personas[0].persona }}: {{ potential_personas[0].description }}"
-            required
-          />
+          <input type="text" id="persona" name="persona" class="input-with-icon" placeholder="Select your persona" required />
         </div>
       </div>
 
-      <div class="persona-buttons">
-        <h2>Potential Personas for this API</h2>
-        {% for persona in potential_personas %}
-        <button
-          type="button"
-          onclick="selectPersona('{{persona.persona}}', '{{ persona.description }}')"
-        >
-          {{ persona.persona }}
-        </button>
-        {% endfor %}
-      </div>
       <div>
         <div id="buttonOrSpinner">
           <button type="submit" class="generate-btn" onclick="showSpinner()">
@@ -235,10 +248,36 @@ <h2>Potential Personas for this API</h2>
     </form>
     <script>
       function selectPersona(persona, description) {
-        document.getElementById("persona").value = persona + ": " + description;
+        const personaInput = document.getElementById('persona');
+        const buttons = document.querySelectorAll('.persona-buttons-container button');
+      
+        // Remove 'active' class from all buttons
+        buttons.forEach(button => {
+          button.classList.remove('active');
+        });
+      
+        // Add 'active' class to the clicked button
+        // This requires passing an additional parameter to the function to identify the clicked button
+        if(persona !== 'Other') {
+          event.currentTarget.classList.add('active');
+          personaInput.setAttribute('readonly', 'readonly'); // Make it readonly
+          personaInput.value = persona + ": " + description; // Set the selected persona
+        } else {
+          // If "Other" is clicked, make sure to handle it accordingly
+          personaInput.removeAttribute('readonly'); // Allow text entry
+          personaInput.value = ''; // Clear the input field
+          personaInput.placeholder = 'Enter your persona'; // Update placeholder
+          personaInput.focus(); // Focus on the input field
+          event.currentTarget.classList.add('active');
+        }
       }
+      
 
       function showSpinner() {
+        // Only if there is value in the input field
+        if (document.getElementById("persona").value === "") {
+          return;
+        }
         document.querySelector(".generate-btn").style.display = "none"; // Hide button
         document.getElementById("spinner").style.display = "block"; // Show spinner
 

From 5d693212136190e0877481d4bee8421526c3a004 Mon Sep 17 00:00:00 2001
From: Axel Peytavin <peytavin@stanford.edu>
Date: Thu, 8 Feb 2024 15:02:32 -0800
Subject: [PATCH 2/3] Allow to modify frontend endpoint properly

---
 examples/social_network/main.py      | 8 ++++++--
 natural_frontend/natural_frontend.py | 7 ++++---
 2 files changed, 10 insertions(+), 5 deletions(-)

diff --git a/examples/social_network/main.py b/examples/social_network/main.py
index 14937c8..5746133 100644
--- a/examples/social_network/main.py
+++ b/examples/social_network/main.py
@@ -4,7 +4,7 @@
 from datetime import datetime
 import json
 
-from natural_frontend.natural_frontend import NaturalFrontend
+from natural_frontend.natural_frontend import NaturalFrontend, NaturalFrontendOptions
 
 app = FastAPI()
 
@@ -12,7 +12,11 @@
 with open("creds.json") as f:
     creds = json.load(f)
 
-app = NaturalFrontend(app, openai_api_key=creds["key"])
+app = NaturalFrontend(
+    app,
+    openai_api_key=creds["key"],
+    options=NaturalFrontendOptions(frontend_endpoint=""),
+)
 
 
 class User(BaseModel):
diff --git a/natural_frontend/natural_frontend.py b/natural_frontend/natural_frontend.py
index 10a04dd..bb42f2e 100644
--- a/natural_frontend/natural_frontend.py
+++ b/natural_frontend/natural_frontend.py
@@ -115,9 +115,9 @@ async def on_startup():
         frontend_generator.seed_prompt("FastAPI")
         frontend_generator.add_api_source(aggregated_api_source)
 
-        print("Natural Frontend was initiated successfully")
+        logging.info("Natural Frontend was initiated successfully")
 
-    @app.get(f"/{frontend_endpoint}/", response_class=HTMLResponse)
+    @app.get(f"/{frontend_endpoint}", response_class=HTMLResponse)
     async def frontend(request: Request):
         cache_key = "frontend_personas"
 
@@ -206,10 +206,11 @@ def parse_potential_personas(personas: str, retries=5):
                     "pink",
                     "lightblue",
                 ],  # Replace with your actual colors
+                "frontend_endpoint": f"gen_{frontend_endpoint}"
             },
         )
 
-    @app.post(f"/gen_{frontend_endpoint}/", response_class=HTMLResponse)
+    @app.post(f"/gen_{frontend_endpoint}", response_class=HTMLResponse)
     async def handle_form(request: Request, persona: str = Form(...)):
         scheme = request.url.scheme
         server_host = request.headers.get('host')

From 0f3c8047b03b62898de65d7254b3e65097578d23 Mon Sep 17 00:00:00 2001
From: Axel Peytavin <peytavin@stanford.edu>
Date: Thu, 8 Feb 2024 15:38:28 -0800
Subject: [PATCH 3/3] Remove trailing whitespace in natural_frontend.py

---
 natural_frontend/natural_frontend.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/natural_frontend/natural_frontend.py b/natural_frontend/natural_frontend.py
index bb42f2e..9d663fd 100644
--- a/natural_frontend/natural_frontend.py
+++ b/natural_frontend/natural_frontend.py
@@ -226,7 +226,7 @@ async def handle_form(request: Request, persona: str = Form(...)):
         # With the query in hand, send it to the NLP model
         # Handle the processed query
         response_content = frontend_generator.generate_frontend_code(
-            persona, full_url, options.colors, 
+            persona, full_url, options.colors
         )
 
         cache.set(cache_key, response_content)
