robert/PedalPebble
Watch
1
0
Fork
You've already forked PedalPebble
 0
Code Issues 43 Pull requests Projects Releases 1 Packages Wiki Activity Actions

Page carousel navigation with long-press jumps #60

New issue
Closed
opened 2026-08-31 17:55:58 +02:00 by robert · 1 comment
robert commented 2026-08-31 17:55:58 +02:00 (Migrated from git.butzei.de)
Copy link

Goal

A single flat ring of pages, wrapping at both ends, with two shortcuts so nothing is more than two presses away.

Acceptance criteria

  • Up and Down move to the previous and next enabled page, wrapping at both ends
  • Long Up jumps to page 1; long Down jumps to the map
  • Select starts and pauses; long Select stops with confirmation; Back exits with confirmation while riding
  • Navigation page inserts itself into the ring only when a route is loaded
  • Disabled and auto-hidden pages are skipped without a visible flicker
  • Current page persisted across app restarts
  • Long-press does not also fire the short-press action
  • Buttons stay responsive while map frames are arriving

Files

  • watchapp/src/c/main.c
  • watchapp/src/c/carousel.c

Notes

See docs/DESIGN.md section 5. Chosen over a Back-toggles-groups model because Back not going back is surprising on Pebble and risks accidental exits.

## Goal A single flat ring of pages, wrapping at both ends, with two shortcuts so nothing is more than two presses away. ## Acceptance criteria - [ ] Up and Down move to the previous and next enabled page, wrapping at both ends - [ ] **Long Up jumps to page 1**; **long Down jumps to the map** - [ ] Select starts and pauses; long Select stops with confirmation; Back exits with confirmation while riding - [ ] Navigation page inserts itself into the ring only when a route is loaded - [ ] Disabled and auto-hidden pages are skipped without a visible flicker - [ ] Current page persisted across app restarts - [ ] Long-press does not also fire the short-press action - [ ] Buttons stay responsive while map frames are arriving ## Files - `watchapp/src/c/main.c` - `watchapp/src/c/carousel.c` ## Notes See docs/DESIGN.md section 5. Chosen over a Back-toggles-groups model because Back not going back is surprising on Pebble and risks accidental exits.
robert commented 2026-09-04 09:48:16 +02:00
Owner
Copy link

Closed by PR #85 (merged): carousel.c (renamed in place from #8's page_view.c, git history preserved) implements the real DESIGN.md section 5 button model — Up/Down step to the next visible page with a cold-boot all-hidden fallback, long Up jumps to page 1 (bypassing the visibility skip on purpose, an explicit request always wins), long Down is a documented no-op (no map page exists yet), and Select/long-Select/Back are wired to #11's real ride_state_/ride_link_ functions with a shared confirmation-prompt model for stop-vs-exit. Active page persisted via a second persist key. Verified live in emery+gabbro emulators: heap flat across every transition (107512\u2192107512), 0 bytes leaked at exit, persistence survives a reinstall.\n\nHonestly scoped: nav-page insertion (no route-loaded state exists) and the "disabled" half of page-skipping (no #61 settings surface yet) are both documented as not-yet-backed-by-real-data rather than faked. Page rendering is still #8's one-TextLayer placeholder — the template renderer (HERO1/HERO2/QUAD/GRID6) is #59's job.

Closed by PR #85 (merged): carousel.c (renamed in place from #8's page_view.c, git history preserved) implements the real DESIGN.md section 5 button model — Up/Down step to the next visible page with a cold-boot all-hidden fallback, long Up jumps to page 1 (bypassing the visibility skip on purpose, an explicit request always wins), long Down is a documented no-op (no map page exists yet), and Select/long-Select/Back are wired to #11's real ride_state_*/ride_link_* functions with a shared confirmation-prompt model for stop-vs-exit. Active page persisted via a second persist key. Verified live in emery+gabbro emulators: heap flat across every transition (107512\u2192107512), 0 bytes leaked at exit, persistence survives a reinstall.\n\nHonestly scoped: nav-page insertion (no route-loaded state exists) and the "disabled" half of page-skipping (no #61 settings surface yet) are both documented as not-yet-backed-by-real-data rather than faked. Page *rendering* is still #8's one-TextLayer placeholder — the template renderer (HERO1/HERO2/QUAD/GRID6) is #59's job.
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
area/gpx-cue-enricher-d59
area/round-display-support
area/rejoin-point-selection
area/digit-height-content-aware
area/geometry-enricher
area/page-template-hero1-grid6
area/nav-engine
area/cue-sheet-review
area/map-viewport-geometry
area/navigation-snapping-test-audit
area/lap-function
area/navigation-snapping
area/ble-power-meter
area/gpx-export
area/gpx-recording
area/pkjs-speed-feed
tooling/watchapp-distribution
area/csc-cadence
area/about-screen
area/backlight-policy
tooling/gpx-replay
area/derived-metrics
area/speed-pipeline-coverage-audit
area/speed-source-arbitration
area/speed-pipeline
area/stop-detector
area/ble-csc-sensor
area/persist-ride-state
area/foreground-location-service
area/protocol-version-handshake
area/real-komoot-fixture
tooling/message-keys-codegen
tooling/fix-android-platform-doc
area/settings-screen
tooling/first-real-android-build
tooling/fix-runner-label
area/cue-sheet-cache
area/route-library
area/page-template-renderer
area/colour-palette
area/page-carousel
area/ride-state-machine
area/watchapp-skeleton
tooling/host-test-harness
area/field-store-page-descriptors
tooling/ci-pipeline
area/cue-sheet-enricher-interface
area/gpx-share-import
process/security-persona
area/android-companion-skeleton
tooling/pebble-sdk-emulators
process/team-and-pipeline
dev-latest
v0.5.0
Labels
Clear labels   
area:companion
Android companion app

  
area:docs
Documentation

  
area:shared
Message contract shared by both sides

  
area:tooling
Build tooling, SDK, dev scripts

  
area:watchapp
Pebble C watchapp

  
blocker
Blocks a large part of the project until resolved

  
kind:chore
Setup, scaffolding, maintenance

  
kind:feature
New functionality

  
kind:spike
Time-boxed investigation to de-risk a decision

  
kind:test
Tests and fixtures

No labels
area:companion
area:docs
area:shared
area:tooling
area:watchapp
blocker
kind:chore
kind:feature
kind:spike
kind:test
Milestone
Clear milestone
No items
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Blocks
#37 Auto-switch between ride and nav views near a turn
robert/PedalPebble
Depends on
#8 Watchapp skeleton: window stack, view switching, button handling
robert/PedalPebble
#58 Field value store and page descriptor model
robert/PedalPebble
Reference
robert/PedalPebble#60
No description provided.