|
| 1 | +# Open issue triage - 2026-07-06 |
| 2 | + |
| 3 | +Fork `main` fast-forwarded to upstream tip `2f9af13` before this triage (157 commits pulled in, |
| 4 | +fork was previously 83 commits behind upstream). All 12 open issues on |
| 5 | +https://github.com/User-green/Aster-Android/issues checked against the updated `main`. |
| 6 | + |
| 7 | +| # | Title | Verdict | |
| 8 | +|---|---|---| |
| 9 | +| 26 | Text formatting and padding issues in mail body | Partially fixed | |
| 10 | +| 25 | Sidebar categories re-expand after relaunch | Fixed | |
| 11 | +| 24 | Verification email not received for Aster-to-Aster auto-forward | Not fixed | |
| 12 | +| 23 | Dark mode delay, inconsistency, content visibility | Unclear / not fixed | |
| 13 | +| 22 | Dead CSRF endpoint client still calls | Not fixed | |
| 14 | +| 21 | Crash loading 500-600 aliases | Fixed | |
| 15 | +| 19 | Avatar not displayed | Fixed (already merged) | |
| 16 | +| 18 | Drawer scroll lag | Not fixed | |
| 17 | +| 17 | Inconsistent alias-add dialog | Fixed | |
| 18 | +| 16 | Custom domain aliases not visible | Fixed (visibility); toggle still unmerged | |
| 19 | +| 13 | Inbox stuck on archive/all-mail | Not fixed / unclear | |
| 20 | +| 3 | Upgrade section shows USD not local currency | Not fixed | |
| 21 | + |
| 22 | +## Details |
| 23 | + |
| 24 | +### #26 - Text formatting and padding issues |
| 25 | +Partially fixed. `543b73c` made the reveal-container padding unconditional |
| 26 | +(`padding(horizontal = 16.dp)` for all mail, previously 12dp only for newsletter-style |
| 27 | +mail). Plain HTML mail body CSS still only carries `padding:6px 10px`, which is thin - |
| 28 | +worth a visual check before closing. |
| 29 | + |
| 30 | +### #25 - Sidebar categories re-expand after relaunch |
| 31 | +Fixed. `MainActivity.kt` now reads `sidebar_more_collapsed` / `folders_collapsed` / |
| 32 | +`labels_collapsed` / `aliases_collapsed` from preferences and restores them in |
| 33 | +`DrawerContent`; `SettingsViewModel` persists them via `on_sidebar_toggle`. |
| 34 | + |
| 35 | +### #24 - Aster-to-Aster auto-forward verification email missing |
| 36 | +Not fixed. Only related commit (`b76621d`) is a request/response DTO alignment fix, not |
| 37 | +verification-email dispatch. This is very likely a backend-only bug (Aster-Backend), not |
| 38 | +something fixable in this repo. |
| 39 | + |
| 40 | +### #23 - Dark mode delay/inconsistency/invisible content |
| 41 | +Unclear. No commit specifically targets a delayed or inconsistent dark-mode application; |
| 42 | +existing dark-mode commits predate this issue. Needs manual repro to confirm current state. |
| 43 | + |
| 44 | +### #22 - Dead CSRF endpoint |
| 45 | +Not fixed. `ApiClient.kt` still declares `csrf_endpoint = "/api/csrf/token"` and |
| 46 | +`fetch_csrf_if_needed()`, still called from `AccountApi`, `CsrfApi`, `FamilyApi`, |
| 47 | +`MailRulesApi`. Unchanged from when the issue was filed. |
| 48 | + |
| 49 | +### #21 - Crash loading 500-600 aliases |
| 50 | +Fixed. `9ef79af` ("perf(aliases): paginate alias list and decrypt off main thread") replaced |
| 51 | +eager `forEachIndexed` rendering with paged/lazy rendering and moved decrypt to |
| 52 | +`Dispatchers.Default`. `AliasesLargeListTest.kt` is an explicit regression guard citing this |
| 53 | +exact ANR crash. |
| 54 | + |
| 55 | +### #19 - Avatar not displayed |
| 56 | +Fixed, already merged. `fix/account-avatar-display` (tip `9af7331`, "render account avatars |
| 57 | +in drawer header and switcher") is an ancestor of `main` - nothing left unmerged on that |
| 58 | +branch. |
| 59 | + |
| 60 | +### #18 - Drawer scroll lag |
| 61 | +Not fixed. `drawer_content.kt` still uses a plain `Column(...).verticalScroll(...)` for the |
| 62 | +whole drawer body rather than a `LazyColumn`. No perf-specific commit found for this. |
| 63 | + |
| 64 | +### #17 - Inconsistent alias-add dialog |
| 65 | +Fixed. `f02cc17` ("route drawer alias add to settings create dialog") removed the drawer's |
| 66 | +separate dialog and routes to the same Settings create-dialog via a `create=true` nav arg. |
| 67 | + |
| 68 | +### #16 - Custom domain aliases not visible |
| 69 | +Visibility fixed, one gap remains. `f77a0a5` ("show custom domain aliases in drawer and |
| 70 | +settings") loads custom-domain addresses into the drawer/settings alias list - the reported |
| 71 | +visibility complaint is resolved. Branch `fix/show-custom-domain-aliases` is fully merged. |
| 72 | +Branch `fix/toggle-custom-domain-alias` (`354f51d`, enable/disable toggle for custom-domain |
| 73 | +aliases) is **not** merged into main - separate unmerged work still sitting in a branch. |
| 74 | + |
| 75 | +### #13 - Inbox stuck on archive/all-mail |
| 76 | +Not fixed / unclear. Cited related commits (`a44ec73`, `0962ed7`, `c95b776`) are tangential - |
| 77 | +next/prev swipe ordering, scroll-position restore, and settings-save conflicts respectively. |
| 78 | +No commit touches folder-switch/list-republish logic in `MailViewModel`/`MailRepository`. |
| 79 | +Needs manual repro. |
| 80 | + |
| 81 | +### #3 - Upgrade section shows USD |
| 82 | +Not fixed. `subscriptions_screen.kt` hardcodes `val detected_currency = "usd"` with no locale |
| 83 | +detection; `start_checkout` doesn't pass currency at all. Issue is fully unaddressed. |
| 84 | + |
| 85 | +## Branch merge status |
| 86 | +- `fix/account-avatar-display` - merged (ancestor of `main`) |
| 87 | +- `fix/show-custom-domain-aliases` - merged (`main..branch` empty) |
| 88 | +- `fix/toggle-custom-domain-alias` - unmerged, contains a distinct unlanded feature |
0 commit comments