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

Update Chaplin app #713

Merged
merged 3 commits into from
Nov 5, 2013
Merged

Update Chaplin app #713

merged 3 commits into from
Nov 5, 2013

Conversation

paulmillr
Copy link
Contributor

  1. Improve styling in Chaplin app.
  2. Switch Chaplin app from Backbone to Exoskeleton.

These two commits should be separated from each other since I want to show Chaplin users one particular commit with the switch to Exos.

@passy
Copy link
Member

passy commented Nov 5, 2013

Could you also update the entry on the homepage? It still says it's using Backbone.

@paulmillr
Copy link
Contributor Author

Well...Chaplin uses Backbone and this is the main selling point. Exoskeleton is just a better Backbone, but it is not widely known.

So I suggest to leave the "Backbone" there. But description could also be:

"description": "Chaplin is an architecture for JavaScript applications using the Backbone.js library. Chaplin addresses Backbone’s limitations by providing a lightweight and flexible structure that features well-proven design patterns and best practices. In the app, Backbone is replaced with [Exoskeleton](http://exosjs.com), faster and leaner Backbone without dependencies on jQuery and underscore.",

but it will be kinda big, won't it?

@passy
Copy link
Member

passy commented Nov 5, 2013

I agree, the shorter one is better. You explain your switch in the README, so that should be fine.

@passy
Copy link
Member

passy commented Nov 5, 2013

Hm, for some reason I'm not seeing the learn bar. Neither on the live version nor in your update. Do you know what's going on there?

@paulmillr
Copy link
Contributor Author

nope this was weird for me too

passy added a commit that referenced this pull request Nov 5, 2013
@passy
Copy link
Member

passy commented Nov 5, 2013

Haha, I think I know why now. :)

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 this pull request may close these issues.

3 participants