[MOD-2268] iOS/Android: Ti.Map - Support for the visible property, add circle-opacity, refactor old view, fix deprecations
GitHub Issue | n/a |
---|---|
Type | Improvement |
Priority | Critical |
Status | Closed |
Resolution | Fixed |
Resolution Date | 2017-03-20T19:20:49.000+0000 |
Affected Version/s | n/a |
Fix Version/s | Release 6.1.0 |
Components | Map |
Labels | Ti.Map, android, module |
Reporter | Rodolfo Perottoni |
Assignee | Hans Knöchel |
Created | 2016-07-10T22:27:00.000+0000 |
Updated | 2017-05-18T22:11:19.000+0000 |
Description
Currently we don't have access to the *visible* property of *Markers*. It would be good to have access to that. Also, it would be nice to support the *alpha* property of the native MKCircle.
*EDIT by Hans*: Additionally, I'd like to remove the legacy
TiMapIOS7View
class that has been around for < iOS 7 support. Nowadays we do not support this anymore, so we should clean it up.
PR (cherry-picked): https://github.com/appcelerator-modules/ti.map/pull/192
Test code for visible property of annotation-
[~rdperottoni] The
visible
property does not seem to work when you set it after the initial state. I'll try to figure out why.Verified implemented using iOS module 2.9.0 and Android module 3.1.1. Visible property (hidden on iOS), Circles, Opacity property for circles and color values for fill-color on circles and pin colors are all working. A parity issue was found during testing where iOS will accept strings and hex values for colors on circles, pins, and annotations, where Android will only accept the constants. This difference is documented, however. Thanks [~smohammed] for the Android testing.