Skip to content

Added read-only dynamic member lookup support to ListProperties#484

Merged
robmaceachern merged 1 commit intomainfrom
robmaceachern/read-only-listproperties-lookup
May 12, 2023
Merged

Added read-only dynamic member lookup support to ListProperties#484
robmaceachern merged 1 commit intomainfrom
robmaceachern/read-only-listproperties-lookup

Conversation

@robmaceachern
Copy link
Member

  • Added read-only dynamic member lookup support to ListProperties.
  • This fixes a confusing compilation error when omitting the .content property on read-only KeyPath lookups on ListProperties:
// Cannot assign to property: 'lastItem' is a get-only property
let item = properties.lastItem

Checklist

Please do the following before merging:

  • Ensure any public-facing changes are reflected in the changelog. Include them in the Main section.

@robmaceachern robmaceachern requested a review from a team May 12, 2023 18:20
@robmaceachern robmaceachern merged commit 3bf4b51 into main May 12, 2023
@robmaceachern robmaceachern deleted the robmaceachern/read-only-listproperties-lookup branch May 12, 2023 19:13
kyleve added a commit that referenced this pull request Jul 10, 2023
* origin/main: (88 commits)
  Prep 11.0 release
  Paged list layouts now support using bounds, instead of itemInsets.
  Prep 10.3.1 release
  Update CHANGELOG
  iOS 17.0b1 first responder changes
  Update docs
  Prep 10.3.0
  Allow programmatically showing leading and trailing swipe actions
  Update docs
  Prepare 10.2.0
  PR feedback updates
  Separated out OverlayDecorationView and UnderlayDecorationView
  Updated Changelog
  Added an underlay view to item cells
  Added read-only dynamic member lookup support to ListProperties (#484)
  Release 10.1.0 (#482)
  Bump Blueprint dependency to 2.0
  Replace xcversion with xcodes
  Fix issue where first tap on a header with no pressed background would eat the tap
  Re-generate the documentation
  ...
@n8chur n8chur mentioned this pull request Aug 8, 2023
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.

2 participants