Skip to content

fix: keep notes open on swipe and remove the modal rim - #379

Merged
fjbarrett merged 1 commit into
mainfrom
fix/ios-note-dismissal
Sep 5, 2026
Merged

fjbarrett merged 1 commit into
mainfrom
fix/ios-note-dismissal

Conversation

@fjbarrett

@fjbarrett fjbarrett commented Sep 5, 2026 •

Copy link
Copy Markdown
Owner

Summary

Existing and new notes now stay open during swipe gestures and close using the existing × button. The editor surface fills its presentation to remove the inset rim; adjusted internal padding preserves text and control positions.

Test plan

  • Regenerated projects; KeepMac, signed iPhone, and SE simulator builds passed.
  • Verified fast and slow downward gestures and a left-edge swipe keep an existing note open, a downward gesture keeps a new note open, and × returns to the grid.
  • Inspected the rim-free surface in light and dark mode; title, body, close, and color positions are preserved. Native card zoom remains active with no nil-source fallback warning.
  • Existing native and web CI checks passed. No new tests for this presentation change.
  • Physical SE build 20260905.379 installed and launched. App inventory confirms that version, and live PID 57797 matches its installed container. The retained Xcode checkout matches the merged source.

Model: GPT-6
Co-Authored-By: Codex noreply@openai.com

Require the existing close button to dismiss notes. Let the editor surface fill its presentation so the inset rim disappears, compensating internal padding to preserve text and control positions.

Validation: KeepMac, signed iPhone, and SE simulator builds passed. Fast and slow downward gestures keep notes open, including new notes; close works. Light and dark surfaces verified.

Model: GPT-6
Co-Authored-By: Codex <noreply@openai.com>
@fjbarrett
fjbarrett merged commit cc96538 into main Sep 5, 2026
3 checks passed
@fjbarrett
fjbarrett deleted the fix/ios-note-dismissal branch September 5, 2026 21:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant