From a7b5c936913bdf671b83ca0697cdbd3630163e9c Mon Sep 17 00:00:00 2001
From: Yukino Song <nutosservice@gmail.com>
Date: Tue, 10 Sep 2024 18:13:44 +0800
Subject: [PATCH] Jump to PIN directly from tray pairing message

---
 src/system_tray.cpp                   |  2 +-
 src_assets/common/assets/web/pin.html | 27 ++++++++++++++-------------
 2 files changed, 15 insertions(+), 14 deletions(-)

diff --git a/src/system_tray.cpp b/src/system_tray.cpp
index fd6bfcb9..fff08267 100644
--- a/src/system_tray.cpp
+++ b/src/system_tray.cpp
@@ -376,7 +376,7 @@ namespace system_tray {
     tray.notification_icon = TRAY_ICON_LOCKED;
     tray.tooltip = PROJECT_NAME;
     tray.notification_cb = []() {
-      launch_ui_with_path("/pin");
+      launch_ui_with_path("/pin#PIN");
     };
     tray_update(&tray);
   }
diff --git a/src_assets/common/assets/web/pin.html b/src_assets/common/assets/web/pin.html
index 4d046219..5f5b576b 100644
--- a/src_assets/common/assets/web/pin.html
+++ b/src_assets/common/assets/web/pin.html
@@ -21,13 +21,21 @@
   <div id="content" class="container content-container d-flex flex-column align-items-center">
     <ul class="nav nav-pills pin-tab-bar justify-content-center">
       <li class="nav-item">
-        <a class="nav-link" :class="{active: currentTab === 'OTP'}" href="#" @click.prevent="switchTab('OTP')">{{ $t('pin.otp_pairing') }}</a>
+        <a class="nav-link" :class="{active: currentTab !== '#PIN'}" href="#OTP" @click.prevent="switchTab('OTP')">{{ $t('pin.otp_pairing') }}</a>
       </li>
       <li class="nav-item">
-        <a class="nav-link" :class="{active: currentTab === 'PIN'}" href="#" @click.prevent="switchTab('PIN')">{{ $t('pin.pin_pairing') }}</a>
+        <a class="nav-link" :class="{active: currentTab === '#PIN'}" href="#PIN" @click.prevent="switchTab('PIN')">{{ $t('pin.pin_pairing') }}</a>
       </li>
     </ul>
-    <form v-if="currentTab === 'OTP'" class="form d-flex flex-column align-items-center" @submit.prevent="requestOTP">
+    <form v-if="currentTab === '#PIN'" class="form d-flex flex-column align-items-center" id="form" @submit.prevent="registerDevice">
+      <div class="card flex-column d-flex p-4 mb-4">
+        <input type="text" pattern="\d*" :placeholder="`${$t('navbar.pin')}`" autofocus id="pin-input" class="form-control mt-2" required />
+        <input type="text" :placeholder="`${$t('pin.device_name')}`" id="name-input" class="form-control my-4" />
+        <button class="btn btn-primary">{{ $t('pin.send') }}</button>
+      </div>
+      <div id="status"></div>
+    </form>
+    <form v-else class="form d-flex flex-column align-items-center" @submit.prevent="requestOTP">
       <div class="card flex-column d-flex p-4 mb-4">
         <div v-show="editingHost || (otp && hostAddr)" id="qrRef"></div>
         <p v-if="editingHost || (otp && hostAddr)" class="text-center text-secondary"><a class="text-secondary" :href="deepLink">art://{{ hostAddr }}:{{ hostPort }}</a> <i class="fas fa-fw fa-pen-to-square" @click="editHost"></i></p>
@@ -46,14 +54,6 @@
       <div v-if="otpMessage" class="alert" :class="['alert-' + otpStatus]">{{ otpMessage }}</div>
       <div class="alert alert-info">{{ $t('pin.otp_msg') }}</div>
     </form>
-    <form v-if="currentTab === 'PIN'" class="form d-flex flex-column align-items-center" id="form" @submit.prevent="registerDevice">
-      <div class="card flex-column d-flex p-4 mb-4">
-        <input type="text" pattern="\d*" :placeholder="`${$t('navbar.pin')}`" autofocus id="pin-input" class="form-control mt-2" required />
-        <input type="text" :placeholder="`${$t('pin.device_name')}`" id="name-input" class="form-control my-4" />
-        <button class="btn btn-primary">{{ $t('pin.send') }}</button>
-      </div>
-      <div id="status"></div>
-    </form>
     <div class="alert alert-warning">
       <b>{{ $t('_common.warning') }}</b> {{ $t('pin.warning_msg') }}
     </div>
@@ -94,7 +94,7 @@
   const data = () => {
     return {
       editingHost: false,
-      currentTab: 'OTP',
+      currentTab: location.hash || '#OTP',
       otp: '',
       passphrase: '',
       otpMessage: '',
@@ -122,7 +122,8 @@
     },
     methods: {
       switchTab(currentTab) {
-        Object.assign(this, data(), {currentTab});
+        location.hash = currentTab;
+        Object.assign(this, data());
         hostInfoCache = null;
         clearTimeout(resetOTPTimeout);
       },