migrations/egui
1
0
Fork 0
mirror of https://github.com/emilk/egui.git synced 2026-06-26 14:49:06 -04:00
Code Activity

Make Slider::new take impl Into<RangeInclusive (#8260)

Browse Source 
That makes it more ergonomic in some cases, e.g. allows you to use
`Rangef`
This commit is contained in:
Emil Ernerfeldt
2026-06-24 21:03:17 +02:00
committed by GitHub
parent 7c600b3c76
commit e8d96525f4
2 changed files with 6 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
crates/egui/src/style.rs
View File

@@ -2963,7 +2963,7 @@ pub fn font_tweak_ui(ui: &mut Ui, tweak: &mut FontTweak, axes: &[FontVariationAx
let mut value = existing.map_or(axis.default, |i| coords.as_ref()[i].1); let mut value = existing.map_or(axis.default, |i| coords.as_ref()[i].1);
ui.horizontal(|ui| { ui.horizontal(|ui| {
if ui.add(Slider::new(&mut value, axis.range.into())).changed() { if ui.add(Slider::new(&mut value, axis.range)).changed() {
match existing { match existing {
Some(i) => coords.as_mut()[i].1 = value, Some(i) => coords.as_mut()[i].1 = value,
None => coords.push(axis.tag, value), None => coords.push(axis.tag, value),

6
crates/egui/src/widgets/slider.rs
View File

@@ -125,7 +125,11 @@ impl<'a> Slider<'a> {
/// ///
/// The `value` given will be clamped to the `range`, /// The `value` given will be clamped to the `range`,
/// unless you change this behavior with [`Self::clamping`]. /// unless you change this behavior with [`Self::clamping`].
pub fn new<Num: emath::Numeric>(value: &'a mut Num, range: RangeInclusive<Num>) -> Self { pub fn new<Num: emath::Numeric>(
value: &'a mut Num,
range: impl Into<RangeInclusive<Num>>,
) -> Self {
let range = range.into();
let range_f64 = range.start().to_f64()..=range.end().to_f64(); let range_f64 = range.start().to_f64()..=range.end().to_f64();
let slf = Self::from_get_set(range_f64, move |v: Option<f64>| { let slf = Self::from_get_set(range_f64, move |v: Option<f64>| {
if let Some(v) = v { if let Some(v) = v {