[openstreetmap-website] Move sidebar JS to application bundle (#76)

John Firebaugh notifications at github.com
Thu Aug 30 17:47:27 BST 2012


It's required on the main page so will nearly always be loaded anyway.

Enclosed in anonymous function to avoid leaking `onclose` global variable.

You can merge this Pull Request by running:

  git pull https://github.com/jfirebaugh/openstreetmap-website sidebar_js

Or you can view, comment on it, or merge it online at:

  https://github.com/openstreetmap/openstreetmap-website/pull/76

-- Commit Summary --

  * Remove unnecessary arguments to sidebar partial
  * Remove unused function
  * Move sidebar JS to application bundle

-- File Changes --

M app/assets/javascripts/application.js (1)
A app/assets/javascripts/sidebar.js (39)
M app/views/site/_sidebar.html.erb (44)
M app/views/site/edit.html.erb (2)
M app/views/site/index.html.erb (2)

-- Patch Links --

https://github.com/openstreetmap/openstreetmap-website/pull/76.patch
https://github.com/openstreetmap/openstreetmap-website/pull/76.diff


---
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/76
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20120830/7fc1a996/attachment.html>


More information about the rails-dev mailing list