Skip to content

Commit

Permalink
app/settings: Don't use strip_option (#137)
Browse files Browse the repository at this point in the history
Since these don't default to `None`, and the fields aren't public, using
`strip_option` makes it impossible to change them to `None`.
  • Loading branch information
ids1024 authored Aug 3, 2023
1 parent a223b60 commit 54d47a1
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions src/app/settings.rs
Original file line number Diff line number Diff line change
Expand Up @@ -26,15 +26,14 @@ pub struct Settings {
pub(crate) default_font: Font,

/// Name of the icon theme to search by default.
#[setters(strip_option, into)]
#[setters(into)]
pub(crate) default_icon_theme: Option<String>,

/// Default size of fonts.
pub(crate) default_text_size: f32,

/// Whether the window should be resizable or not.
/// and the size of the window border which can be dragged for a resize
#[setters(strip_option)]
pub(crate) resizable: Option<f64>,

/// Scale factor to use by default.
Expand Down

0 comments on commit 54d47a1

Please sign in to comment.