SoftVarE-Group / SlideTemplate

A beamer latex template for slides and lectures at University of Ulm
Creative Commons Zero v1.0 Universal
13 stars 3 forks source link

fix: replace fitbox in frame title for using hfill #71

Closed Benno2000 closed 1 year ago

Benno2000 commented 1 year ago

I implemented @EagleoutIce's solution to fix #30. It works fine, now \hfill can be used in frame titles to align parts of the title to the right.

However, the titles now seem to be a little bit smaller in general, which does not seem like a huge problem to me. What do you think @tthuem?

Bildschirmfoto 2023-08-22 um 10 04 44
EagleoutIce commented 1 year ago

I probably have time to check this in the evening

EagleoutIce commented 1 year ago

This is due to \fitbox. The documentation defines under the second step of the scaling algorithm:

The box is scaled up to the maximal dimensions specified by the user (while keeping the aspect ratio).

In other words, the original title is by default scaled up to the maximum height given. We can change this behavior accordingly, but to be honest i prefer the a little bit smaller titles. What is your opinion @tthuem?

tthuem commented 1 year ago

The slightly smaller titles look fine to me.

@Benno2000 Please also check what happens for other font sizes as this is a parameter of the template. Ideally, you could post screenshots here for different font sizes for documentation.

Benno2000 commented 1 year ago

I tried different font sizes from 8pt up to 12pt. Interenstingly, the size of the frame title does not seem to change between 8pt and 9pt as well as between 10pt and 11pt:

8pt: 8pt

9pt: 9pt

10pt: 10pt

11pt: 11pt

12pt: 12pt

EagleoutIce commented 1 year ago

That is to be expected as font sizes depend on 1) what the font can provide, 2) a predefined table of what size to use for titles etc. See: https://tex.stackexchange.com/questions/24599/what-point-pt-font-size-are-large-etc#24600 for an example of the definitions of the article class.

Benno2000 commented 1 year ago

I see, so I guess since this is a normal behavior we are fine and can merge this? @tthuem

tthuem commented 1 year ago

Great, please merge then.