Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PDF preview in fullscreen mode - SumatraPDF #4301

Closed
gk-crop opened this issue Feb 12, 2019 · 1 comment · Fixed by #5827
Closed

PDF preview in fullscreen mode - SumatraPDF #4301

gk-crop opened this issue Feb 12, 2019 · 1 comment · Fixed by #5827

Comments

@gk-crop
Copy link

@gk-crop gk-crop commented Feb 12, 2019

When knitting a rmarkdown document to pdf, the bundled SumatraPDF viewer is started. Unfortunately fullscreeen mode is disabled, as the file bin/sumatra/sumatrapdfrestrict.ini is missing the setting
FullscreenAccess = 1
Adding this option by default would be very helpful, especially when working with pdf beamer presentations.

System details

RStudio Edition : Desktop
RStudio Version : 1.2.1268
OS Version      : Win7
R Version       : 3.52

Steps to reproduce the problem

Knit RMarkdown (beamer presentation) to PDF, view the file in bundled SumatraPDF, hit F5 (or CTRL+L) to switch to fullscreen mode.

Describe the problem in detail

SumatraPDF does not allow fullscreen mode as it uses bin/sumatra/sumatrapdfrestrict.ini, where fullscreen mode has to be explicitely enabled via FullscreenAccess = 1

Describe the behavior you expected

SumatraPDF switches to fullscreen when hitting F5

@kevinushey
Copy link
Contributor

@kevinushey kevinushey commented Feb 13, 2019

Perhaps worth comparing what we have with the default version at https://github.com/sumatrapdfreader/sumatrapdf/blob/master/docs/sumatrapdfrestrict.ini now. (looks like FullScreenAccess is the only new field)

@kevinushey kevinushey added this to the v1.3 milestone Feb 19, 2019
@jmcphers jmcphers self-assigned this Dec 2, 2019
@jmcphers jmcphers removed this from the v1.3 milestone Mar 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants