Default config should come first

[guipatriota]

Hello, Patrik.

Just some suggestion. If you change the order and put:

'
  ' Default font styles
  '
  DefaultFontColor $color_fg
  ' https://github.com/twbs/bootstrap/blob/v4.5.3/scss/_variables.scss#L277
  DefaultFontName -apple-system, BlinkMacSystemFont, "Segoe UI", Roboto, "Helvetica Neue", Arial, "Noto Sans", sans-serif, "Apple Color Emoji", "Segoe UI Emoji", "Segoe UI Symbol", "Noto Color Emoji"
  ' DefaultFontSize
  ' DefaultFontStyle

  ' TODO: get this to work
  ' https://github.com/twbs/bootstrap/blob/v4.5.3/scss/_variables.scss#L278
  ' DefaultMonospacedFontName SFMono-Regular, Menlo, Monaco, Consolas, "Liberation Mono", "Courier New", monospace

  ' DefaultTextAlignment

just after "skinparam {", all defaults will work and all specific changes, like DefaultFontColor to ClassFontColor will work too.

Have a nice day!
Guilherme.

[patrik-csak]

Thanks for the suggestion, @guipatriota

Is this what you mean? : 41dcd67

Do you have an example of something that doesn’t work but is fixed by your suggested change?

[guipatriota]

Hi, Patrik.

That's exactly the change I made (41dcd67). I used your code as a starting point to create a theme in the style of DraculaPro Theme. However, I noticed that in some class and use case diagrams, the fonts, colors, and sizes were not being changed as expected. I even tried using DefaultMonospacedFontName instead of DefaultFontName. It only started working correctly after I placed these lines at the beginning of the skinparam.

In my code, I did:

skinparam {
  '
  ' Default font styles
  '  
  DefaultFontColor $color_pink
  DefaultFontName Fira Mono
  DefaultFontSize 20
...