I'm willing to live with it being a bit "off" -- where we stand with
options right now is that if we add another category of option or even move
something into "General" there will be some work to make more room. Hoping
we can avoid this until next release....


On Wed, Mar 19, 2014 at 1:08 PM, Jonathan <notifications@github.com> wrote:

> In
> src/gwt/src/org/rstudio/studio/client/workbench/prefs/views/CompilePdfPreferencesPane.java:
>
> > @@ -157,8 +158,11 @@ protected void initialize(RPrefs prefs)
> >                                 UIPrefsAccessor.PDF_PREVIEW_DESKTOP_SYNCTEX);
> >        }
> >
> > -      pdfPreview_.addChoice("RStudio Viewer",
> > -                            UIPrefsAccessor.PDF_PREVIEW_RSTUDIO);
> > +      if (UIPrefsAccessor.internalPdfPreviewSupported())
> > +      {
> > +         pdfPreview_.addChoice("RStudio Viewer",
> > +                               UIPrefsAccessor.PDF_PREVIEW_RSTUDIO);
> > +      }
> >
> >        pdfPreview_.addChoice("System Viewer",
> >                              UIPrefsAccessor.PDF_PREVIEW_SYSTEM);
>
> We might also want to consider moving that option--for anyone who wants to
> change the viewer for R Markdown PDFs, I'm not sure they'll ever find the
> option under Sweave (unless they happen to be Sweave users too, of course).
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/rstudio/rstudio/pull/101/files#r10759454>
> .
>