That's a nice improvement. Looks functionally equivalent. As long as the
tests pass it's good.

On 29 January 2013 23:11, Philip Arndt <notifications@github.com> wrote:

> In images/app/views/refinery/admin/images/insert.html.erb:
>
> > @@ -42,6 +42,11 @@
> >          callback: <%= @callback.present? ? "self.parent.#{h @callback}" : 'null' %>,
> >          multiple: <%= @multiple == 'true' %>
> >        }).create();
> > +
> > +      var $uploadRadio = $('input[type=radio][value=upload_image]');
> > +      if ($uploadRadio.is(':checked')) {
> > +        $uploadRadio.click();
>
> Could we query this and only find it when checked using:
>
> $('input[type=radio][value=upload_image]:checked').click();
>
> ?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/refinery/refinerycms/pull/2168/files#r2816051>.
>
>