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

Support title format modifications #18

Merged
merged 4 commits into from
Jul 9, 2024
Merged

Support title format modifications #18

merged 4 commits into from
Jul 9, 2024

Conversation

jolars
Copy link
Owner

@jolars jolars commented Jul 9, 2024

Fixes #17. This pull requests brings back support for title format modifications that were previously removed since they patched beamer commands, which I want to avoid. Many of the problems that were listed as issues in the metropolis documentation seem to have magically disappeared also, so the options now seem much more robust.

This PR also includes a small modification to the titleformat spacing, but it should hardly be noticeable.

Everything seems to be working fine for me, but I would appreciate help to bug test this.

jolars added 4 commits July 9, 2024 10:58
Fixes #17. Setting this up on the user side via beamer templates turned
out to be too much of a hassle for users. It seems that the
implementation here works better out of the box compared to what was
originally the case in the metropolis theme, and we don't have to rely
on patching commands as the previous implementation did, so this seems
like a reasonable feature to add back support for.
@jolars jolars merged commit 0b25d18 into main Jul 9, 2024
2 checks passed
@jolars jolars deleted the support-titleformats branch July 9, 2024 18:36
jolars added a commit that referenced this pull request Jul 9, 2024
* chore: add "to"

* docs: small change to example

* docs: provide longest label in description

* feat: reinstitute support for titleformat settings

Fixes #17. Setting this up on the user side via beamer templates turned
out to be too much of a hassle for users. It seems that the
implementation here works better out of the box compared to what was
originally the case in the metropolis theme, and we don't have to rely
on patching commands as the previous implementation did, so this seems
like a reasonable feature to add back support for.
@jolars jolars mentioned this pull request Jul 9, 2024
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.

Uppercase frame titles do not work
1 participant