From 7f4202afd73105b850498b55ad66922b3505f0e3 Mon Sep 17 00:00:00 2001
From: Evan Simkowitz <esimkowitz@users.noreply.github.com>
Date: Sun, 14 Dec 2025 16:39:25 -0800
Subject: feat: Add unified reader settings with local overrides (#2230)

* Add initial impl

* fix some format inconsistencies, add indicator in user settings when local is out of sync

* Fix sliders in user settings, unify constants and formatting

* address CodeRabbit suggestions

* add mobile implementation

* address coderabbit nitpicks

* fix responsiveness of the reader settings popover

* Move more of the web UI strings to i18n

* update translations for more coverage

* remove duplicate logic/definitions

* fix android font family

* add shared reading setting hook between web and mobile

* unify reader settings context for both web and mobile

* remove unused export

* address coderabbit suggestions

* fix tests
---
 packages/trpc/models/users.ts | 37 +++++++++++++++++++++++--------------
 1 file changed, 23 insertions(+), 14 deletions(-)

(limited to 'packages/trpc/models')

diff --git a/packages/trpc/models/users.ts b/packages/trpc/models/users.ts
index a1f32f02..85aed639 100644
--- a/packages/trpc/models/users.ts
+++ b/packages/trpc/models/users.ts
@@ -433,6 +433,9 @@ export class User {
         backupsEnabled: true,
         backupsFrequency: true,
         backupsRetentionDays: true,
+        readerFontSize: true,
+        readerLineHeight: true,
+        readerFontFamily: true,
       },
     });
 
@@ -450,6 +453,9 @@ export class User {
       backupsEnabled: settings.backupsEnabled,
       backupsFrequency: settings.backupsFrequency,
       backupsRetentionDays: settings.backupsRetentionDays,
+      readerFontSize: settings.readerFontSize,
+      readerLineHeight: settings.readerLineHeight,
+      readerFontFamily: settings.readerFontFamily,
     };
   }
 
@@ -472,6 +478,9 @@ export class User {
         backupsEnabled: input.backupsEnabled,
         backupsFrequency: input.backupsFrequency,
         backupsRetentionDays: input.backupsRetentionDays,
+        readerFontSize: input.readerFontSize,
+        readerLineHeight: input.readerLineHeight,
+        readerFontFamily: input.readerFontFamily,
       })
       .where(eq(users.id, this.user.id));
   }
@@ -553,23 +562,23 @@ export class User {
       // Top domains
       this.ctx.db
         .select({
-          domain: sql<string>`CASE 
-            WHEN ${bookmarkLinks.url} LIKE 'https://%' THEN 
-              CASE 
+          domain: sql<string>`CASE
+            WHEN ${bookmarkLinks.url} LIKE 'https://%' THEN
+              CASE
                 WHEN INSTR(SUBSTR(${bookmarkLinks.url}, 9), '/') > 0 THEN
                   SUBSTR(${bookmarkLinks.url}, 9, INSTR(SUBSTR(${bookmarkLinks.url}, 9), '/') - 1)
                 ELSE
                   SUBSTR(${bookmarkLinks.url}, 9)
               END
-            WHEN ${bookmarkLinks.url} LIKE 'http://%' THEN 
-              CASE 
+            WHEN ${bookmarkLinks.url} LIKE 'http://%' THEN
+              CASE
                 WHEN INSTR(SUBSTR(${bookmarkLinks.url}, 8), '/') > 0 THEN
                   SUBSTR(${bookmarkLinks.url}, 8, INSTR(SUBSTR(${bookmarkLinks.url}, 8), '/') - 1)
                 ELSE
                   SUBSTR(${bookmarkLinks.url}, 8)
               END
-            ELSE 
-              CASE 
+            ELSE
+              CASE
                 WHEN INSTR(${bookmarkLinks.url}, '/') > 0 THEN
                   SUBSTR(${bookmarkLinks.url}, 1, INSTR(${bookmarkLinks.url}, '/') - 1)
                 ELSE
@@ -582,23 +591,23 @@ export class User {
         .innerJoin(bookmarks, eq(bookmarks.id, bookmarkLinks.id))
         .where(eq(bookmarks.userId, this.user.id))
         .groupBy(
-          sql`CASE 
-          WHEN ${bookmarkLinks.url} LIKE 'https://%' THEN 
-            CASE 
+          sql`CASE
+          WHEN ${bookmarkLinks.url} LIKE 'https://%' THEN
+            CASE
               WHEN INSTR(SUBSTR(${bookmarkLinks.url}, 9), '/') > 0 THEN
                 SUBSTR(${bookmarkLinks.url}, 9, INSTR(SUBSTR(${bookmarkLinks.url}, 9), '/') - 1)
               ELSE
                 SUBSTR(${bookmarkLinks.url}, 9)
             END
-          WHEN ${bookmarkLinks.url} LIKE 'http://%' THEN 
-            CASE 
+          WHEN ${bookmarkLinks.url} LIKE 'http://%' THEN
+            CASE
               WHEN INSTR(SUBSTR(${bookmarkLinks.url}, 8), '/') > 0 THEN
                 SUBSTR(${bookmarkLinks.url}, 8, INSTR(SUBSTR(${bookmarkLinks.url}, 8), '/') - 1)
               ELSE
                 SUBSTR(${bookmarkLinks.url}, 8)
             END
-          ELSE 
-            CASE 
+          ELSE
+            CASE
               WHEN INSTR(${bookmarkLinks.url}, '/') > 0 THEN
                 SUBSTR(${bookmarkLinks.url}, 1, INSTR(${bookmarkLinks.url}, '/') - 1)
               ELSE
-- 
cgit v1.2.3-70-g09d2