pdf viewer should occupy ide window size (capped at 1200 pixel width)

rstudio · Feb 27, 2012 · ec4d996 · ec4d996
1 parent 4b96cda
commit ec4d996
Showing 1 changed file with 11 additions and 1 deletion.
diff --git a/src/gwt/src/org/rstudio/studio/client/pdfviewer/PDFViewer.java b/src/gwt/src/org/rstudio/studio/client/pdfviewer/PDFViewer.java
@@ -19,6 +19,7 @@
 import org.rstudio.studio.client.pdfviewer.events.ViewPdfHandler;
 import org.rstudio.studio.client.pdfviewer.model.PDFViewerParams;
 
+import com.google.gwt.user.client.Window;
 import com.google.inject.Inject;
 import com.google.inject.Singleton;
 
@@ -42,10 +43,19 @@ public void onViewPdf(ViewPdfEvent event)
       // open the window 
       satelliteManager_.openSatellite(PDFViewerApplication.NAME,     
                                       params,
-                                      new Size(800,900)); 
+                                      getPreferredWindowSize()); 
 
    }
 
+   private Size getPreferredWindowSize()
+   {
+      Size windowBounds = new Size(Window.getClientWidth(),
+                                   Window.getClientHeight());
+
+      return new Size(Math.min(windowBounds.width - 100, 1200),
+                      windowBounds.height - 25);
+   }
+
 
    private final SatelliteManager satelliteManager_;
 }