Skip to content

Commit

Permalink
Pass visible bounds to Element::dispatch_event
Browse files Browse the repository at this point in the history
  • Loading branch information
as-cii committed Apr 7, 2022
1 parent ae415ee commit 73f2fd6
Show file tree
Hide file tree
Showing 23 changed files with 37 additions and 4 deletions.
1 change: 1 addition & 0 deletions crates/editor/src/element.rs
Original file line number Diff line number Diff line change
Expand Up @@ -1147,6 +1147,7 @@ impl Element for EditorElement {
&mut self,
event: &Event,
_: RectF,
_: RectF,
layout: &mut LayoutState,
paint: &mut PaintState,
cx: &mut EventContext,
Expand Down
1 change: 1 addition & 0 deletions crates/gpui/examples/text.rs
Original file line number Diff line number Diff line change
Expand Up @@ -104,6 +104,7 @@ impl gpui::Element for TextElement {
&mut self,
_: &gpui::Event,
_: RectF,
_: RectF,
_: &mut Self::LayoutState,
_: &mut Self::PaintState,
_: &mut gpui::EventContext,
Expand Down
10 changes: 9 additions & 1 deletion crates/gpui/src/elements.rs
Original file line number Diff line number Diff line change
Expand Up @@ -74,6 +74,7 @@ pub trait Element {
&mut self,
event: &Event,
bounds: RectF,
visible_bounds: RectF,
layout: &mut Self::LayoutState,
paint: &mut Self::PaintState,
cx: &mut EventContext,
Expand Down Expand Up @@ -169,6 +170,7 @@ pub enum Lifecycle<T: Element> {
element: T,
constraint: SizeConstraint,
bounds: RectF,
visible_bounds: RectF,
layout: T::LayoutState,
paint: T::PaintState,
},
Expand Down Expand Up @@ -222,6 +224,7 @@ impl<T: Element> AnyElement for Lifecycle<T> {
element,
constraint,
bounds,
visible_bounds,
layout,
paint,
}
Expand All @@ -242,6 +245,7 @@ impl<T: Element> AnyElement for Lifecycle<T> {
element,
constraint,
bounds,
visible_bounds,
layout,
paint,
}
Expand All @@ -254,12 +258,13 @@ impl<T: Element> AnyElement for Lifecycle<T> {
if let Lifecycle::PostPaint {
element,
bounds,
visible_bounds,
layout,
paint,
..
} = self
{
element.dispatch_event(event, *bounds, layout, paint, cx)
element.dispatch_event(event, *bounds, *visible_bounds, layout, paint, cx)
} else {
panic!("invalid element lifecycle state");
}
Expand Down Expand Up @@ -288,6 +293,7 @@ impl<T: Element> AnyElement for Lifecycle<T> {
element,
constraint,
bounds,
visible_bounds,
layout,
paint,
} => {
Expand All @@ -299,6 +305,8 @@ impl<T: Element> AnyElement for Lifecycle<T> {
new_map.insert("type".into(), typ);
}
new_map.insert("constraint".into(), constraint.to_json());
new_map.insert("bounds".into(), bounds.to_json());
new_map.insert("visible_bounds".into(), visible_bounds.to_json());
new_map.append(map);
json::Value::Object(new_map)
} else {
Expand Down
3 changes: 2 additions & 1 deletion crates/gpui/src/elements/align.rs
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,8 @@ impl Element for Align {
fn dispatch_event(
&mut self,
event: &Event,
_: pathfinder_geometry::rect::RectF,
_: RectF,
_: RectF,
_: &mut Self::LayoutState,
_: &mut Self::PaintState,
cx: &mut EventContext,
Expand Down
1 change: 1 addition & 0 deletions crates/gpui/src/elements/canvas.rs
Original file line number Diff line number Diff line change
Expand Up @@ -59,6 +59,7 @@ where
&mut self,
_: &crate::Event,
_: RectF,
_: RectF,
_: &mut Self::LayoutState,
_: &mut Self::PaintState,
_: &mut crate::EventContext,
Expand Down
1 change: 1 addition & 0 deletions crates/gpui/src/elements/constrained_box.rs
Original file line number Diff line number Diff line change
Expand Up @@ -81,6 +81,7 @@ impl Element for ConstrainedBox {
&mut self,
event: &Event,
_: RectF,
_: RectF,
_: &mut Self::LayoutState,
_: &mut Self::PaintState,
cx: &mut EventContext,
Expand Down
1 change: 1 addition & 0 deletions crates/gpui/src/elements/container.rs
Original file line number Diff line number Diff line change
Expand Up @@ -247,6 +247,7 @@ impl Element for Container {
&mut self,
event: &Event,
_: RectF,
_: RectF,
_: &mut Self::LayoutState,
_: &mut Self::PaintState,
cx: &mut EventContext,
Expand Down
1 change: 1 addition & 0 deletions crates/gpui/src/elements/empty.rs
Original file line number Diff line number Diff line change
Expand Up @@ -52,6 +52,7 @@ impl Element for Empty {
&mut self,
_: &Event,
_: RectF,
_: RectF,
_: &mut Self::LayoutState,
_: &mut Self::PaintState,
_: &mut EventContext,
Expand Down
2 changes: 2 additions & 0 deletions crates/gpui/src/elements/event_handler.rs
Original file line number Diff line number Diff line change
Expand Up @@ -85,6 +85,8 @@ impl Element for EventHandler {
&mut self,
event: &Event,
bounds: RectF,
_: RectF,
visible_bounds: RectF,
_: &mut Self::LayoutState,
_: &mut Self::PaintState,
cx: &mut EventContext,
Expand Down
1 change: 1 addition & 0 deletions crates/gpui/src/elements/expanded.rs
Original file line number Diff line number Diff line change
Expand Up @@ -66,6 +66,7 @@ impl Element for Expanded {
&mut self,
event: &Event,
_: RectF,
_: RectF,
_: &mut Self::LayoutState,
_: &mut Self::PaintState,
cx: &mut EventContext,
Expand Down
2 changes: 2 additions & 0 deletions crates/gpui/src/elements/flex.rs
Original file line number Diff line number Diff line change
Expand Up @@ -266,6 +266,7 @@ impl Element for Flex {
&mut self,
event: &Event,
bounds: RectF,
_: RectF,
remaining_space: &mut Self::LayoutState,
_: &mut Self::PaintState,
cx: &mut EventContext,
Expand Down Expand Up @@ -391,6 +392,7 @@ impl Element for FlexItem {
&mut self,
event: &Event,
_: RectF,
_: RectF,
_: &mut Self::LayoutState,
_: &mut Self::PaintState,
cx: &mut EventContext,
Expand Down
1 change: 1 addition & 0 deletions crates/gpui/src/elements/hook.rs
Original file line number Diff line number Diff line change
Expand Up @@ -57,6 +57,7 @@ impl Element for Hook {
&mut self,
event: &Event,
_: RectF,
_: RectF,
_: &mut Self::LayoutState,
_: &mut Self::PaintState,
cx: &mut EventContext,
Expand Down
1 change: 1 addition & 0 deletions crates/gpui/src/elements/image.rs
Original file line number Diff line number Diff line change
Expand Up @@ -81,6 +81,7 @@ impl Element for Image {
&mut self,
_: &Event,
_: RectF,
_: RectF,
_: &mut Self::LayoutState,
_: &mut Self::PaintState,
_: &mut EventContext,
Expand Down
1 change: 1 addition & 0 deletions crates/gpui/src/elements/label.rs
Original file line number Diff line number Diff line change
Expand Up @@ -166,6 +166,7 @@ impl Element for Label {
&mut self,
_: &Event,
_: RectF,
_: RectF,
_: &mut Self::LayoutState,
_: &mut Self::PaintState,
_: &mut EventContext,
Expand Down
2 changes: 2 additions & 0 deletions crates/gpui/src/elements/list.rs
Original file line number Diff line number Diff line change
Expand Up @@ -253,6 +253,7 @@ impl Element for List {
&mut self,
event: &Event,
bounds: RectF,
_: RectF,
scroll_top: &mut ListOffset,
_: &mut (),
cx: &mut EventContext,
Expand Down Expand Up @@ -872,6 +873,7 @@ mod tests {
&mut self,
_: &Event,
_: RectF,
_: RectF,
_: &mut (),
_: &mut (),
_: &mut EventContext,
Expand Down
1 change: 1 addition & 0 deletions crates/gpui/src/elements/mouse_event_handler.rs
Original file line number Diff line number Diff line change
Expand Up @@ -100,6 +100,7 @@ impl Element for MouseEventHandler {
&mut self,
event: &Event,
bounds: RectF,
_: RectF,
_: &mut Self::LayoutState,
_: &mut Self::PaintState,
cx: &mut EventContext,
Expand Down
1 change: 1 addition & 0 deletions crates/gpui/src/elements/overlay.rs
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,7 @@ impl Element for Overlay {
&mut self,
event: &Event,
_: RectF,
_: RectF,
_: &mut Self::LayoutState,
_: &mut Self::PaintState,
cx: &mut EventContext,
Expand Down
1 change: 1 addition & 0 deletions crates/gpui/src/elements/stack.rs
Original file line number Diff line number Diff line change
Expand Up @@ -51,6 +51,7 @@ impl Element for Stack {
&mut self,
event: &Event,
_: RectF,
_: RectF,
_: &mut Self::LayoutState,
_: &mut Self::PaintState,
cx: &mut EventContext,
Expand Down
1 change: 1 addition & 0 deletions crates/gpui/src/elements/svg.rs
Original file line number Diff line number Diff line change
Expand Up @@ -76,6 +76,7 @@ impl Element for Svg {
&mut self,
_: &Event,
_: RectF,
_: RectF,
_: &mut Self::LayoutState,
_: &mut Self::PaintState,
_: &mut EventContext,
Expand Down
1 change: 1 addition & 0 deletions crates/gpui/src/elements/text.rs
Original file line number Diff line number Diff line change
Expand Up @@ -172,6 +172,7 @@ impl Element for Text {
&mut self,
_: &Event,
_: RectF,
_: RectF,
_: &mut Self::LayoutState,
_: &mut Self::PaintState,
_: &mut EventContext,
Expand Down
1 change: 1 addition & 0 deletions crates/gpui/src/elements/uniform_list.rs
Original file line number Diff line number Diff line change
Expand Up @@ -281,6 +281,7 @@ where
&mut self,
event: &Event,
bounds: RectF,
_: RectF,
layout: &mut Self::LayoutState,
_: &mut Self::PaintState,
cx: &mut EventContext,
Expand Down
1 change: 1 addition & 0 deletions crates/gpui/src/presenter.rs
Original file line number Diff line number Diff line change
Expand Up @@ -535,6 +535,7 @@ impl Element for ChildView {
&mut self,
event: &Event,
_: RectF,
_: RectF,
_: &mut Self::LayoutState,
_: &mut Self::PaintState,
cx: &mut EventContext,
Expand Down
5 changes: 3 additions & 2 deletions crates/workspace/src/workspace.rs
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ use gpui::{
action,
color::Color,
elements::*,
geometry::{vector::vec2f, PathBuilder},
geometry::{rect::RectF, vector::vec2f, PathBuilder},
json::{self, to_string_pretty, ToJson},
keymap::Binding,
platform::{CursorStyle, WindowOptions},
Expand Down Expand Up @@ -2068,7 +2068,8 @@ impl Element for AvatarRibbon {
fn dispatch_event(
&mut self,
_: &gpui::Event,
_: gpui::geometry::rect::RectF,
_: RectF,
_: RectF,
_: &mut Self::LayoutState,
_: &mut Self::PaintState,
_: &mut gpui::EventContext,
Expand Down

0 comments on commit 73f2fd6

Please sign in to comment.