default theme to "plain" even if [theme] section is absent

greshake / i3status-rust

Very resourcefriendly and feature-rich replacement for i3status, written in pure Rust

GNU General Public License v3.0

2.81k stars 472 forks source link

default theme to "plain" even if [theme] section is absent #1993

Closed MaxVerevkin closed 5 months ago

MaxVerevkin commented 5 months ago

Also switch from container-level serde(default) to field-level serde(default) in some places to avoid unnecessary file reads and deserialization, because container-level attribute creates a default instance of the entire struct and thus calls default() even for specified fields. In particullar, this change avoids reading plain.toml if theme is set.

Fixes #1992