From b1a1e35790ae87d04f4ad86a8640c30fc721a9c2 Mon Sep 17 00:00:00 2001
From: Huang Xin <chrox.huang@gmail.com>
Date: Thu, 5 Feb 2026 17:08:17 +0800
Subject: [PATCH] fix(layout): move sync options inside of account section in
 settings menu (#3176)

---
 .../app/library/components/SettingsMenu.tsx   | 77 +++++++++----------
 apps/readest-app/src/services/appService.ts   |  6 +-
 2 files changed, 43 insertions(+), 40 deletions(-)

diff --git a/apps/readest-app/src/app/library/components/SettingsMenu.tsx b/apps/readest-app/src/app/library/components/SettingsMenu.tsx
index c6bdeb35..601099ea 100644
--- a/apps/readest-app/src/app/library/components/SettingsMenu.tsx
+++ b/apps/readest-app/src/app/library/components/SettingsMenu.tsx
@@ -257,57 +257,56 @@ const SettingsMenu: React.FC<SettingsMenuProps> = ({ onPullLibrary, setIsDropdow
             )
           }
         >
-          <ul
-            className='ms-0 flex flex-col before:hidden'
-            style={{
-              paddingInlineStart: `${iconSize}px`,
-            }}
-          >
-            <button onClick={handleUserProfile} className='w-full'>
+          <ul className='ms-0 flex flex-col ps-0 before:hidden'>
+            <MenuItem
+              label={_('Cloud File Transfers')}
+              Icon={MdCloudSync}
+              description={
+                hasActiveTransfers
+                  ? _('{{activeCount}} active, {{pendingCount}} pending', {
+                      activeCount: stats.active,
+                      pendingCount: stats.pending,
+                    })
+                  : stats.failed > 0
+                    ? _('{{failedCount}} failed', { failedCount: stats.failed })
+                    : ''
+              }
+              onClick={openTransferQueue}
+            />
+            <MenuItem
+              label={
+                settings.lastSyncedAtBooks
+                  ? _('Synced at {{time}}', {
+                      time: new Date(settings.lastSyncedAtBooks).toLocaleString(),
+                    })
+                  : _('Never synced')
+              }
+              Icon={user ? MdSync : MdSyncProblem}
+              iconClassName={user && isSyncing ? 'animate-reverse-spin' : ''}
+              onClick={onPullLibrary.bind(null, true, true)}
+            />
+            <button
+              onClick={handleUserProfile}
+              className='hover:bg-base-300 w-full rounded-md'
+              style={{
+                paddingInlineStart: `${iconSize}px`,
+              }}
+            >
               <Quota quotas={quotas} labelClassName='h-10 pl-3 pr-2' />
             </button>
-            <MenuItem label={_('Account')} noIcon onClick={handleUserProfile} />
+            <MenuItem label={_('Account')} onClick={handleUserProfile} />
           </ul>
         </MenuItem>
       ) : (
         <MenuItem label={_('Sign In')} Icon={PiUserCircle} onClick={handleUserLogin}></MenuItem>
       )}
-      {user && (
-        <MenuItem
-          label={_('Cloud File Transfers')}
-          Icon={MdCloudSync}
-          description={
-            hasActiveTransfers
-              ? _('{{activeCount}} active, {{pendingCount}} pending', {
-                  activeCount: stats.active,
-                  pendingCount: stats.pending,
-                })
-              : stats.failed > 0
-                ? _('{{failedCount}} failed', { failedCount: stats.failed })
-                : ''
-          }
-          onClick={openTransferQueue}
-        />
-      )}
+
       <MenuItem
         label={_('Auto Upload Books to Cloud')}
         toggled={isAutoUpload}
         onClick={toggleAutoUploadBooks}
       />
-      <MenuItem
-        label={
-          !user
-            ? _('Sign in to Sync')
-            : settings.lastSyncedAtBooks
-              ? _('Synced at {{time}}', {
-                  time: new Date(settings.lastSyncedAtBooks).toLocaleString(),
-                })
-              : _('Never synced')
-        }
-        Icon={user ? MdSync : MdSyncProblem}
-        iconClassName={user && isSyncing ? 'animate-reverse-spin' : ''}
-        onClick={onPullLibrary.bind(null, true, true)}
-      />
+
       {isTauriAppPlatform() && !appService?.isMobile && (
         <MenuItem
           label={_('Auto Import on File Open')}
diff --git a/apps/readest-app/src/services/appService.ts b/apps/readest-app/src/services/appService.ts
index 110312f6..28b93282 100644
--- a/apps/readest-app/src/services/appService.ts
+++ b/apps/readest-app/src/services/appService.ts
@@ -695,8 +695,12 @@ export abstract class BaseAppService implements AppService {
     const { file } = await this.loadBookContent(book);
     const content = await file.arrayBuffer();
     const filename = `${makeSafeFilename(book.title)}.${book.format.toLowerCase()}`;
-    const filepath = await this.resolveFilePath(getLocalBookFilename(book), 'Books');
+    let filepath = await this.resolveFilePath(getLocalBookFilename(book), 'Books');
     const fileType = file.type || 'application/octet-stream';
+    if (getFilename(filepath) !== filename) {
+      this.copyFile(filepath, filename, 'Temp');
+      filepath = await this.resolveFilePath(filename, 'Temp');
+    }
     return await this.saveFile(filename, content, filepath, fileType);
   }