From 9e9ae328878bb8502e95cc6f971098b792fd1854 Mon Sep 17 00:00:00 2001
From: Glenn Rice <grice1@missouriwestern.edu>
Date: Sun, 17 Mar 2024 05:20:52 -0500
Subject: [PATCH] Switch to the QR code being a link and fix wording.

---
 .../TwoFactorAuthentication.html.ep              | 16 +++++++---------
 1 file changed, 7 insertions(+), 9 deletions(-)

diff --git a/templates/ContentGenerator/TwoFactorAuthentication.html.ep b/templates/ContentGenerator/TwoFactorAuthentication.html.ep
index 50a30eac88..dff26753df 100644
--- a/templates/ContentGenerator/TwoFactorAuthentication.html.ep
+++ b/templates/ContentGenerator/TwoFactorAuthentication.html.ep
@@ -2,19 +2,17 @@
 %
 % if ($otp_link) {
 	<p>
-		<%= maketext('To set up one-time password generation, scan the QRCode below with an authenticator app '
+		<%= maketext('To set up one-time password generation, scan the QR code below with an authenticator app '
 			. '(such as Google Authenticator, Microsoft Authenticator, Twilio Authy, etc.) installed on a mobile '
-			. 'device.') =%>
+			. 'device. Alternatively, after installing an authenticator app on a mobile device, open this page on '
+			. 'that device, and click on the QR code below.') =%>
 	</p>
 	<div class="mb-3 d-flex align-items-center">
-		<%= image 'data:image/png;base64,' . b64_encode($otp_qrcode),
-			alt => maketext('One-time password generator setup QR Code'), class => 'mx-auto' =%>
+		<%= link_to $otp_link, class => 'mx-auto', begin =%>
+			<%= image 'data:image/png;base64,' . b64_encode($otp_qrcode),
+				alt => maketext('One-time password generator setup QR Code') =%>
+		<% end =%>
 	</div>
-	<p>
-		<%== maketext('Alternately, after installing an authenticator app on a mobile device, '
-			. 'open this page on that device, and click <a href="[_1]">here</a>.',
-			$otp_link) =%>
-	</p>
 	<p>
 		<%= maketext('Once the authenticator app is set up, enter the code it generates below.') =%>
 	</p>