From de51bc3f41752634c0ccce8484d0a9bade62a45a Mon Sep 17 00:00:00 2001 From: Night_Hunter Date: Fri, 6 Jan 2023 14:55:09 +1300 Subject: Introduce left and right colors for slider rails --- native/src/widget/slider.rs | 92 ++++++++++++++++++++++++++------------------- 1 file changed, 54 insertions(+), 38 deletions(-) (limited to 'native/src/widget/slider.rs') diff --git a/native/src/widget/slider.rs b/native/src/widget/slider.rs index d3715b1c..b3f3306c 100644 --- a/native/src/widget/slider.rs +++ b/native/src/widget/slider.rs @@ -8,13 +8,15 @@ use crate::renderer; use crate::touch; use crate::widget::tree::{self, Tree}; use crate::{ - Background, Clipboard, Color, Element, Layout, Length, Pixels, Point, - Rectangle, Shell, Size, Widget, + Clipboard, Element, Layout, Length, Pixels, Point, Rectangle, Shell, Size, + Widget, }; use std::ops::RangeInclusive; -pub use iced_style::slider::{Appearance, Handle, HandleShape, StyleSheet}; +pub use iced_style::slider::{ + Appearance, Handle, HandleShape, Rail, StyleSheet, +}; /// An horizontal bar and a handle that selects a single value from a range of /// values. @@ -368,37 +370,7 @@ pub fn draw( style_sheet.active(style) }; - let rail_y = bounds.y + (bounds.height / 2.0).round(); - - renderer.fill_quad( - renderer::Quad { - bounds: Rectangle { - x: bounds.x, - y: rail_y - 1.0, - width: bounds.width, - height: 2.0, - }, - border_radius: 0.0.into(), - border_width: 0.0, - border_color: Color::TRANSPARENT, - }, - style.rail_colors.0, - ); - - renderer.fill_quad( - renderer::Quad { - bounds: Rectangle { - x: bounds.x, - y: rail_y + 1.0, - width: bounds.width, - height: 2.0, - }, - border_radius: 0.0.into(), - border_width: 0.0, - border_color: Color::TRANSPARENT, - }, - Background::Color(style.rail_colors.1), - ); + let value = value.into() as f32; let (handle_width, handle_height, handle_border_radius) = match style .handle @@ -411,25 +383,69 @@ pub fn draw( } => (f32::from(width), bounds.height, border_radius), }; - let value = value.into() as f32; let (range_start, range_end) = { let (start, end) = range.clone().into_inner(); (start.into() as f32, end.into() as f32) }; - let handle_offset = if range_start >= range_end { + let offset = if range_start >= range_end { 0.0 } else { (bounds.width - handle_width) * (value - range_start) / (range_end - range_start) }; + let line_y = bounds.y + bounds.height / 2.0 - style.rail.size / 2.0; + let line_offset = offset + handle_width / 2.0; + + renderer.fill_quad( + renderer::Quad { + bounds: Rectangle { + x: bounds.x, + y: line_y, + width: line_offset, + height: style.rail.size, + }, + border_radius: [ + style.rail.border_radius, + 0.0, + 0.0, + style.rail.border_radius, + ] + .into(), + border_width: style.rail.border_width, + border_color: style.rail.border_color, + }, + style.rail.colors.0, + ); + + renderer.fill_quad( + renderer::Quad { + bounds: Rectangle { + x: bounds.x + line_offset.round(), + y: line_y, + width: bounds.width - line_offset, + height: style.rail.size, + }, + border_radius: [ + 0.0, + style.rail.border_radius, + style.rail.border_radius, + 0.0, + ] + .into(), + border_width: style.rail.border_width, + border_color: style.rail.border_color, + }, + style.rail.colors.1, + ); + renderer.fill_quad( renderer::Quad { bounds: Rectangle { - x: bounds.x + handle_offset.round(), - y: rail_y - handle_height / 2.0, + x: bounds.x + offset.round(), + y: bounds.y + bounds.height / 2.0 - handle_height / 2.0, width: handle_width, height: handle_height, }, -- cgit From 45cfce3f6dcf2773bc8ccc7e356906cb778f2f27 Mon Sep 17 00:00:00 2001 From: Héctor Ramón Jiménez Date: Wed, 12 Apr 2023 05:19:01 +0200 Subject: Simplify `draw` logic of sliders --- native/src/widget/slider.rs | 18 ++++++++---------- 1 file changed, 8 insertions(+), 10 deletions(-) (limited to 'native/src/widget/slider.rs') diff --git a/native/src/widget/slider.rs b/native/src/widget/slider.rs index b3f3306c..2f946a8a 100644 --- a/native/src/widget/slider.rs +++ b/native/src/widget/slider.rs @@ -370,8 +370,6 @@ pub fn draw( style_sheet.active(style) }; - let value = value.into() as f32; - let (handle_width, handle_height, handle_border_radius) = match style .handle .shape @@ -383,6 +381,7 @@ pub fn draw( } => (f32::from(width), bounds.height, border_radius), }; + let value = value.into() as f32; let (range_start, range_end) = { let (start, end) = range.clone().into_inner(); @@ -396,15 +395,14 @@ pub fn draw( / (range_end - range_start) }; - let line_y = bounds.y + bounds.height / 2.0 - style.rail.size / 2.0; - let line_offset = offset + handle_width / 2.0; + let rail_y = bounds.y + bounds.height / 2.0; renderer.fill_quad( renderer::Quad { bounds: Rectangle { x: bounds.x, - y: line_y, - width: line_offset, + y: rail_y - style.rail.size / 2.0, + width: offset, height: style.rail.size, }, border_radius: [ @@ -423,9 +421,9 @@ pub fn draw( renderer.fill_quad( renderer::Quad { bounds: Rectangle { - x: bounds.x + line_offset.round(), - y: line_y, - width: bounds.width - line_offset, + x: bounds.x + offset, + y: rail_y - style.rail.size / 2.0, + width: bounds.width - offset, height: style.rail.size, }, border_radius: [ @@ -445,7 +443,7 @@ pub fn draw( renderer::Quad { bounds: Rectangle { x: bounds.x + offset.round(), - y: bounds.y + bounds.height / 2.0 - handle_height / 2.0, + y: rail_y - handle_height / 2.0, width: handle_width, height: handle_height, }, -- cgit From c2cc9a835d8281c1d26c861ecac7991a924c3785 Mon Sep 17 00:00:00 2001 From: Héctor Ramón Jiménez Date: Wed, 12 Apr 2023 05:21:46 +0200 Subject: Remove `border_width` support in `slider::Rail` --- native/src/widget/slider.rs | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) (limited to 'native/src/widget/slider.rs') diff --git a/native/src/widget/slider.rs b/native/src/widget/slider.rs index 2f946a8a..5b26ae01 100644 --- a/native/src/widget/slider.rs +++ b/native/src/widget/slider.rs @@ -8,8 +8,8 @@ use crate::renderer; use crate::touch; use crate::widget::tree::{self, Tree}; use crate::{ - Clipboard, Element, Layout, Length, Pixels, Point, Rectangle, Shell, Size, - Widget, + Clipboard, Color, Element, Layout, Length, Pixels, Point, Rectangle, Shell, + Size, Widget, }; use std::ops::RangeInclusive; @@ -412,8 +412,8 @@ pub fn draw( style.rail.border_radius, ] .into(), - border_width: style.rail.border_width, - border_color: style.rail.border_color, + border_width: 0.0, + border_color: Color::TRANSPARENT, }, style.rail.colors.0, ); @@ -433,8 +433,8 @@ pub fn draw( 0.0, ] .into(), - border_width: style.rail.border_width, - border_color: style.rail.border_color, + border_width: 0.0, + border_color: Color::TRANSPARENT, }, style.rail.colors.1, ); -- cgit From 9b39a17628e92b66ac3649e879478ed23635fa76 Mon Sep 17 00:00:00 2001 From: Héctor Ramón Jiménez Date: Wed, 12 Apr 2023 05:27:32 +0200 Subject: Rename `Rail::size` to `width` --- native/src/widget/slider.rs | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'native/src/widget/slider.rs') diff --git a/native/src/widget/slider.rs b/native/src/widget/slider.rs index 5b26ae01..69c06140 100644 --- a/native/src/widget/slider.rs +++ b/native/src/widget/slider.rs @@ -401,9 +401,9 @@ pub fn draw( renderer::Quad { bounds: Rectangle { x: bounds.x, - y: rail_y - style.rail.size / 2.0, + y: rail_y - style.rail.width / 2.0, width: offset, - height: style.rail.size, + height: style.rail.width, }, border_radius: [ style.rail.border_radius, @@ -422,9 +422,9 @@ pub fn draw( renderer::Quad { bounds: Rectangle { x: bounds.x + offset, - y: rail_y - style.rail.size / 2.0, + y: rail_y - style.rail.width / 2.0, width: bounds.width - offset, - height: style.rail.size, + height: style.rail.width, }, border_radius: [ 0.0, -- cgit