Skip to content

Add privacy passport cards to Data and About - #195

Merged
kyleve merged 9 commits into
mainfrom
codex/add-privacy-passport-cards
Aug 6, 2026
Merged

Add privacy passport cards to Data and About#195
kyleve merged 9 commits into
mainfrom
codex/add-privacy-passport-cards

Conversation

@kyleve

@kyleve kyleve commented Aug 5, 2026

Copy link
Copy Markdown
Owner

Posted by an AI agent on kve's behalf.

Summary

  • add an inset “Private by design” passport card at the top of Data and About
  • state the privacy model directly: location history stays on the user’s devices and in their private iCloud account, and Where sends no data to anyone
  • render the privacy card as a passport-navy, gold-accented reflective surface using the location cards’ snapshot- and Reduce Motion-aware tilt treatment
  • keep the security-print rosette and Liquid Glass treatment independent from motion availability, so the complete surface remains visible before or without a tilt sample
  • share the passport-card structure with the GitHub link while retaining standard Form insets and the native two-shadow treatment from main; the privacy variant uses a quieter dark-navy glow
  • cover the complete Data and About pages with intrinsic-size iPhone and iPad snapshots across appearance, contrast, and accessibility variants

Testing

  • ./test WhereUITests (354 tests passed before the visual follow-ups)
  • focused PassportCardSurfaceKindTests, PassportCardSurfaceTests, PassportCardTests, PrivacyPassportCardTests, and passportCardStyle tests
  • focused AboutSettingsViewSnapshotTests and DataSettingsViewSnapshotTests
  • ./swiftformat --lint
  • ./xcstrings --lint
  • swift run bumper lint . --timings

Comment thread Where/WhereUI/Sources/Shared/PassportCardSurface.swift
secondaryOpacity: style.rosette.secondaryOpacity,
)
}
.glassEffect(

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure why this would differ either, lets remove

@kyleve
kyleve enabled auto-merge (squash) August 6, 2026 00:24
@kyleve
kyleve merged commit c3df786 into main Aug 6, 2026
9 of 10 checks passed
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