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

Ensure Presenter height never grows beyond screen height #4

Merged
merged 2 commits into from
May 12, 2012

Conversation

rschroll
Copy link

A full-screen window that has a height greater than the screen height experiences all sorts of weird, wonderful breakage, at least in Gnome 3. I found two ways in which this could happen:

  1. The current view + strict view + bottom bar could be taller than the screen.
  2. The notes view could extend downwards as text was added.
    These commits fix both issues.

Otherwise, the fullscreening may be broken in odd ways, at least in
Gnome 3. We give the current slide view up to 0.9*0.8 of the height and
the strict views up to 0.9*0.2 (minus padding) of the height, with the
status bar at the bottom getting the last 0.1.  We look at the size
request for the current view and set the width of the next view to fill
the remaining space (and hope that the allocation will be as requested).
This ensures that it doesn't grow in height and change the window's
total height, which can mess up the full-screen effect.
@davvil
Copy link
Owner

davvil commented May 12, 2012

Thanks for the patch!

For the notes I can see the problem is solved. For the slides I do not have a real-life example to work on and test it. Could you PM some slides where the problem occurs? I tried opening a "book" (i.e. several portrait A4 pages) and the window was resized in a strange way, but this is not a case we should concentrate on, I think. Although it should not break there either...

@rschroll
Copy link
Author

Try launching a normal presentation with '-u 90'. With a wide-screen monitor, I actually had problems with the default settings. I can send you screenshots, if that would help.

@davvil davvil merged commit f56c1b5 into davvil:master May 12, 2012
@davvil
Copy link
Owner

davvil commented May 12, 2012

Merged. There may still problems with patological cases though, e.g. when opening A4 pages. With normal usage there shouldn't be issues, but I will add similar checks also for the preview so that pdfpc is more robust.

@tinloaf tinloaf mentioned this pull request Jan 2, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants