make :toggle-option print the new value (#6774)

Co-authored-by: Michael Davis <mcarsondavis@gmail.com>
This commit is contained in:
Aleksey Kuznetsov 2023-04-16 23:44:12 +05:00 committed by GitHub
parent 7607727483
commit 7706ff77eb
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -1770,12 +1770,12 @@ fn toggle_option(
let pointer = format!("/{}", key.replace('.', "/"));
let value = config.pointer_mut(&pointer).ok_or_else(key_error)?;
if let Value::Bool(b) = *value {
*value = Value::Bool(!b);
} else {
let Value::Bool(old_value) = *value else {
anyhow::bail!("Key `{}` is not toggle-able", key)
}
};
let new_value = !old_value;
*value = Value::Bool(new_value);
// This unwrap should never fail because we only replace one boolean value
// with another, maintaining a valid json config
let config = serde_json::from_value(config).unwrap();
@ -1784,6 +1784,8 @@ fn toggle_option(
.config_events
.0
.send(ConfigEvent::Update(config))?;
cx.editor
.set_status(format!("Option `{}` is now set to `{}`", key, new_value));
Ok(())
}