-
Notifications
You must be signed in to change notification settings - Fork 8
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
[New] Show result of spatial relationships #28
Conversation
private var pointGraphic: Graphic { graphicsOverlay.graphics.last! } | ||
/// The polyline graphic. | ||
private var polylineGraphic: Graphic { graphicsOverlay.graphics[1] } | ||
/// The polygon graphic. | ||
private var polygonGraphic: Graphic { graphicsOverlay.graphics.first! } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
private var pointGraphic: Graphic { graphicsOverlay.graphics.last! } | |
/// The polyline graphic. | |
private var polylineGraphic: Graphic { graphicsOverlay.graphics[1] } | |
/// The polygon graphic. | |
private var polygonGraphic: Graphic { graphicsOverlay.graphics.first! } | |
private var pointGraphic: Graphic { graphicsOverlay.graphics[2] } | |
/// The polyline graphic. | |
private var polylineGraphic: Graphic { graphicsOverlay.graphics[1] } | |
/// The polygon graphic. | |
private var polygonGraphic: Graphic { graphicsOverlay.graphics[0] } |
😂 I'm not sure if it's the way to go, but using all subscripts make more sense to me.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think using first
and last
is more swifty and safer even though we are force unwrapping in this case.
Shared/Samples/Show result of spatial relationships/ShowResultOfSpatialRelationshipsView.swift
Outdated
Show resolved
Hide resolved
Shared/Samples/Show result of spatial relationships/ShowResultOfSpatialRelationshipsView.swift
Outdated
Show resolved
Hide resolved
Shared/Samples/Show result of spatial relationships/ShowResultOfSpatialRelationshipsView.swift
Outdated
Show resolved
Hide resolved
Shared/Samples/Show result of spatial relationships/ShowResultOfSpatialRelationshipsView.swift
Outdated
Show resolved
Hide resolved
Shared/Samples/Show result of spatial relationships/ShowResultOfSpatialRelationshipsView.swift
Outdated
Show resolved
Hide resolved
Shared/Samples/Show result of spatial relationships/ShowResultOfSpatialRelationshipsView.swift
Show resolved
Hide resolved
Shared/Samples/Show result of spatial relationships/ShowResultOfSpatialRelationshipsView.swift
Show resolved
Hide resolved
Looks good with the changes. Please resolve the conflict and I'll approve again. |
Description
This PR implements
Show result of spatial relationships
inGeometry
category.URL to README: URL
Linked Issue(s)
common-samples/issues/3627
How To Test
Screenshots
To Discuss
Callout
How the information in the callout is shown could probably be improved. Maybe the addition of a divider could be nice; however, I would then have to specify a frame for the callout as the divider expands the width of the view.