Skip to content

Commit

Permalink
Revert "Bump jquery-rails version to prevent breakage."
Browse files Browse the repository at this point in the history
As it turns out, jQuery 1.8 has a few bugs in it. We'll suggest
jquery-rails 2.0 for the time being.

This reverts commit c547ad5.
  • Loading branch information
robyurkowski committed Aug 17, 2012
1 parent c547ad5 commit ea24cd6
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion Gemfile
Expand Up @@ -81,7 +81,7 @@ group :assets do
gem 'uglifier'
end

gem 'jquery-rails', '~> 2.1.0'
gem 'jquery-rails', '~> 2.0.0'

# To use debugger
# gem 'ruby-debug', :platform => :mri_18
Expand Down
2 changes: 1 addition & 1 deletion core/lib/generators/refinery/engine/templates/Gemfile
Expand Up @@ -68,4 +68,4 @@ group :assets do
gem 'uglifier'
end

gem 'jquery-rails', '~> 2.1.0'
gem 'jquery-rails', '~> 2.0.0'
2 changes: 1 addition & 1 deletion core/refinerycms-core.gemspec
Expand Up @@ -29,5 +29,5 @@ Gem::Specification.new do |s|
s.add_dependency 'will_paginate', '~> 3.0.2'
s.add_dependency 'sass-rails', '~> 3.2.3'
s.add_dependency 'coffee-rails', '~> 3.2.1'
s.add_dependency 'jquery-rails', '~> 2.1.0'
s.add_dependency 'jquery-rails', '~> 2.0.0'
end

3 comments on commit ea24cd6

@simi
Copy link
Member

@simi simi commented on ea24cd6 Jan 14, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@robyurkowski still an issue?

@ericboehs
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I too am curious.

@robyurkowski
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It was tested in IRC and we had a bunch of tests fail still. Someone is free to check if jQuery 1.9 will solve the issue, but I think this will have to stay into place until jQuery becomes compliant or we throw in the towel.

Please sign in to comment.