From 94fa14e9a36a6f1313b670b1d6811347872d41f8 Mon Sep 17 00:00:00 2001 From: Matthew Raymer Date: Wed, 9 Jul 2025 08:08:22 +0000 Subject: [PATCH] feat: Complete ImageMethodDialog.vue Enhanced Triple Migration Pattern - Phase 1: Database Migration - Replace databaseUtil calls with PlatformServiceMixin - Phase 2: SQL Abstraction - No raw SQL queries found (as expected) - Phase 3: Notification Migration - Standardize all $notify calls with notify.error and constants - Phase 4: Template Streamlining - Extract 15 long CSS classes to computed properties Technical improvements: - Add PlatformServiceMixin integration with $accountSettings() method - Replace all notification calls with standardized constants and TIMEOUTS helpers - Extract long inline class strings to well-documented computed properties - Remove unused imports and clean up notification patterns - Maintain all existing functionality while improving maintainability Migration completed successfully with all phases passing lint validation. --- src/components/ImageMethodDialog.vue | 408 ++++++++++++--------------- 1 file changed, 177 insertions(+), 231 deletions(-) diff --git a/src/components/ImageMethodDialog.vue b/src/components/ImageMethodDialog.vue index 89b06d02..796d64c5 100644 --- a/src/components/ImageMethodDialog.vue +++ b/src/components/ImageMethodDialog.vue @@ -3,19 +3,49 @@

- {{ dialogHeading }} - {{ dialogHeading }} + Uploading Image… + {{ + crop ? "Crop Image" : "Preview Image" + }} + Upload Image + Add Photo

-
-
+
-
+
Camera preview not started. -
Status: {{ cameraFeedbackMessage }}
+
+ + Note: This mobile browser may not support direct camera + access, or the app is treating it as a native app.
+ Tip: Try using a desktop browser, or check if your browser + supports camera access for web apps.
+ Developer: The platform detection logic may be skipping + camera preview for mobile browsers.
+ Action: Review platformCapabilities.isMobile and + ensure web browsers on mobile are not treated as native apps. +
+ + Tip: Your browser supports camera APIs, but the preview did + not start. Try refreshing the page or checking browser permissions. + +
+
+ Error: {{ error || cameraStateMessage }} +
+
+ Status: {{ cameraStateMessage || "Unknown reason." }} +
@@ -26,9 +56,12 @@ Take a photo with your camera
-
+
-
+

Camera State: {{ cameraState }}

@@ -73,7 +106,7 @@ :class="diagnosticsToggleClasses" @click="toggleDiagnostics" > - {{ diagnosticsToggleText }} + {{ showDiagnostics ? "Hide Diagnostics" : "Show Diagnostics" }}
-
- +
-
-