[
https://issues.apache.org/jira/browse/CB-7973?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
jcesarmobile closed CB-7973.
----------------------------
Resolution: Won't Do
Closing as won't fix.
> js/index.js is TL;DR and obscures the key point
> -----------------------------------------------
>
> Key: CB-7973
> URL: https://issues.apache.org/jira/browse/CB-7973
> Project: Apache Cordova
> Issue Type: Bug
> Components: cordova-app-hello-world
> Affects Versions: 3.5.0
> Reporter: Josh Soref
> Priority: Major
>
> Today, on #cordova, we had a user acknowledge that they looked over
> js/index.js, didn't understand that it was doing anything important, and
> replaced it.
> Unfortunately, in doing this, they removed
> document.addEventListener('deviceready', ..., false);
> Which is really the only important part of the code.
> Object Oriented Code is nice for reuse, but it's awful for samples where
> people aren't going to use your code and won't be able to tease out what's
> important, and what's just app-specific.
> A number of suggestions were given.
> * highlight jQuery equivalence (jQuery(...).ready() ?))
> * drop OOP
> I toyed with the idea of including a link to the documentation on deviceready
> with a comment explaining that you can't use any cordova stuff before it has
> been called.
> But, I'm just opening this bug to track the issue.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org
