From 04fa43e1df374a876ab086fb7601e7d007cc5120 Mon Sep 17 00:00:00 2001
From: Aman Agrawal <amanagr@zulip.com>
Date: Thu, 21 Aug 2025 16:16:23 +0530
Subject: [PATCH] slack_import: Don't use `a` tag for form submission.

This can mislead user into thinking this is a link they can
open in a new tab, which we don't want to allow.
---
 templates/zerver/slack_import.html    |  2 +-
 web/styles/portico/portico_signin.css | 12 ++++++++++++
 2 files changed, 13 insertions(+), 1 deletion(-)

diff --git a/templates/zerver/slack_import.html b/templates/zerver/slack_import.html
index f9e7dd2e0e..0fc5b8468c 100644
--- a/templates/zerver/slack_import.html
+++ b/templates/zerver/slack_import.html
@@ -100,7 +100,7 @@
             </form>
             <div class="bottom-text">
                 {% trans %}
-                Or <a href="#" id="cancel-slack-import">create organization</a> without importing data.
+                Or <span id="cancel-slack-import" tabindex="0">create organization</span> without importing data.
                 {% endtrans %}
             </div>
             {% endif %}
diff --git a/web/styles/portico/portico_signin.css b/web/styles/portico/portico_signin.css
index 4f045a0cb0..2b7527da3d 100644
--- a/web/styles/portico/portico_signin.css
+++ b/web/styles/portico/portico_signin.css
@@ -1556,3 +1556,15 @@ button#register_auth_button_gitlab {
     overflow: hidden;
     white-space: nowrap;
 }
+
+#cancel-slack-import {
+    color: hsl(200deg 100% 40%);
+    text-decoration: none;
+
+    &:hover,
+    &:focus {
+        cursor: pointer;
+        color: hsl(200.16deg 100% 25.1%);
+        text-decoration: underline;
+    }
+}