From 0f72048b116bea756939e07b2042fe73ccebd242 Mon Sep 17 00:00:00 2001
From: Arya Kiran <arya@projectsegfau.lt>
Date: Fri, 27 Dec 2024 14:40:02 +0530
Subject: [PATCH] move translate buttons up for better mobile usability (closes
 #16)

---
 pages/index.go   |  2 ++
 views/index.html | 50 ++++++++++++++++++++++++------------------------
 2 files changed, 27 insertions(+), 25 deletions(-)

diff --git a/pages/index.go b/pages/index.go
index 0319b88..31824a2 100644
--- a/pages/index.go
+++ b/pages/index.go
@@ -97,9 +97,11 @@ func HandleIndex(c *fiber.Ctx) error {
 	if engine != "" && originalText != "" && from != "" && to != "" {
 		if engine == "all" {
 			transmany = libmozhi.TranslateAll(to, from, originalText)
+			translationExists = true
 		} else if engine == "some" {
 			// The error doesn't really matter since it just checks if the engines are valid, which is already checked in the code at the beginning of the func
 			transmany, _ = libmozhi.TranslateSome(enginesSome.Engines, to, from, originalText)
+			translationExists = true
 		} else {
 			translation, tlerr = libmozhi.Translate(engine, to, from, originalText)
 			if tlerr != nil {
diff --git a/views/index.html b/views/index.html
index 7562a00..b87496a 100644
--- a/views/index.html
+++ b/views/index.html
@@ -73,6 +73,22 @@
         </select>
       </div>
     </div>
+      {{ if .TranslationExists }}
+      <div style="display:flex;
+                  justify-content:space-around;
+                  align-items:center">
+        <button class="wrap" type="button" onclick="copyToClipboard()">Copy the translation</button>
+        {{ if and .Engine .From .To .OriginalText }}
+        <p>
+          <a id="url"
+             class="button"
+             onclick="copyLinkToClipboard(event)"
+             href="/?engine={{.Engine}}&from={{.From}}&to={{.To}}&text={{.OriginalText}}">Copy translation link</a>
+        </p>
+        {{ end }}
+        <button class="wrap" type="submit">Translate!</button>
+      </div>
+      {{ end }}
     <div class="center-area2 item-wrapper">
       Source Text:
       <textarea autofocus class="item" id="input" name="text" dir="auto" placeholder="Enter Text Here">
@@ -128,31 +144,15 @@
       <br>
       {{ end }}
       {{ end }}
-      <div style="display:flex;
-                  justify-content:space-around;
-                  align-items:center">
-        {{ if .TranslationExists }}
-        <button class="wrap" type="button" onclick="copyToClipboard()">Copy the translation</button>
-        {{ end }}
-        {{ if and .Engine .From .To .OriginalText }}
-        <p>
-          <a id="url"
-             class="button"
-             onclick="copyLinkToClipboard(event)"
-             href="/?engine={{.Engine}}&from={{.From}}&to={{.To}}&text={{.OriginalText}}">Copy translation link</a>
-        </p>
-        {{ end }}
-        {{ if .TranslationExists }}
-        <button class="wrap" type="submit">Translate!</button>
-      {{ else }}
-        <button class="wrap"
-                style="margin-left:auto;
-                       position:relative;
-                       left:-10%;
-                       margin-top: 1%"
-                type="submit">Translate!</button>
-        {{ end }}
-      </div>
+	  {{ if not .TranslationExists }}
+	  <button class="wrap"
+              style="margin-left:auto;
+                     position:relative;
+                     left:-10%;
+                     margin-top: 1%"
+              type="submit">Translate!</button>
+	  {{ end }}
+
     </div>
     {{ if eq .Engine "yandex" }}
     {{ template "yandex_extras" . }}