Skip to content
This repository has been archived by the owner on Jun 14, 2024. It is now read-only.

symbolZOrder example #245

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

symbolZOrder example #245

wants to merge 11 commits into from

Conversation

samfader
Copy link

Just a first draft for now!

@samfader
Copy link
Author

@captainbarbosa ready for review, pending addition of new images (as discussed)!

@samfader samfader changed the title WIP: symbolZOrder example symbolZOrder example Nov 19, 2018
Fix product bundle identifier

Fix SwiftLint line spacing warnings
Copy link
Contributor

@captainbarbosa captainbarbosa left a comment

Choose a reason for hiding this comment

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

Nice start! I left some minor style/formatting comments for the Objective-C example that also applies to the Swift example too. Reach out to me if you have any questions 🌟

Examples/Examples.m Outdated Show resolved Hide resolved
Examples/ObjectiveC/Headers/SymbolLayerZOrderExample.h Outdated Show resolved Hide resolved
Examples/ObjectiveC/SymbolLayerZOrderExample.m Outdated Show resolved Hide resolved
Examples/ObjectiveC/SymbolLayerZOrderExample.m Outdated Show resolved Hide resolved
// Add icons to the map's style.
[style setImage:([UIImage imageNamed:@"oval"]) forName:@"oval"];
[style setImage:([UIImage imageNamed:@"squircle"]) forName:@"squircle"];
[style setImage:([UIImage imageNamed:@"star"]) forName:@"star"];
Copy link
Contributor

Choose a reason for hiding this comment

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

Just for clarity, I recommend making the string used for the image name something more unique so that it doesn't match, like this:

[style setImage:([UIImage imageNamed:@"oval-image"]) forName:@"oval"];

That way once you set up your dictionary it would illustrate that each pair of the key/value strings are different values.

@samfader
Copy link
Author

Thanks, @captainbarbosa. Code updated per your comments. Now just waiting for you to create new images (thank you!) and then figure out about this Podfile conflict and we should be good to go.

@samfader
Copy link
Author

Might have found a bug blocking this from working: mapbox/mapbox-gl-native#13490

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants