migrations/egui
1
0
Fork 0
mirror of https://github.com/emilk/egui.git synced 2026-09-02 06:40:06 -04:00
Code Activity

Properly handle pixels_per_point in kittest

Browse Source
This commit is contained in:
Lucas Meurer
2026-07-21 11:33:50 +02:00
parent 67d0ed276d
commit 726a496eb5
2 changed files with 10 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
crates/egui_kittest/src/lib.rs
View File

@@ -735,7 +735,7 @@ impl<'a, State> Harness<'a, State> {
/// The root node of the test harness. /// The root node of the test harness.
pub fn root(&self) -> Node<'_> { pub fn root(&self) -> Node<'_> {
Node::new(self.kittest.root(), &self.queued_events) Node::new(self.kittest.root(), &self.queued_events, self.ctx.pixels_per_point())
} }
/// Spawn a real native eframe window running this harness's app, reusing its [`egui::Context`]. /// Spawn a real native eframe window running this harness's app, reusing its [`egui::Context`].

13
crates/egui_kittest/src/node.rs
View File

@@ -19,6 +19,7 @@ pub type EventQueue = Mutex<Vec<EventType>>;
pub struct Node<'tree> { pub struct Node<'tree> {
pub(crate) accesskit_node: AccessKitNode<'tree>, pub(crate) accesskit_node: AccessKitNode<'tree>,
pub(crate) queue: &'tree EventQueue, pub(crate) queue: &'tree EventQueue,
pub(crate) pixels_per_point: f32,
} }
impl Debug for Node<'_> { impl Debug for Node<'_> {
@@ -33,16 +34,17 @@ impl<'tree> NodeT<'tree> for Node<'tree> {
} }
fn new_related(&self, child_node: AccessKitNode<'tree>) -> Self { fn new_related(&self, child_node: AccessKitNode<'tree>) -> Self {
Self::new(child_node, self.queue) Self::new(child_node, self.queue, self.pixels_per_point)
} }
} }
impl<'tree> Node<'tree> { impl<'tree> Node<'tree> {
/// Construct a new accesskit node /// Construct a new accesskit node
pub fn new(accesskit_node: AccessKitNode<'tree>, queue: &'tree EventQueue) -> Self { pub fn new(accesskit_node: AccessKitNode<'tree>, queue: &'tree EventQueue, pixels_per_point: f32) -> Self {
Self { Self {
queue, queue,
accesskit_node, accesskit_node,
pixels_per_point,
} }
} }
@@ -113,14 +115,17 @@ impl<'tree> Node<'tree> {
)); ));
} }
/// This returns the rect in logical ui coordinates while the underlying [`accesskit::Node`] has it
/// in physical screen coordinates.
pub fn rect(&self) -> egui::Rect { pub fn rect(&self) -> egui::Rect {
let rect = self let rect = self
.accesskit_node .accesskit_node
.bounding_box() .bounding_box()
.expect("Every egui node should have a rect"); .expect("Every egui node should have a rect");
let ppp = self.pixels_per_point;
egui::Rect { egui::Rect {
min: Pos2::new(rect.x0 as f32, rect.y0 as f32), min: Pos2::new(rect.x0 as f32 / ppp, rect.y0 as f32 / ppp),
max: Pos2::new(rect.x1 as f32, rect.y1 as f32), max: Pos2::new(rect.x1 as f32 / ppp, rect.y1 as f32 / ppp),
} }
} }