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

Fix for lack of ActionView::Base::InstanceTag in Rails 4 #16

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

Conversation

maxmcd
Copy link

@maxmcd maxmcd commented Oct 18, 2013

Based on discussion here: galetahub/ckeditor#269
Code here: mjhoy/ckeditor@fca2402

@romanenko
Copy link

Fixed issues with Rails 4 for me. +1 for merge

@binary1101
Copy link

Fixed issue with Rails 4 for me too. +1

icedtoast added a commit to icedtoast/sir-trevor-rails that referenced this pull request Jan 5, 2014
Use InstanceTag for Rails 3 and TextArea otherwise. This allows
sir-trevor-rails to work on Rails 4.

This is based on the fix for the same issue contained within
madebymany#16
@Fudoshiki
Copy link

dont work

@bswinnerton
Copy link

Works for me as well. 👍

@thebucknerlife
Copy link

Was this fixed? Still having this issue using Rails 4.1.4 and sir-travis-rails 0.2.4.

dwb pushed a commit that referenced this pull request Jan 9, 2015
Use InstanceTag for Rails 3 and TextArea otherwise. This allows
sir-trevor-rails to work on Rails 4.

This is based on the fix for the same issue contained within
#16
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.

6 participants