using image as input type in submit_to_remote

Hi all,

After searching for a way to set the input type of a submit tag to image when using submit_to_remote I realized that the method call given below, while seemingly valid, wasn't working since the method button_to_function in actionpack/lib/action_view/helpers/ javascript_helper.rb was forcing the type to button.

1. The method call in the view: <%=    submit_to_remote(      'update_resource_img',      'Update Resource 2',      :url => { :controller => 'hello', :action => 'update' },      :before => "$('loading').style.display = 'block'",      :after => "return false",      :html => { :src => '/images/button.gif', :type => 'image' }    ) %>

2. def button_to_function in action_view/helpers/javascript_helper.rb does the following(note the :type => 'button') : tag(:input, html_options.merge(:type => 'button', :value => name, :onclick => onclick))

3. It is my opinion (finally :p) that the def button_to_remote should allow the input type to be either a button or an image since the w3c page on form input control types states that the input type image "Creates a graphical submit button." http://www.w3.org/TR/html401/interact/forms.html#h-17.4.1

4. Hence, to ensure that button_to_remote allows me to set the type of either of button or image, I place the following in a file in $RAILS_ROOT/lib and require it at the end of $RAILS_ROOT/config/ environment.rb

module ActionView   module Helpers     module JavaScriptHelper

      def button_to_function(name, *args, &block)         html_options = args.extract_options!.symbolize_keys

        function = block_given? ? update_page(&block) : args[0] || ''         onclick = "#{"#{html_options[:onclick]}; " if html_options [:onclick]}#{function};"

        html_options[:type] = 'button' if( !html_options[:type] || html_options[:type] !~ /^image|button$/i )

        tag(:input, html_options.merge(:value => name, :onclick => onclick))       end

    end   end end

5. Hope you guys have an opinion to share.

- Gautam   Programmer @ Azri