search

makew0rld / amfora

A fancy terminal browser for the Gemini protocol.
GNU General Public License v3.0
1.14k stars 68 forks source link

Fixed error when including theme. #290

Closed Arkaeriit closed 2 years ago

Arkaeriit commented 2 years ago

To get a new theme for Amfora, I went to the theme page, I downloaded one and I used the include key to import it. I want to use the include key because it feels way more convenient than copy-pasting a theme in the configuration file.

When the include directive was not under the [theme] tag, it was ignored, when the include key was under the [theme] tag, I got the following error:

Config error: invalid color format for "include": <My home folder>/.config/amfora/one_dark.toml

This PR make so that the theme parser ignores the include keys in order to prevent the error.

makew0rld commented 2 years ago

Yikes, nice catch! Thanks for your PR.