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

default.latex: wrong beamer color in (sub)section page #10026

Merged
merged 1 commit into from
Jul 29, 2024

Conversation

jdgx
Copy link
Contributor

@jdgx jdgx commented Jul 24, 2024

The default.latex template defines "part title", "section title" and "subsection title" via \setbeamertemplate. But for "section title" and "subsection title", the beamercolorbox also used the "part title" parameter instead of the "(sub)section title". This had the effect that for (sub)section pages the color attributes of the part title were used instead of the ones for (sub)section title.

PS: It's my first pull request, please let me know if I'm doing something wrong or against good practice. I wasn't sure if I should open an issue first (or in addition to the pull request). Since this fix seems quite trivial, I just created the pull request.

Best,
Jonathan

@jgm
Copy link
Owner

jgm commented Jul 28, 2024

Thanks, this looks great! Only thing is to reformat the commit message so that the lines are not more than 80 characters wrong (hard wrap). That makes it easier when it comes time to make a changelog. See CONTRIBUTING.md under Pull Requests.
If you make that change and force push, I'll merge this.

@jdgx jdgx force-pushed the fix-beamercolorbox-for-sections branch 2 times, most recently from 23d53e8 to d1c47e3 Compare July 29, 2024 07:15
The default.latex template defines "part title", "section title" and
"subsection title" via \setbeamertemplate. But for "section title" and
"subsection title", the beamercolorbox also used the "part title"
parameter instead of the "(sub)section title". This had the effect that
for (sub)section pages the color attributes of the part title were used.
@jdgx jdgx force-pushed the fix-beamercolorbox-for-sections branch from d1c47e3 to 9c97112 Compare July 29, 2024 07:23
@jdgx
Copy link
Contributor Author

jdgx commented Jul 29, 2024

Ok, the commit message is now fixed.

@jgm jgm merged commit d896811 into jgm:main Jul 29, 2024
9 of 12 checks passed
@jgm
Copy link
Owner

jgm commented Jul 29, 2024

thanks!

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