Return-Path: X-Original-To: apmail-cordova-dev-archive@www.apache.org Delivered-To: apmail-cordova-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 7E20719218 for ; Wed, 30 Mar 2016 18:59:34 +0000 (UTC) Received: (qmail 61059 invoked by uid 500); 30 Mar 2016 18:59:33 -0000 Delivered-To: apmail-cordova-dev-archive@cordova.apache.org Received: (qmail 61026 invoked by uid 500); 30 Mar 2016 18:59:33 -0000 Mailing-List: contact dev-help@cordova.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@cordova.apache.org Delivered-To: mailing list dev@cordova.apache.org Received: (qmail 60082 invoked by uid 99); 30 Mar 2016 18:59:33 -0000 Received: from git1-us-west.apache.org (HELO git1-us-west.apache.org) (140.211.11.23) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 30 Mar 2016 18:59:33 +0000 Received: by git1-us-west.apache.org (ASF Mail Server at git1-us-west.apache.org, from userid 33) id EE517DFC8E; Wed, 30 Mar 2016 18:59:32 +0000 (UTC) From: Panajev To: dev@cordova.apache.org Reply-To: dev@cordova.apache.org References: In-Reply-To: Subject: [GitHub] cordova-plugin-splashscreen pull request: Show splashscreen everyt... Content-Type: text/plain Message-Id: <20160330185932.EE517DFC8E@git1-us-west.apache.org> Date: Wed, 30 Mar 2016 18:59:32 +0000 (UTC) Github user Panajev commented on the pull request: https://github.com/apache/cordova-plugin-splashscreen/pull/95#issuecomment-203582719 @jcesarmobile Sorry for the early PR and for not really following all the steps of the process in the right order :/. **Signing** Goff Marocchi is me (actually Goffredo Marocchi) and yes I do still need to sign it, but I will ask James and Gabriela to sign it too. **Multiplatform Concerns** * The changes apply to both Android and iOS and we did pull out anything referring to custom UI for the splashscreen when the main URL is changed. The splashscreen being shown is always the same. * A point could be made about the Material Design like spinner. We would like to uniform the spinner design where it is possible and make it look more modern, but for example on iOS 6 this is not a trivial change for a platform which is less and less the target of developers worldwide. Also we may still need to commit work for the Android version of the plugin for the spinner too. * I understand your point about separating the spinner from the current PR, possibly breaking it into plugins this very plugin would be able to optonally use. Maybe broadcasting a message and expecting a new UI Element back? Will address this concern. **iPad Pro** I will look into a JIRA ticket for it. I will look for a default image for the cordova-ios project. We have forked it off for internal reasons so we are using a slightly modified version of cordova-ios 3.8.0. Still, I take your point and will try to address them. **iOS 8 bugfixes** I will look into a JIRA ticket for it. Should I still split it into a different PR here or should I include both JIRA ticket numbers in the PR? **HTML5 boilerplate license** Sorry, cruft I forgot to delete. Will remove it. **Clobber change** The thought behind the clobber was to namespace the cordova plugins, like we do for all other plugins we use internally, to a window.nativeWrapper. space to avoid collisions with other JavaScript libraries. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastructure@apache.org or file a JIRA ticket with INFRA. --- --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscribe@cordova.apache.org For additional commands, e-mail: dev-help@cordova.apache.org