Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Gemmakbarlow/xcode 16 library evolution #1

Open
wants to merge 21 commits into
base: main
Choose a base branch
from

Conversation

gemmakbarlow
Copy link
Owner

Notes to self:

  • Use for Sterling
  • Once some of the dependency repos have been updated, open a PR on the pointfree repository

@gemmakbarlow gemmakbarlow force-pushed the gemmakbarlow/xcode-16-library-evolution branch 2 times, most recently from 5bb8734 to 8766c77 Compare September 29, 2024 02:28
@gemmakbarlow gemmakbarlow force-pushed the gemmakbarlow/xcode-16-library-evolution branch from 0398acd to 6655437 Compare November 1, 2024 13:58
@gemmakbarlow gemmakbarlow force-pushed the gemmakbarlow/xcode-16-library-evolution branch 5 times, most recently from 82a4dc1 to da539ff Compare January 15, 2025 14:17
@gemmakbarlow gemmakbarlow force-pushed the gemmakbarlow/xcode-16-library-evolution branch from 0fe3ec3 to 1c434e3 Compare January 29, 2025 20:00
stephencelis and others added 17 commits February 19, 2025 16:58
…eeco#274)

* Promote `Void?` bindings to `Bool` via dynamic member lookup

We have an explicit initializer for this conversion, but it's not super
discoverable and it's generalized to all optionals. We can make a
shorthand for `Void` payloads, which simplifies presentation:

```diff
-.sheet(isPresented: Binding($destination.sheet)) {
+.sheet(isPresented: $destination.sheet) {
   // ...
 }
```

Other changes:

- `SwiftUI.Binding<Bool>(Binding<Optional>)`'s implementation strayed
  slightly from UIKit's `UIBinding` implementation, where issues were
  not reported. This is fixed.

- Added missing documentation.

* Add test for writing true to a UIBinding derived from optional.

---------

Co-authored-by: Brandon Williams <mbrandonw@hey.com>
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.

3 participants