mfontanini / presenterm

A markdown terminal slideshow tool
https://mfontanini.github.io/presenterm/
BSD 2-Clause "Simplified" License
1.35k stars 34 forks source link

presenterm not respecting config.yaml #104

Closed thisismygitrepo closed 10 months ago

thisismygitrepo commented 10 months ago

I'm on a linux machine, kitty terminal, and I added a config.yaml @ ~/.config/presenterm

but presenterm is not respecting it at all, no matter what I put there.


defaults:
  themes: terminal-dark
  footer:
    style: progress_bar

    # Optional!
    character: 🚀

options:
  footer:
    style: progress_bar

    # Optional!
    character: 🚀
mfontanini commented 10 months ago

What version of presenterm are you using?

thisismygitrepo commented 10 months ago

The latest, 0.4.1 I just did more experiments, it seems to complain if the config file is corrupt, e.g. it says invalid configuration: unknown fieldfooter, expected one ofdefaults,typst,optionsat line 17 column 1

But when I fix it and put something like the config you see at the begining of this thread, it doesn't complain anymore but it doesn't respect the config at all.

mfontanini commented 10 months ago

The only fields you can use in your config file are these ones so yes, those complaints are real. I did make a mistake on the docs apparently, the key you want is defaults.theme, not defaults.themes, e.g.

defaults:
  theme: terminal-dark

I'll fix that in a bit. Looks like this is missing a lot of validations for unknown fields, pretty much everything in your example config file is unsupported and it should complain.