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

fixed spacing problem for right-aligned frametitles without background #898

Merged
merged 1 commit into from
Jun 14, 2024

Conversation

samcarter
Copy link
Collaborator

@samcarter samcarter commented Jun 14, 2024

For users of the right aligned default frametitle template, beamer currently inserts additional vertical and horizontal space:

\documentclass{beamer}

\usepackage{lipsum}
\usepackage{ragged2e}

%\setbeamercolor{frametitle}{bg=red}
\setbeamertemplate{frametitle}[default][right]

\begin{document}

\begin{frame}
\frametitle{title title title title title title title title title title title title title title title}
  \justifying
  \lipsum[]
\end{frame}

\end{document}

document-1

This does not seem consistent with the left aligned template:

document-1

nor with the right aligned frametitle if a background colour is set

document-1


Simple fix:

One could avoid this by removing the line

https://github.com/josephwright/beamer/blob/main/base/themes/outer/beamerouterthemedefault.sty#L184

The result would look like this:

document-1


The question is: what was the purpose of this line?

  • the line was added at the same time as the frametitle template, empty commit message 1d2dc8b . So no clues from there
  • I thought maybe for sidebar themes, but they don't seem to need the additional spacing

@josephwright
Copy link
Owner

We'll go for it and see if we get complaints - it's one of the Till things we have no idea about!

@josephwright josephwright merged commit f7ec244 into josephwright:main Jun 14, 2024
1 check passed
@samcarter
Copy link
Collaborator Author

@josephwright Thanks for taking a look!

@samcarter samcarter deleted the right_frametitle branch June 14, 2024 16:29
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