Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Whitespace/Semicolons #29

Merged
merged 1 commit into from Aug 27, 2012

Conversation

Projects
None yet
3 participants
Contributor

brettz9 commented Aug 27, 2012

  1. except in output and libraries, change tab to two-space convention already established by project
  2. Remove trailing spaces (unless prepared at indent for new insertion)
  3. Add a couple missing semicolons as per project's established convention

(Whitespace esp. was making difficult to read on Github; editors (like Notepad++) can be configured to convert tabs to 2 spaces)

1) except in output and libraries, change tab to two-space convention…
… already established by project

2) Remove trailing spaces (unless prepared at indent for new insertion)
3) Add a couple missing semicolons as per project's established convention

thomasdavis added a commit that referenced this pull request Aug 27, 2012

@thomasdavis thomasdavis merged commit 5a14b5a into thomasdavis:gh-pages Aug 27, 2012

1 check was pending

default The Travis build is in progress
Details
Owner

thomasdavis commented Aug 27, 2012

Looking sexy, thanks! should collaborate on browser extensions one day, i'd build the chrome side lol

Contributor

brettz9 commented Aug 27, 2012

Mp, thank you...Hehe--I've been moving more toward HTML5 (and hoping HTML5 editors may be convinced to expose more privileges with permissions, or at least building or using an extension/API that allows such requests within HTML). But my cosmetic changes are mostly for my getting a good understanding of Backbone, rather than out of any mastery of it. My hopes with Backbone (to the extent I have free time) at present are:

  1. See if I can use on Common-Node+Mongoose and share with the client for it to reuse MVC code with an adapter on IndexedDB.
  2. Allow the templating to plug-in a JS or JSON object to be converted to HTML (or even CSS). As you can see, I am really keen on the ability to use the same syntax everywhere. :)
    But feel free to share any ideas if you've got something and are serious about it...

This pull request fails (merged f29d09c into cc43726).

This pull request passes (merged f29d09c into cc43726).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment