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

vNext #9

Open
wants to merge 96 commits into
base: master
Choose a base branch
from
Open

vNext #9

wants to merge 96 commits into from

Conversation

joscha
Copy link
Contributor

@joscha joscha commented Oct 5, 2012

Hi there,

as I am planning to use GMailr for a current project, I started to work on it quite a lot. It would be great if we could pull all the changes I made in, which include:

  • I fixed most of the broken pieces, including the numChange observer, the canvas, etc.
  • I implemented parts of the TODO (email details)
  • I switched from LABjs to yepnope.js, as LABjs is a discontinued project and made yepnope.js a submodule
  • I cleaned the loader, so the workaround with the chrome URLs in the div elements is not needed any more
  • I made jquery-bbq a submodule
  • I switched to the jQuery Google CDN module, as it runs off HTTPS
  • I cleaned the permissions in the manifest file
  • I fixed the unread count and pulled in all fixes from other people's forks
  • draft save/discard and starring/unstarring observers
  • I cleaned the code a lot
  • I switched it to use CoffeeScript to make it cleaner
  • I defined constants for the observers
  • I updated the documentation

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.

5 participants