Skip to content
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.

Rename $featureProperties to $featureAttributes #11739

Closed
1ec5 opened this issue Apr 19, 2018 · 1 comment
Closed

Rename $featureProperties to $featureAttributes #11739

1ec5 opened this issue Apr 19, 2018 · 1 comment
Labels
iOS Mapbox Maps SDK for iOS macOS Mapbox Maps SDK for macOS refactor runtime styling
Milestone

Comments

@1ec5
Copy link
Contributor

1ec5 commented Apr 19, 2018

The $featureProperties expression variable should be renamed $featureAttributes to match the MGLFeature.attributes property. The iOS/macOS map SDK avoids using the term “property” to refer to things that aren’t represented by class or instance properties in Objective-C or Swift.

If this change doesn’t take place before the iOS map SDK v4.0.0 release, we can support both variable names but prefer $featureAttributes. ($featureProperties isn’t documented, in any case.) The NSExpression.featurePropertiesVariableExpression property would ideally be renamed as well; we’d have to leave a deprecated alias in place for it.

/cc @fabian-guerra

@1ec5 1ec5 added iOS Mapbox Maps SDK for iOS refactor macOS Mapbox Maps SDK for macOS runtime styling labels Apr 19, 2018
@1ec5 1ec5 added this to the ios-v4.0.1 milestone Apr 19, 2018
@fabian-guerra
Copy link
Contributor

Fixed in #11748

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
iOS Mapbox Maps SDK for iOS macOS Mapbox Maps SDK for macOS refactor runtime styling
Projects
None yet
Development

No branches or pull requests

2 participants