Skip to content
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

Question: cordova-plugin-safariviewcontroller options #1489

Closed
SebastianSchirmer opened this issue May 6, 2017 · 3 comments · Fixed by #1490
Closed

Question: cordova-plugin-safariviewcontroller options #1489

SebastianSchirmer opened this issue May 6, 2017 · 3 comments · Fixed by #1490

Comments

@SebastianSchirmer
Copy link

The plugin cordova-plugin-safariviewcontroller (https://github.com/EddyVerbruggen/cordova-plugin-safariviewcontroller) offers the options "barColor" and "controlTintColor" to be passed in to the show method.

However, ionic-native/safari-view-controller, respectively the "SafariViewControllerOptions" interface does not specify these 2 options but option "toolbarColor" which the plugin doesn't offer.

My question is if and how the different options are connected.

  1. Am I right with my assumption that the ionic-native "toolbarColor" option value is passed to the plugin as its "barColor" option value?
  2. Is the plugin option "controlTintColor" just not supported by ionic-native? Or will a value for this option passed to the plugin but derived from any other option set for ionic-native?

Thanks very much for any clarification on this,
Sebastian

@danielsogl
Copy link
Owner

Hey @SebastianSchirmer,

I will take a look into this.

@danielsogl
Copy link
Owner

danielsogl commented May 6, 2017

I added the missing options.

Thank you for creating that issue.

And the toolbarColor is valid, I checked the plugin source.

final String toolbarColor = options.optString("toolbarColor");

@SebastianSchirmer
Copy link
Author

Thanks very much for adding the options. I checked the plugin source as well and found out that "toolbarColor" is used for Android only whereas "barColor", "tintColor" and "controlTintColor" are used for iOS only.

@SebastianSchirmer SebastianSchirmer changed the title Question: Question: cordova-plugin-safariviewcontroller options May 7, 2017
ihadeed pushed a commit that referenced this issue May 9, 2017
* update(SafariViewController): added missing options

fix #1489

* alphabetical

* fix linter

* Update index.ts

* Update index.ts
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 a pull request may close this issue.

2 participants