That sounds just fine, let's leave it as is.

J


On Tue, Sep 24, 2013 at 12:05 PM, Jonathan <notifications@github.com> wrote:

> In
> src/gwt/src/org/rstudio/studio/client/workbench/views/environment/EnvironmentPane.java:
>
> > @@ -358,6 +360,27 @@ public void viewObject(String objectName)
> >        executeFunctionForObject("View", objectName);
> >     }
> >
> > +   public void fillObjectContents(final RObject object,
> > +                                  final Operation onCompleted)
> > +   {
> > +      server_.getObjectContents(object.getName(),
>
> Right now progress is shown immediately--clicking on the expander causes
> it to be replaced with a spinner until the operation finishes. Let me know
> if this feels too heavy in practice; we could also use a delayed indicator.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/rstudio/rstudio/pull/66/files#r6547808>
> .
>