From 6bb0189078d13567edb2700b0a1f63758f9e2ea9 Mon Sep 17 00:00:00 2001
From: 10G Meow <10gmeow@gmail.com>
Date: Sun, 21 Dec 2025 22:23:09 +0200
Subject: [PATCH] Confirmation dialog ux: Logout, set password and change name

---
 src/assets/translations/en.json     |  4 +-
 src/components/LogoutRoomDialog.vue |  9 ++-
 src/components/Profile.vue          | 91 +++++++++++++++--------------
 3 files changed, 57 insertions(+), 47 deletions(-)

diff --git a/src/assets/translations/en.json b/src/assets/translations/en.json
index aecbbdd..2b3115d 100644
--- a/src/assets/translations/en.json
+++ b/src/assets/translations/en.json
@@ -253,6 +253,7 @@
     "title": "My Profile",
     "temporary_identity": "This identity is temporary. Set a password to use it again",
     "set_password": "Set password",
+    "set_password_desc": "Convene does not store passwords and cannot recover them. We recommend storing your password in a secure location before clicking save.",
     "change_name": "Change name",
     "change_password": "Change password",
     "my_rooms": "My rooms",
@@ -323,7 +324,8 @@
     "copy_credentials": "Copy Username & Password",
     "copied_credentials": "Copied Username & Password",
     "copied_credentials_value": "Username: {userId} \nPassword: {password}",
-    "copy_credentials_desc": "Your username and password are required to regain access to your chats from a new device or browser. We recommend storing these credentials in a secure location."
+    "copy_credentials_desc": "Your username and password are required to regain access to your chats from a new device or browser. We recommend storing these credentials in a secure location.",
+    "copy_credentials_desc_for_real_user": "Your username (ex: {'@user:server'}) and your password are required to regain access to your chats from a new device or browser."
   },
   "delete_post": {
     "confirm_text": "Are you sure you want to delete this message?",
diff --git a/src/components/LogoutRoomDialog.vue b/src/components/LogoutRoomDialog.vue
index 8a9b6ff..3a24676 100644
--- a/src/components/LogoutRoomDialog.vue
+++ b/src/components/LogoutRoomDialog.vue
@@ -4,8 +4,8 @@
     @click:outside="$emit('onOutsideLogoutPopupClicked')">
     <div class="dialog-content text-center">
       <h2 class="dialog-title">{{ $t("logout.confirm_text") }}</h2>
-      <div class="dialog-text">{{ $t("logout.copy_credentials_desc") }}</div>
-      <v-row>
+      <div class="dialog-text">{{ copyCredentialsSubText }}</div>
+      <v-row v-if="this.$matrix.currentUser.is_guest">
         <v-col sm="12" md="6" offset-md="3" class="d-flex justify-center">
           <v-btn
             ref="copyCredentialsBtn"
@@ -59,7 +59,10 @@ export default {
     ]),
     credentials() {
       return this.$t(`logout.copied_credentials_value`, { userId: this.auth.user.user_id, password: this.auth.user.password})
-    }
+    },
+    copyCredentialsSubText() {
+      return this.$t(`${this.$matrix.currentUser.is_guest ? 'logout.copy_credentials_desc': 'logout.copy_credentials_desc_for_real_user'}`)
+    },
   },
   methods: {
     onCopyCredentials() {
diff --git a/src/components/Profile.vue b/src/components/Profile.vue
index 556a7e0..bce814c 100644
--- a/src/components/Profile.vue
+++ b/src/components/Profile.vue
@@ -116,6 +116,7 @@
     >
       <v-card :disabled="settingPassword" class="dialog-content">
         <h2 class="dialog-title">{{ $matrix.currentUser.is_guest ? $t("profile.set_password") : $t("profile.change_password") }}</h2>
+        <div class="dialog-text mb-4">{{ $t("profile.set_password_desc") }}</div>
         <div class="dialog-text">
           <v-text-field
             v-if="!$matrix.currentUser.is_guest"
@@ -149,28 +150,30 @@
             {{ passwordErrorMessage }}
           </div>
         </div>
-        <v-divider></v-divider>
-        <v-card-actions>
-          <v-spacer></v-spacer>
-          <v-btn id="btn-password-cancel" variant="text" @click="closeEditPasswordDialog">{{
-            $t("menu.cancel")
-          }}</v-btn>
-          <v-btn
-            id="btn-password-set"
-            :disabled="!passwordsMatch"
-            color="primary"
-            variant="text"
-            @click="
-              setPassword(
-                $matrix.currentUser.is_guest
-                  ? $matrix.currentUser.password
-                  : password,
-                newPassword1
-              )
-            "
-            >{{ $t("global.save") }}</v-btn
-          >
-        </v-card-actions>
+        <v-container fluid>
+          <v-row cols="12">
+            <v-col cols="6">
+              <v-btn variant="flat" block class="text-button" @click.stop="closeEditPasswordDialog">{{
+                $t("menu.cancel") }}</v-btn>
+            </v-col>
+            <v-col cols="6" align="center">
+              <v-btn
+                :disabled="!passwordsMatch"
+                color="#4642F1"
+                variant="flat"
+                block
+                class="filled-button"
+                @click="
+                  setPassword(
+                    $matrix.currentUser.is_guest
+                      ? $matrix.currentUser.password
+                      : password,
+                    newPassword1
+                  )
+                ">{{ $t("global.save") }}</v-btn>
+            </v-col>
+          </v-row>
+        </v-container>
       </v-card>
     </v-dialog>
 
@@ -180,33 +183,35 @@
       class="ma-0 pa-0"
       :width="$vuetify.display.smAndUp ? '940px' : '80%'"
     >
-      <v-card>
+      <v-card class="dialog-content">
         <v-card-title>{{ $t("profile.display_name") }}</v-card-title>
-        <v-card-text>
+        <div class="dialog-text">
           <v-text-field
             v-model="editValue"
             :rules="[(v) => !!v || $t('profile.display_name_required')]"
             required
           />
-        </v-card-text>
-        <v-divider></v-divider>
-        <v-card-actions>
-          <v-spacer></v-spacer>
-          <v-btn id="btn-displayname-cancel" variant="text" @click="showEditDisplaynameDialog = false">{{
-            $t("menu.cancel")
-          }}</v-btn>
-          <v-btn
-            id="btn-displayname-set"
-            color="primary"
-            variant="text"
-            @click="
-              updateDisplayName(editValue);
-              showEditDisplaynameDialog = false;
-            "
-            :disabled="!editValue"
-            >{{ $t("global.save") }}</v-btn
-          >
-        </v-card-actions>
+          <v-container fluid>
+            <v-row cols="12">
+              <v-col cols="6">
+                <v-btn variant="flat" block class="text-button" @click="showEditDisplaynameDialog = false">{{
+                  $t("menu.cancel") }}</v-btn>
+              </v-col>
+              <v-col cols="6" align="center">
+                <v-btn
+                  :disabled="!editValue"
+                  color="#4642F1"
+                  variant="flat"
+                  block
+                  class="filled-button"
+                  @click="
+                    updateDisplayName(editValue);
+                    showEditDisplaynameDialog = false;
+                  ">{{ $t("global.save") }}</v-btn>
+              </v-col>
+            </v-row>
+          </v-container>
+        </div>
       </v-card>
     </RoundedDialog>