From d35410c707900075bf4207bab518ead44d43c6e7 Mon Sep 17 00:00:00 2001 From: Claude Date: Tue, 26 May 2026 04:57:28 +0000 Subject: Fix camera to open system camera app; harden logbook storage; add backlog Camera: - Replace ActivityResultContracts.TakePicture() with explicit StartActivityForResult + Intent(ACTION_IMAGE_CAPTURE) + EXTRA_OUTPUT so the full system camera app always opens (not a sub-view preview) - FLAG_GRANT_WRITE_URI_PERMISSION added for camera app FileProvider access LogbookStorage: - getExternalFilesDir() can return null when external storage is unavailable; fall back to filesDir for both storageFile and photoDir so saves never silently fail on devices without external storage Backlog (worklog.md): - Add notes/photos to finished tracks + include in trip report - Trip report voice: replace generic style picker with single Nav voice - Fix tack detection false positives at low SOG - Investigate/fix speed coloring in MapLibre Android 11.x https://claude.ai/code/session_01YUbuZNDAoLea4cf9UGQ9qn --- .agent/worklog.md | 10 ++++++++++ 1 file changed, 10 insertions(+) (limited to '.agent/worklog.md') diff --git a/.agent/worklog.md b/.agent/worklog.md index 55fd5a7..cfae053 100644 --- a/.agent/worklog.md +++ b/.agent/worklog.md @@ -1,5 +1,15 @@ # SESSION_STATE.md +## Backlog (fix later) + +- **Add notes/photos to a finished track** — UI to attach logbook entries to a past track by timestamp/geo proximity. Must flow into the trip report (currently only live-session entries are scoped). + +- **Trip report voice** — The current narrative styles (Professional / Adventurous / Journal / Pirate) are generic and not right for Nav. Nav is an opinionated sailing app; the report should have a single voice that reflects that — concise, knowledgeable, seamanlike. Rip out the style picker and write one good template. + +- **Tack detection is buggy, especially at low speed** — False positives at low SOG (GPS heading noise looks like tacks). Need minimum-SOG gate (e.g. skip detection when SOG < 1.5 kt) and review the stability window / MIN_DELTA thresholds against real tracks. + +- **Speed coloring on the map doesn't work** — `Expression.get("color")` on a FeatureCollection of 2-point LineStrings fails silently in MapLibre Android 11.x. The focused-track layer in MapHandler still uses this broken approach. Needs investigation: either find a working data-driven expression syntax for this MapLibre version, or pre-segment the GeoJSON differently. + ## Current Task Goal Instrument Sheet Visual Redesign — IN PROGRESS (2026-04-06) -- cgit v1.2.3