Fix VisibilityWrapper widget

I believe this fixes how the widget works, though it's hard to tell
since the behavior was inconsistent.

There are still issues related to drag-and-drop in cosmic-comp or
`iced_sctk`, which need to be fixed to get things working fully.
This commit is contained in:
Ian Douglas Scott 2024-04-22 10:16:41 -07:00
parent 6f45933971
commit 8365efb30d

View file

@ -32,37 +32,23 @@ pub struct VisibilityWrapper<'a, Msg> {
impl<'a, Msg> Widget<Msg, cosmic::Theme, cosmic::Renderer> for VisibilityWrapper<'a, Msg> { impl<'a, Msg> Widget<Msg, cosmic::Theme, cosmic::Renderer> for VisibilityWrapper<'a, Msg> {
delegate::delegate! { delegate::delegate! {
to self.content.as_widget() { to self.content.as_widget() {
fn tag(&self) -> tree::Tag;
fn state(&self) -> tree::State;
fn children(&self) -> Vec<Tree>;
fn size(&self) -> Size<Length>; fn size(&self) -> Size<Length>;
fn size_hint(&self) -> Size<Length>; fn size_hint(&self) -> Size<Length>;
fn layout( }
&self, }
tree: &mut Tree,
renderer: &cosmic::Renderer,
limits: &layout::Limits,
) -> layout::Node;
fn operate( fn operate(
&self, &self,
tree: &mut Tree, tree: &mut Tree,
layout: Layout<'_>, layout: Layout<'_>,
renderer: &cosmic::Renderer, renderer: &cosmic::Renderer,
operation: &mut dyn Operation<OperationOutputWrapper<Msg>>, operation: &mut dyn Operation<OperationOutputWrapper<Msg>>,
); ) {
fn mouse_interaction( self.content
&self, .as_widget()
_tree: &Tree, .operate(&mut tree.children[0], layout, renderer, operation);
_layout: Layout<'_>,
_cursor: mouse::Cursor,
_viewport: &Rectangle,
_renderer: &cosmic::Renderer,
) -> mouse::Interaction;
fn id(&self) -> Option<Id>;
} }
to self.content.as_widget_mut() {
fn diff(&mut self, tree: &mut Tree);
fn on_event( fn on_event(
&mut self, &mut self,
tree: &mut Tree, tree: &mut Tree,
@ -73,20 +59,50 @@ impl<'a, Msg> Widget<Msg, cosmic::Theme, cosmic::Renderer> for VisibilityWrapper
clipboard: &mut dyn Clipboard, clipboard: &mut dyn Clipboard,
shell: &mut Shell<'_, Msg>, shell: &mut Shell<'_, Msg>,
viewport: &Rectangle, viewport: &Rectangle,
) -> event::Status; ) -> event::Status {
fn overlay<'b>( self.content.as_widget_mut().on_event(
&'b mut self, &mut tree.children[0],
tree: &'b mut Tree, event,
layout: Layout<'_>, layout,
renderer: &cosmic::Renderer, cursor,
) -> Option<overlay::Element<'b, Msg, cosmic::Theme, cosmic::Renderer>>; renderer,
fn set_id(&mut self, id: Id); clipboard,
shell,
viewport,
)
} }
fn mouse_interaction(
&self,
tree: &Tree,
layout: Layout<'_>,
cursor: mouse::Cursor,
viewport: &Rectangle,
renderer: &cosmic::Renderer,
) -> mouse::Interaction {
self.content.as_widget().mouse_interaction(
&tree.children[0],
layout,
cursor,
viewport,
renderer,
)
}
fn layout(
&self,
tree: &mut Tree,
renderer: &cosmic::Renderer,
limits: &layout::Limits,
) -> layout::Node {
self.content
.as_widget()
.layout(&mut tree.children[0], renderer, limits)
} }
fn draw( fn draw(
&self, &self,
state: &Tree, tree: &Tree,
renderer: &mut cosmic::Renderer, renderer: &mut cosmic::Renderer,
theme: &cosmic::Theme, theme: &cosmic::Theme,
style: &renderer::Style, style: &renderer::Style,
@ -95,11 +111,25 @@ impl<'a, Msg> Widget<Msg, cosmic::Theme, cosmic::Renderer> for VisibilityWrapper
viewport: &Rectangle, viewport: &Rectangle,
) { ) {
if self.visible { if self.visible {
self.content self.content.as_widget().draw(
.as_widget() &tree.children[0],
.draw(state, renderer, theme, style, layout, cursor, viewport); renderer,
theme,
style,
layout,
cursor,
viewport,
);
} }
} }
fn children(&self) -> Vec<Tree> {
vec![Tree::new(&self.content)]
}
fn diff(&mut self, tree: &mut Tree) {
tree.diff_children(&mut [&mut self.content]);
}
} }
impl<'a, Msg: 'a> From<VisibilityWrapper<'a, Msg>> for cosmic::Element<'a, Msg> { impl<'a, Msg: 'a> From<VisibilityWrapper<'a, Msg>> for cosmic::Element<'a, Msg> {