Mailing-List: contact dev-help@cordova.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cordova.apache.org
MIME-Version: 1.0
In-Reply-To: 
 <DM2PR03MB3495DFBBE00F5D9E562C6B0DAAD0@DM2PR03MB349.namprd03.prod.outlook.com>
References: 
 <BL2PR03MB1164800815D06D1342FF1A7D6A90@BL2PR03MB116.namprd03.prod.outlook.com>
	<CAHYtTqkUX94YbfJiMXbZ1g-uN30rTdw5BmQPNTzPewOzrD8mCQ@mail.gmail.com>
	<92D789D9-B047-47EA-9856-330F63EC7AD2@microsoft.com>
	<CAHYtTqmSiYGM94RW7O3WU5_YYevYD14Gjj1E7PHtHz4kFLkMnw@mail.gmail.com>
	<BL2PR03MB1160B7E8A36D6E79AB75EADD6AD0@BL2PR03MB116.namprd03.prod.outlook.com>
	<CAMSbhBg933wFKNwWcbgonEYZ3xj7ryvR7sE87ihFEthYgdwGbg@mail.gmail.com>
	<BL2PR03MB116DFF8A1BB29B4B4E9FBC9D6AD0@BL2PR03MB116.namprd03.prod.outlook.com>
	<CAKP6k2tA+V8HGT=kKAW3RF7dGAoG7zOE6RHp1toLX1RJ9BKBAQ@mail.gmail.com>
	<BL2PR03MB116EA1E253ACC885D841010D6AD0@BL2PR03MB116.namprd03.prod.outlook.com>
	<DM2PR03MB3495DFBBE00F5D9E562C6B0DAAD0@DM2PR03MB349.namprd03.prod.outlook.com>
Date: Thu, 18 Feb 2016 10:46:53 -0600
Message-ID: 
 <CAK2avF_=wXKhndejqUUU24c_gQe3i_X5-Pqu0ADH-PG3Z8NZfQ@mail.gmail.com>
Subject: Re: [DISCUSS] Update to starter template
From: Raymond Camden <raymondcamden@gmail.com>
To: dev@cordova.apache.org
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

Big plus one to Jesse's comment. I've felt like the template is too
over complex for quite some time.

On Tue, Feb 16, 2016 at 3:21 PM, Ryan J. Salva <rsalva@microsoft.com> wrote=
:
> Check out the default template we use in TACO-CLI.
>
> Index.js is a lot simpler and we adopted the new Cordova typeface. If fol=
ks like this template, we could issue a PR for it.
>
> rjs
>
>
> Ryan J. Salva  |  Principal PM Manager
> Visual Studio Tools for Apache Cordova
> rsalva@microsoft.com
> 425 706 5270 office
> 206 612 5079 mobile
>
> -----Original Message-----
> From: Parashuram N [mailto:panarasi@microsoft.com]
> Sent: Tuesday, February 16, 2016 12:46 PM
> To: dev@cordova.apache.org
> Subject: RE: [DISCUSS] Update to starter template
>
> I do agree that our starter app needs to change. However, we encountered =
this when we were looking at documenting, and our object of concern was the=
 way events are bound. My suggestion would be to simply fix the event bindi=
ng so that the docs are less confusing.
>
> We could then look at re-writing the starter-app itself.
>
> -----Original Message-----
> From: Jesse [mailto:purplecabbage@gmail.com]
> Sent: Tuesday, February 16, 2016 12:28 PM
> To: dev@cordova.apache.org
> Subject: Re: [DISCUSS] Update to starter template
>
> Personally, I think we should rewrite/simplify that template ALOT.
> The app binding stuff is confusing to users, but not potentially any less=
 confusing with PR#15. The use of an object is the issue in my mind.
>
> Also, if this is the starting point for a new developer, they will soon w=
onder why everything they add to the page is UPPERCASE, part of the css, bu=
t potentially confusing.
>
> Ultimately, I think the whole starting point is over-organized with it's =
multiple folders.  PR#5 is closer to the goal I think, although I would sti=
ll leave js in an external file, and not directly in index.html and I would=
 leave quotes on html tag attributes.
>
>
>
>
>
> @purplecabbage
> https://na01.safelinks.protection.outlook.com/?url=3Drisingj.com&data=3D0=
1%7c01%7cpanarasi%40microsoft.com%7ca55c7ca45ac24069c54708d337118610%7c72f9=
88bf86f141af91ab2d7cd011db47%7c1&sdata=3D%2bCP4GywoLaqppsEXaqZxo%2br%2baBb0=
%2b%2babaAImEmW4ZMg%3d
>
> On Tue, Feb 16, 2016 at 11:07 AM, Parashuram N <panarasi@microsoft.com>
> wrote:
>
>> Will do.
>>
>> -----Original Message-----
>> From: Steven Gill [mailto:stevengill97@gmail.com]
>> Sent: Tuesday, February 16, 2016 10:44 AM
>> To: dev@cordova.apache.org
>> Subject: Re: [DISCUSS] Update to starter template
>>
>> LGTM
>>
>> Could you also add the keyword 'cordova:template' to package.json?
>>
>> On Tue, Feb 16, 2016 at 9:58 AM, Parashuram N <panarasi@microsoft.com>
>> wrote:
>>
>> > :) Sounds good
>> >
>> > Anyone else have any objections to the change ?
>> >
>> > -----Original Message-----
>> > From: Carlos Santana [mailto:csantana23@gmail.com]
>> > Sent: Monday, February 15, 2016 7:51 PM
>> > To: dev@cordova.apache.org
>> > Subject: Re: [DISCUSS] Update to starter template
>> >
>> > I'm always in favor of having simple answers and guidelines, and we
>> > can take this as a good convention/practice
>> >
>> > "Always bind/attach after deviceready even and you will be in a
>> > happy place"
>> >
>> > But between you and me you know that we can attach before, will
>> > leave that little secret between us :-)
>> >
>> >
>> > On Mon, Feb 15, 2016 at 10:44 PM Parashuram N
>> > <panarasi@microsoft.com>
>> > wrote:
>> >
>> > > The reason to say that all events are fired after device ready is
>> > > just to make it simpler. As you say, events by webviews are indeed
>> > > fired before that, but I think it is safe to say that if we attach
>> > > listeners after deviceready, we are guaranteed to have the plugins
>> > > also initialized. This makes for a more consistent story.
>> > >
>> > >
>> > >
>> > > On 2/15/16, 7:25 PM, "Carlos Santana" <csantana23@gmail.com> wrote:
>> > >
>> > > >Why do you think that binding is required after "deviceready" ?
>> > > >
>> > > >Attaching before "deviceready" fires is still supported for
>> > > >events like resume, pause, offline, online, etc.. At least that's
>> > > >always being my impression for a long time
>> > > >
>> > > >The only thing I would think it could be a potential confusion
>> > > >that would benefit to attached after "deviceready" is that if
>> > > >there is an "online" or "offline" event fire by webview [1], [2]
>> > > >and not plugin those handlers
>> > > will
>> > > >fire but will not be from the network plugin.
>> > > >
>> > > >One thing I like thought about the PR is making index.js less
>> > > >verbose and simpler to follow +1
>> > > >
>> > > >[1]:
>> > > https://na01.safelinks.protection.outlook.com/?url=3Dhttps%3a%2f%2fd
>> > > ev
>> > > el
>> > > oper.mozilla.org%2fen-US%2fdocs%2fWeb%2fEvents%2foffline&data=3D01%7
>> > > c0
>> > > 1%
>> > > 7cpanarasi%40microsoft.com%7c54f8ca7873ae4337a50608d33680f157%7c72
>> > > f9
>> > > 88
>> > > bf86f141af91ab2d7cd011db47%7c1&sdata=3D8%2b4NzK8zAB%2ffqryeM%2fN9XYn
>> > > mu
>> > > lm
>> > > bzlAeIogT9H1b4ic%3d
>> > > >[2]:
>> > > https://na01.safelinks.protection.outlook.com/?url=3Dhttps%3a%2f%2fd
>> > > ev
>> > > el
>> > > oper.mozilla.org%2fen-US%2fdocs%2fWeb%2fEvents%2fonline&data=3D01%7c
>> > > 01
>> > > %7
>> > > cpanarasi%40microsoft.com%7c54f8ca7873ae4337a50608d33680f157%7c72f
>> > > 98
>> > > 8b
>> > > f86f141af91ab2d7cd011db47%7c1&sdata=3DVLBX15Tz2VxbDC1JuS0H3qm1KGuRv7
>> > > Cd
>> > > IA
>> > > XaRnZbqR4%3d
>> > > >
>> > > >
>> > > >On Mon, Feb 15, 2016 at 12:36 PM Parashuram N
>> > > ><panarasi@microsoft.com>
>> > > >wrote:
>> > > >
>> > > >> While working on documentation about events for Cordova, we
>> > > >> noticed that in our Cordova starter template [1] seems to
>> > > >> suggest that we bind events like =E2=80=9Coffline=E2=80=9D and =
=E2=80=9Conline=E2=80=9D
>> > > >> alongside the
>> =E2=80=9Cdeviceready=E2=80=9D
>> > > >> event. As I understand, the =E2=80=9Coffline=E2=80=9D and =E2=80=
=9Conline=E2=80=9D events are
>> > > >> something that the NetworkInformation plugin provides, and thus
>> > > >> should only be bound after cordova-js has been initialized.
>> > > >> Here [2] is a change where we only bind =E2=80=9Cdeviceready=E2=
=80=9D first,
>> > > >> and then recommend binding all events when the handler for
>> > > >> =E2=80=9Cdeviceready=E2=80=9D is
>> > > called.
>> > > >> Do you folks think this is a good change ? Here [3] is the JIRA
>> > > >> for
>> > it.
>> > > >>
>> > > >>
>> > > >> [1] Cordova Started template -
>> > > >> https://github.com/apache/cordova-app-hello-world/
>> > > >> [2] Pull request with change to starter template -
>> > > >> https://github.com/apache/cordova-app-hello-world/pull/15/files
>> > > >> [3] JIRA for PR with change to starter template -
>> > > >>
>> > > https://na01.safelinks.protection.outlook.com/?url=3Dhttps%3a%2f%2fi
>> > > ss
>> > > ue
>> > > s.apache.org%2fjira%2fbrowse%2fCB-10522&data=3D01%7c01%7cpanarasi%40
>> > > mi
>> > > cr
>> > > osoft.com%7c54f8ca7873ae4337a50608d33680f157%7c72f988bf86f141af91a
>> > > b2
>> > > d7
>> > > cd011db47%7c1&sdata=3D66cC5xZm3Rh6xzxjqV48u%2bdrZSJZba1g%2f%2beFnwL7
>> > > aG
>> > > k%
>> > > 3d
>> > > >>
>> > > >>
>> > > >>
>> > >
>> > > ------------------------------------------------------------------
>> > > --
>> > > - To unsubscribe, e-mail: dev-unsubscribe@cordova.apache.org
>> > > For additional commands, e-mail: dev-help@cordova.apache.org
>> > >
>> >
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: dev-unsubscribe@cordova.apache.org
>> For additional commands, e-mail: dev-help@cordova.apache.org
>>


--=20
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
Raymond Camden, Developer Advocate for MobileFirst at IBM

Email : raymondcamden@gmail.com
Blog : www.raymondcamden.com
Twitter: raymondcamden

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@cordova.apache.org
For additional commands, e-mail: dev-help@cordova.apache.org