Merge pull request #2302 from iced-rs/fix/themer-text-color

Use `application::StyleSheet` colors in `Themer`
This commit is contained in:
Héctor Ramón 2024-02-27 12:01:01 +01:00 committed by GitHub
commit a1de3f1c55
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 40 additions and 6 deletions

View file

@ -15,6 +15,7 @@ use crate::rule::{self, Rule};
use crate::runtime::Command; use crate::runtime::Command;
use crate::scrollable::{self, Scrollable}; use crate::scrollable::{self, Scrollable};
use crate::slider::{self, Slider}; use crate::slider::{self, Slider};
use crate::style::application;
use crate::text::{self, Text}; use crate::text::{self, Text};
use crate::text_editor::{self, TextEditor}; use crate::text_editor::{self, TextEditor};
use crate::text_input::{self, TextInput}; use crate::text_input::{self, TextInput};
@ -445,6 +446,7 @@ pub fn themer<'a, Message, Theme, Renderer>(
) -> Themer<'a, Message, Theme, Renderer> ) -> Themer<'a, Message, Theme, Renderer>
where where
Renderer: core::Renderer, Renderer: core::Renderer,
Theme: application::StyleSheet,
{ {
Themer::new(theme, content) Themer::new(theme, content)
} }

View file

@ -1,3 +1,4 @@
use crate::container;
use crate::core::event::{self, Event}; use crate::core::event::{self, Event};
use crate::core::layout; use crate::core::layout;
use crate::core::mouse; use crate::core::mouse;
@ -6,9 +7,10 @@ use crate::core::renderer;
use crate::core::widget::tree::{self, Tree}; use crate::core::widget::tree::{self, Tree};
use crate::core::widget::Operation; use crate::core::widget::Operation;
use crate::core::{ use crate::core::{
Clipboard, Element, Layout, Length, Point, Rectangle, Shell, Size, Vector, Background, Clipboard, Element, Layout, Length, Point, Rectangle, Shell,
Widget, Size, Vector, Widget,
}; };
use crate::style::application;
/// A widget that applies any `Theme` to its contents. /// A widget that applies any `Theme` to its contents.
/// ///
@ -18,14 +20,18 @@ use crate::core::{
pub struct Themer<'a, Message, Theme, Renderer> pub struct Themer<'a, Message, Theme, Renderer>
where where
Renderer: crate::core::Renderer, Renderer: crate::core::Renderer,
Theme: application::StyleSheet,
{ {
content: Element<'a, Message, Theme, Renderer>, content: Element<'a, Message, Theme, Renderer>,
theme: Theme, theme: Theme,
style: Theme::Style,
show_background: bool,
} }
impl<'a, Message, Theme, Renderer> Themer<'a, Message, Theme, Renderer> impl<'a, Message, Theme, Renderer> Themer<'a, Message, Theme, Renderer>
where where
Renderer: crate::core::Renderer, Renderer: crate::core::Renderer,
Theme: application::StyleSheet,
{ {
/// Creates an empty [`Themer`] that applies the given `Theme` /// Creates an empty [`Themer`] that applies the given `Theme`
/// to the provided `content`. /// to the provided `content`.
@ -34,16 +40,25 @@ where
T: Into<Element<'a, Message, Theme, Renderer>>, T: Into<Element<'a, Message, Theme, Renderer>>,
{ {
Self { Self {
theme,
content: content.into(), content: content.into(),
theme,
style: Theme::Style::default(),
show_background: false,
} }
} }
/// Sets whether to draw the background color of the `Theme`.
pub fn background(mut self, background: bool) -> Self {
self.show_background = background;
self
}
} }
impl<'a, AnyTheme, Message, Theme, Renderer> Widget<Message, AnyTheme, Renderer> impl<'a, AnyTheme, Message, Theme, Renderer> Widget<Message, AnyTheme, Renderer>
for Themer<'a, Message, Theme, Renderer> for Themer<'a, Message, Theme, Renderer>
where where
Renderer: crate::core::Renderer, Renderer: crate::core::Renderer,
Theme: application::StyleSheet,
{ {
fn tag(&self) -> tree::Tag { fn tag(&self) -> tree::Tag {
self.content.as_widget().tag() self.content.as_widget().tag()
@ -120,16 +135,33 @@ where
tree: &Tree, tree: &Tree,
renderer: &mut Renderer, renderer: &mut Renderer,
_theme: &AnyTheme, _theme: &AnyTheme,
renderer_style: &renderer::Style, _style: &renderer::Style,
layout: Layout<'_>, layout: Layout<'_>,
cursor: mouse::Cursor, cursor: mouse::Cursor,
viewport: &Rectangle, viewport: &Rectangle,
) { ) {
let appearance = self.theme.appearance(&self.style);
if self.show_background {
container::draw_background(
renderer,
&container::Appearance {
background: Some(Background::Color(
appearance.background_color,
)),
..container::Appearance::default()
},
layout.bounds(),
);
}
self.content.as_widget().draw( self.content.as_widget().draw(
tree, tree,
renderer, renderer,
&self.theme, &self.theme,
renderer_style, &renderer::Style {
text_color: appearance.text_color,
},
layout, layout,
cursor, cursor,
viewport, viewport,
@ -248,7 +280,7 @@ impl<'a, AnyTheme, Message, Theme, Renderer>
for Element<'a, Message, AnyTheme, Renderer> for Element<'a, Message, AnyTheme, Renderer>
where where
Message: 'a, Message: 'a,
Theme: 'a, Theme: 'a + application::StyleSheet,
Renderer: 'a + crate::core::Renderer, Renderer: 'a + crate::core::Renderer,
{ {
fn from( fn from(