Cargo.toml

@@ -7,26 +7,26 @@ default-run = "stiletto"
 
 [dependencies]
 log = "0.4"
-druid = { version = "0.7.0", features = ["im", "svg"] }
+druid = { version = "0.7", features = ["im", "svg"] }
 im = { version = "*" }
 serde = { version = "1.0", features = ["derive"] }
 serde_bare = "0.3.0"
 serde_json = "1.0"
-clap = "3.0.0-beta.2"
+clap = "3.1"
 
 [target.'cfg(target_os="linux")'.dependencies.gtk]
-version = "0.9.2"
+version = "0.14"
 features = ["v3_22"]
 
 [target.'cfg(target_os="linux")'.dependencies.gio]
-version = "0.9.1"
+version = "0.14"
 features = ["v2_56"]
 
 [target.'cfg(target_os="linux")'.dependencies.gdk]
-version = "0.13.2"
+version = "0.14"
 features = ["v3_22"]
 
 [patch.crates-io]
-druid = { git = "https://github.com/doppioandante/druid", branch = "v0.7.0_stiletto", features = ["im", "svg"] }
+druid = { git = "https://github.com/doppioandante/druid", branch = "pointer_event_rebase", features = ["im", "svg"] }
-#druid = { path = "../druid/druid/", features = ["im", "svg"] }
+#druid = { path= "../druid/druid/" }
 

src/bin/stlt_migrate.rs

@@ -14,25 +14,29 @@
 //  You should have received a copy of the GNU Affero General Public License
 //  along with this program.  If not, see <https://www.gnu.org/licenses/>.
 
-use clap::{Arg, App};
+use clap::{Arg, Command};
 use stiletto::migration::open_stiletto_document;
 
 use std::fs::File;
 use std::path::PathBuf;
 
 fn main() {
-    let matches = App::new("Stiletto Migration CLI")
+    let matches = Command::new("Stiletto Migration CLI")
         .version("0.1.0")
         .author("Stiletto Authors")
         .about("Migrate a stlt file to the latest version")
-        .arg(Arg::new("INPUT")
+        .arg(
-            .about("Sets the input file to use")
+            Arg::new("INPUT")
+                .help("Sets the input file to use")
                 .required(true)
-            .index(1))
+                .index(1),
-        .arg(Arg::new("OUTPUT")
+        )
-            .about("Sets the output file to use")
+        .arg(
+            Arg::new("OUTPUT")
+                .help("Sets the output file to use")
                 .required(true)
-            .index(2))
+                .index(2),
+        )
         .get_matches();
 
     let input_path = matches.value_of("INPUT").unwrap();

src/canvas.rs

@@ -27,12 +27,6 @@ pub struct Path {
     pub kurbo_path: druid::kurbo::BezPath,
 }
 
-#[derive(Serialize, Deserialize)]
-#[serde(remote = "druid::Color")]
-enum ColorDef {
-    Rgba32(u32),
-}
-
 struct Interval {
     start_pos: f64,
     end_pos: f64,
@@ -86,11 +80,29 @@ pub enum CanvasElement {
     Freehand {
         path: Path,
         thickness: f64,
-        #[serde(with = "ColorDef")]
+        #[serde(
+            serialize_with = "serialize_color",
+            deserialize_with = "deserialize_color"
+        )]
         stroke_color: druid::Color,
     },
 }
 
+fn serialize_color<S>(c: &druid::Color, ser: S) -> Result<S::Ok, S::Error>
+where
+    S: Serializer,
+{
+    ser.serialize_u32(c.as_rgba_u32())
+}
+
+fn deserialize_color<'de, D>(de: D) -> Result<druid::Color, D::Error>
+where
+    D: Deserializer<'de>,
+{
+    let c = u32::deserialize(de)?;
+    Ok(druid::Color::from_rgba32_u32(c))
+}
+
 impl CanvasElement {
     pub fn bounding_box(&self) -> druid::Rect {
         match self {
@@ -193,11 +205,16 @@ impl Serialize for Path {
     {
         use druid::kurbo::PathEl;
 
-        serializer.collect_seq(self.kurbo_path.iter().filter_map(|path_el| match path_el {
+        serializer.collect_seq(
+            self.kurbo_path
+                .iter()
+                .filter_map(|path_el| match path_el {
                     PathEl::MoveTo(pt) => Some(Into::<(f64, f64)>::into(pt)),
                     PathEl::LineTo(pt) => Some(Into::<(f64, f64)>::into(pt)),
                     _ => None,
-        }).collect::<Vec<(f64, f64)>>())
+                })
+                .collect::<Vec<(f64, f64)>>(),
+        )
     }
 }
 

src/lib.rs

@@ -19,9 +19,9 @@ use std::vec::Vec;
 
 pub mod canvas;
 pub mod history;
+pub mod migration;
 pub mod tool;
 pub mod widget;
-pub mod migration;
 
 pub mod commands {
     use druid::Selector;

src/main.rs

@@ -26,18 +26,18 @@ use druid::widget::{
 };
 use druid::{
     AppDelegate, AppLauncher, Color, Command, Data, DelegateCtx, Env, FileDialogOptions, FileSpec,
-    Handled, Lens, Target, WindowDesc,
+    Handled, Lens, Target, WidgetId, WindowDesc,
 };
 
 use im::Vector;
 
 use stiletto::tool::CanvasToolParams;
-use stiletto::widget::tool_ctx::{CanvasToolCtx};
+use stiletto::widget::tool_ctx::CanvasToolCtx;
 use stiletto::widget::{build_simple_tool_widget, CanvasState, CanvasToolIconState, CanvasWidget};
 use stiletto::DocumentSnapshot;
 
 pub fn main() {
-    let window = WindowDesc::new(build_ui)
+    let window = WindowDesc::new(build_ui())
         .window_size((1024.0, 1400.0))
         .title(|data: &StilettoState, _env: &Env| {
             let doc_name = if let Some(path) = &data.current_file_path {
@@ -77,7 +77,7 @@ pub fn main() {
         current_file_path: None,
     };
     AppLauncher::with_window(window)
-        .use_simple_logger()
+        .log_to_console()
         .delegate(Delegate)
         .launch(canvas_data)
         .expect("launch failed");
@@ -107,6 +107,8 @@ impl StilettoState {
 }
 
 fn build_ui() -> impl Widget<StilettoState> {
+    let canvas_id = WidgetId::next();
+
     let history_buttons = Flex::row()
         .cross_axis_alignment(CrossAxisAlignment::Center)
         .with_child(Button::new("Undo").on_click(
@@ -125,6 +127,16 @@ fn build_ui() -> impl Widget<StilettoState> {
 
     let save_buttons = Flex::row()
         .cross_axis_alignment(CrossAxisAlignment::Center)
+        .with_child(Button::new("Up").on_click(
+            move |ctx: &mut EventCtx, _data: &mut StilettoState, _env: &Env| {
+                ctx.submit_command(Command::new(CanvasWidget::SCROLL, -30.0, canvas_id));
+            },
+        ))
+        .with_child(Button::new("Down").on_click(
+            move |ctx: &mut EventCtx, _data: &mut StilettoState, _env: &Env| {
+                ctx.submit_command(Command::new(CanvasWidget::SCROLL, 30.0, canvas_id));
+            },
+        ))
         .with_child(
             Button::new("Save").on_click(move |ctx, data: &mut StilettoState, _| {
                 if data.current_file_path.is_some() {
@@ -183,7 +195,12 @@ fn build_ui() -> impl Widget<StilettoState> {
         .cross_axis_alignment(CrossAxisAlignment::Center)
         .must_fill_main_axis(true)
         .with_child(SizedBox::new(Align::left(toolbar)).height(50.0))
-        .with_flex_child((CanvasWidget).lens(StilettoState::canvas), 1.0)
+        .with_flex_child(
+            CanvasWidget::new()
+                .lens(StilettoState::canvas)
+                .with_id(canvas_id),
+            1.0,
+        )
         .controller(ToolSwitcher::new())
 }
 
@@ -259,8 +276,7 @@ impl AppDelegate<StilettoState> for Delegate {
 
             let res_file = File::create(&path_buf);
             if let Ok(f) = res_file {
-                let write_res =
+                let write_res = data.canvas.get_document_snapshot().to_writer(f);
-                    data.canvas.get_document_snapshot().to_writer(f);
                 if write_res.is_err() {
                     warn!("Error while saving: {:?}", write_res.err());
                 } else {

src/migration.rs

@@ -47,7 +47,9 @@ impl StilettoDocument {
 
     pub fn migrate_to_next(self) -> StilettoDocument {
         match self {
-            StilettoDocument::DocumentSnapshot0_1(snapshot) =>  StilettoDocument::DocumentSnapshot0_2(snapshot) ,
+            StilettoDocument::DocumentSnapshot0_1(snapshot) => {
+                StilettoDocument::DocumentSnapshot0_2(snapshot)
+            }
             StilettoDocument::DocumentSnapshot0_2(_) => self,
         }
     }
@@ -60,12 +62,11 @@ impl StilettoDocument {
         assert!(matches!(self, StilettoDocument::DocumentSnapshot0_2(_)));
         match self {
             StilettoDocument::DocumentSnapshot0_2(snapshot) => snapshot,
-            _ => panic!("Wrong Document Snapshot Version")
+            _ => panic!("Wrong Document Snapshot Version"),
         }
     }
 }
 
-
 impl fmt::Display for MigrationError {
     fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
         match &self {
@@ -74,10 +75,11 @@ impl fmt::Display for MigrationError {
             MigrationError::IoError(e) => e.fmt(f),
             MigrationError::UnexpectedVersion(major, minor, e_major, e_minor) => {
                 write!(
-                    f, "Unexpected version ({}, {}): was expecting ({}, {})",
+                    f,
+                    "Unexpected version ({}, {}): was expecting ({}, {})",
                     major, minor, e_major, e_minor
                 )
-            },
+            }
         }
     }
 }
@@ -114,12 +116,12 @@ impl From<serde_bare::Error> for MigrationError {
 fn open_0_1(path: &PathBuf) -> Result<StilettoDocument, MigrationError> {
     let f = File::open(path)?;
     let document_snapshot: DocumentSnapshot = serde_json::from_reader(f)?;
-    if document_snapshot.format_version_major != 0 ||
+    if document_snapshot.format_version_major != 0 || document_snapshot.format_version_minor != 1 {
-        document_snapshot.format_version_minor != 1 {
         Err(MigrationError::UnexpectedVersion(
             document_snapshot.format_version_major,
             document_snapshot.format_version_minor,
-            0, 1
+            0,
+            1,
         ))
     } else {
         Ok(StilettoDocument::DocumentSnapshot0_1(document_snapshot))
@@ -139,4 +141,3 @@ pub fn open_stiletto_document(path: &PathBuf) -> Result<StilettoDocument, Migrat
 
     open_0_2(path)
 }
-

src/tool.rs

@@ -16,7 +16,7 @@
 
 use druid::{Color, Data};
 
-#[derive(Clone, Data)]
+#[derive(Clone, Data, Debug)]
 pub enum CanvasToolParams {
     Pen { thickness: f64, color: Color },
     Eraser,
@@ -27,4 +27,3 @@ pub enum CanvasToolType {
     Pen,
     Eraser,
 }
-

src/widget/canvas.rs

@@ -16,13 +16,16 @@
 
 use im::Vector;
 
-use super::tool_ctx::{CanvasToolCtx};
+use super::tool_ctx::CanvasToolCtx;
 use crate::canvas::Canvas;
 use crate::history::VersionedCanvas;
 use crate::DocumentSnapshot;
 
+use druid::kurbo::Point;
+use druid::scroll_component::ScrollComponent;
 use druid::widget::prelude::*;
-use druid::{Color, Data, Env, Event, PointerType};
+use druid::widget::Viewport;
+use druid::{Affine, Color, Data, Env, Event, PointerType, Selector};
 
 #[derive(Clone, Data)]
 pub struct CanvasState {
@@ -83,32 +86,97 @@ impl CanvasState {
         &mut self.tool_ctx
     }
 
-    pub fn handle_event(&mut self, mut ctx: &mut EventCtx, event: &Event, env: &Env) {
+    pub fn handle_event(
+        &mut self,
+        ctx: &mut EventCtx,
+        event: &Event,
+        transform: Affine,
+        env: &Env,
+    ) {
         self.tool_ctx
-            .handle_event(ctx, event, &mut self.versioned_canvas, env);
+            .handle_event(ctx, event, &mut self.versioned_canvas, transform, env);
     }
 }
 
-pub struct CanvasWidget;
+pub struct CanvasWidget {
+    viewport: Viewport,
+    scroll_component: ScrollComponent,
+    widget_size: Size,
+}
+
+impl CanvasWidget {
+    pub fn new() -> Self {
+        CanvasWidget {
+            viewport: Viewport {
+                content_size: Size::new(0.0, 0.0),
+                view_origin: Point::new(0.0, 0.0),
+                view_size: Size::new(0.0, 0.0),
+            },
+            widget_size: Size::new(0.0, 0.0),
+            scroll_component: ScrollComponent::new(),
+        }
+    }
+
+    fn widget_to_viewport(&self) -> Affine {
+        if !self.viewport.view_size.is_empty() && !self.widget_size.is_empty() {
+            let scale_x = self.viewport.view_size.width / self.widget_size.width;
+            let scale_y = self.viewport.view_size.height / self.widget_size.height;
+            Affine::translate(self.viewport.view_origin.to_vec2())
+                * Affine::scale_non_uniform(scale_x, scale_y)
+        } else {
+            Affine::scale(1.0) // identity
+        }
+    }
+
+    pub const SCROLL: Selector<f64> = Selector::new("scroll_canvas");
+}
 
 impl Widget<CanvasState> for CanvasWidget {
     fn event(&mut self, ctx: &mut EventCtx, event: &Event, data: &mut CanvasState, env: &Env) {
         ctx.request_focus();
+
+        self.scroll_component
+            .event(&mut self.viewport, ctx, event, env);
+        if !ctx.is_handled() {
+            let mut viewport_transform: Option<Affine> = None;
             let mut toggle_eraser_event = false;
             let mut enable_temporary_erasing = false;
             match event {
-            Event::MouseDown(mouse_event) => {
+                Event::Command(cmd) => {
-                toggle_eraser_event = true;
+                    if let Some(value) = cmd.get(CanvasWidget::SCROLL) {
-                enable_temporary_erasing = mouse_event.pointer_type == PointerType::Eraser;
+                        viewport_transform = Some(Affine::translate((0.0, *value)));
                     }
-            Event::MouseMove(mouse_event) => {
-                toggle_eraser_event = true;
-                enable_temporary_erasing = mouse_event.pointer_type == PointerType::Eraser;
                 }
-            Event::MouseUp(mouse_event) => {
+                Event::KeyDown(key_event) => {
-                toggle_eraser_event = true;
+                    if key_event.code == druid::Code::ArrowDown {
-                enable_temporary_erasing = mouse_event.pointer_type == PointerType::Eraser;
+                        viewport_transform = Some(Affine::translate((0.0, 30.0)));
+                    } else if key_event.code == druid::Code::ArrowUp {
+                        viewport_transform = Some(Affine::translate((0.0, -30.0)));
+                    } else if key_event.code == druid::Code::ArrowRight {
+                        viewport_transform = Some(Affine::translate((30.0, 0.0)));
+                    } else if key_event.code == druid::Code::ArrowLeft {
+                        viewport_transform = Some(Affine::translate((-30.0, 0.0)));
                     }
+                }
+                Event::MouseDown(pointer_event) => {
+                    toggle_eraser_event = true;
+                    enable_temporary_erasing = pointer_event.pointer_type == PointerType::Eraser;
+                }
+                Event::MouseMove(pointer_event) => {
+                    toggle_eraser_event = true;
+                    enable_temporary_erasing = pointer_event.pointer_type == PointerType::Eraser;
+                }
+                Event::MouseUp(pointer_event) => {
+                    toggle_eraser_event = true;
+                    enable_temporary_erasing = pointer_event.pointer_type == PointerType::Eraser;
+                }
+                Event::Wheel(pointer) => {
+                    let transform = self.widget_to_viewport();
+                    viewport_transform = Some(Affine::translate(
+                        (pointer.wheel_delta.to_point()).to_vec2(),
+                    ));
+                }
+
                 _ => {}
             }
             // TODO: the first eraser toggle is not handled
@@ -120,17 +188,35 @@ impl Widget<CanvasState> for CanvasWidget {
                 }
                 data.temporary_erasing = enable_temporary_erasing;
             } else {
-            data.handle_event(ctx, event, env);
+                data.handle_event(ctx, event, self.widget_to_viewport(), env);
+            }
+            if let Some(transform) = viewport_transform {
+                let mut new_rect = transform.transform_rect_bbox(self.viewport.view_rect());
+                if new_rect.x0 <= 0f64 {
+                    new_rect.x1 -= new_rect.x0;
+                    new_rect.x0 = 0f64;
+                }
+                if new_rect.y0 <= 0f64 {
+                    new_rect.y1 -= new_rect.y0;
+                    new_rect.y0 = 0f64;
+                }
+                self.viewport.view_origin = new_rect.origin();
+                self.scroll_component
+                    .reset_scrollbar_fade(|d| ctx.request_timer(d), env);
+                ctx.request_paint();
+                ctx.set_handled();
+            }
         }
     }
 
     fn lifecycle(
         &mut self,
-        _ctx: &mut LifeCycleCtx,
+        ctx: &mut LifeCycleCtx,
-        _event: &LifeCycle,
+        event: &LifeCycle,
         _data: &CanvasState,
-        _env: &Env,
+        env: &Env,
     ) {
+        self.scroll_component.lifecycle(ctx, event, env);
     }
 
     fn update(
@@ -149,6 +235,7 @@ impl Widget<CanvasState> for CanvasWidget {
         //        ctx.request_paint_rect(e.bounding_box());
         //    }
         //} else {
+
         ctx.request_paint();
         //}
     }
@@ -177,12 +264,37 @@ impl Widget<CanvasState> for CanvasWidget {
         let size = ctx.size();
         let rect = size.to_rect();
 
+        ctx.clip(rect);
         ctx.fill(rect, &Color::WHITE);
+
+        let page_content_size = data.versioned_canvas.get().content_size();
+        let transform = self.widget_to_viewport();
+        //self.viewport.rect =
+        //    transform.transform_rect_bbox(rect)
+        //        .with_origin(self.viewport.rect.origin());
+        self.viewport.view_size = transform.transform_rect_bbox(rect).size();
+
+        self.viewport.content_size = druid::Size::new(
+            self.viewport.view_rect().x1.max(page_content_size.width),
+            self.viewport.view_rect().y1.max(page_content_size.height),
+        );
+        //log::debug!("{:#?}", &self.viewport);
+
+        ctx.save().unwrap();
+        ctx.transform(transform.inverse());
         for element in data.versioned_canvas.get().elements().iter() {
             element.draw(ctx);
         }
+        ctx.restore().unwrap();
+
+        self.scroll_component.draw_bars(ctx, &self.viewport, env);
+
+        // tools take care of the transform themselves
         if data.tool_ctx.needs_repaint() {
             data.tool_ctx.paint(ctx, env);
         }
+
+        // update current widget size
+        self.widget_size = size;
     }
 }

src/widget/mod.rs

@@ -1,6 +1,6 @@
 pub mod canvas;
-pub mod tool_icon;
 pub mod tool_ctx;
+pub mod tool_icon;
 
 pub use canvas::*;
 pub use tool_icon::*;

src/widget/tool_ctx.rs

@@ -14,14 +14,14 @@
 //  You should have received a copy of the GNU Affero General Public License
 //  along with this program.  If not, see <https://www.gnu.org/licenses/>.
 
-use crate::tool::{CanvasToolParams, CanvasToolType};
 use crate::canvas::{Canvas, CanvasElement};
 use crate::history::VersionedCanvas;
+use crate::tool::{CanvasToolParams, CanvasToolType};
 
 use druid::kurbo::BezPath;
-use druid::{Data, Env, Event, EventCtx, MouseButton, MouseEvent, PaintCtx};
+use druid::{Affine, Data, Env, Event, EventCtx, PaintCtx, PointerButton, PointerEvent};
 
-#[derive(Clone, Data)]
+#[derive(Clone, Data, Debug)]
 pub enum CanvasToolState {
     Idle,
     DrawingFreehand {
@@ -46,9 +46,14 @@ impl CanvasToolParams {
     }
 }
 
-fn pressed(mouse_event: &MouseEvent) -> bool {
+fn pressed(pointer_event: &PointerEvent) -> bool {
-    mouse_event.buttons.contains(MouseButton::Left)
+    dbg!(&pointer_event);
-        || mouse_event.button == MouseButton::Left
+    dbg!(
+        pointer_event.buttons.contains(PointerButton::Left)
+            || pointer_event.button == PointerButton::Left
+    );
+    pointer_event.buttons.contains(PointerButton::Left)
+        || pointer_event.button == PointerButton::Left
 }
 
 impl CanvasToolCtx {
@@ -61,30 +66,34 @@ impl CanvasToolCtx {
 
     pub fn handle_event(
         &mut self,
-        ctx: &EventCtx,
+        ctx: &mut EventCtx,
         event: &Event,
-        mut vcanvas: &mut VersionedCanvas,
+        vcanvas: &mut VersionedCanvas,
+        transform: Affine,
         env: &Env,
     ) {
         match self.initial_params.tool_type() {
-            CanvasToolType::Pen => self.handle_pen_event(&ctx, &event, &mut vcanvas, &env),
+            CanvasToolType::Pen => self.handle_pen_event(ctx, event, vcanvas, transform, env),
-            CanvasToolType::Eraser => self.handle_erase_event(&ctx, &event, &mut vcanvas, &env),
+            CanvasToolType::Eraser => self.handle_erase_event(ctx, event, vcanvas, transform, env),
         }
     }
 
     pub fn handle_erase_event(
         &mut self,
-        _ctx: &EventCtx,
+        ctx: &mut EventCtx,
         event: &Event,
         vcanvas: &mut VersionedCanvas,
+        transform: Affine,
         _env: &Env,
     ) {
         match (&mut self.state, event) {
-            (CanvasToolState::Idle, Event::MouseDown(mouse_event)) if pressed(mouse_event) => {
+            (CanvasToolState::Idle, Event::MouseDown(pointer_event)) if pressed(pointer_event) => {
                 self.state = CanvasToolState::Erasing;
+                ctx.set_handled();
             }
-            (CanvasToolState::Erasing, Event::MouseMove(mouse_event)) if pressed(mouse_event) => {
+            (CanvasToolState::Erasing, Event::MouseMove(pointer_event)) => {
-                let eraser_rect = druid::Rect::from_center_size(mouse_event.pos, (5.0, 5.0));
+                let eraser_rect =
+                    druid::Rect::from_center_size(transform * pointer_event.pos, (5.0, 5.0));
                 let old_elements = vcanvas.get().elements();
                 let mut new_elements = old_elements.clone();
                 new_elements.retain(|elem| {
@@ -101,9 +110,11 @@ impl CanvasToolCtx {
                         *canvas = Canvas::new_with_elements(new_elements);
                     });
                 }
+                ctx.set_handled();
             }
-            (CanvasToolState::Erasing, Event::MouseUp(mouse_event)) if pressed(mouse_event) => {
+            (CanvasToolState::Erasing, Event::MouseUp(pointer_event)) if pressed(pointer_event) => {
                 self.state = CanvasToolState::Idle;
+                ctx.set_handled();
             }
             _ => {}
         }
@@ -111,15 +122,16 @@ impl CanvasToolCtx {
 
     pub fn handle_pen_event(
         &mut self,
-        _ctx: &EventCtx,
+        ctx: &mut EventCtx,
         event: &Event,
         vcanvas: &mut VersionedCanvas,
+        transform: Affine,
         _env: &Env,
     ) {
         match (&mut self.state, event) {
-            (CanvasToolState::Idle, Event::MouseDown(mouse_event)) if pressed(mouse_event) => {
+            (CanvasToolState::Idle, Event::MouseDown(pointer_event)) if pressed(pointer_event) => {
                 let mut kurbo_path = BezPath::new();
-                kurbo_path.move_to((mouse_event.pos.x, mouse_event.pos.y));
+                kurbo_path.move_to((pointer_event.pos.x, pointer_event.pos.y));
                 if let CanvasToolParams::Pen { thickness, color } = &self.initial_params {
                     self.state = CanvasToolState::DrawingFreehand {
                         pen_params: self.initial_params.clone(),
@@ -130,37 +142,43 @@ impl CanvasToolCtx {
                         },
                     };
                 }
+                ctx.set_handled();
             }
             (
                 CanvasToolState::DrawingFreehand {
                     ref mut current_path,
                     ..
                 },
-                Event::MouseMove(mouse_event),
+                Event::MouseMove(pointer_event),
-            ) => if pressed(mouse_event) {
+            ) => {
-                if let CanvasElement::Freehand { ref mut path, .. } = current_path {
+                // TODO(enrico): investigate why mouse move doesn't carry buttons anymore
+                //if pressed(pointer_event) {
+                let CanvasElement::Freehand { ref mut path, .. } = current_path;
                 path.kurbo_path
-                        .line_to((mouse_event.pos.x, mouse_event.pos.y));
+                    .line_to((pointer_event.pos.x, pointer_event.pos.y));
+                ctx.set_handled();
+                //}
             }
-            }
+            (CanvasToolState::DrawingFreehand { .. }, Event::MouseUp(pointer_event))
-            (CanvasToolState::DrawingFreehand { .. }, Event::MouseUp(mouse_event)) if pressed(mouse_event) => {
+                if pressed(pointer_event) =>
+            {
                 vcanvas.update(move |canvas: &mut Canvas| {
                     let current_state = std::mem::replace(&mut self.state, CanvasToolState::Idle);
                     if let CanvasToolState::DrawingFreehand { current_path, .. } = current_state {
-                        if let CanvasElement::Freehand {
+                        let CanvasElement::Freehand {
                             mut path,
-                            mut thickness,
+                            thickness,
                             stroke_color,
-                        } = current_path
+                        } = current_path;
-                        {
+                        path.kurbo_path.apply_affine(transform);
                         canvas.add_element(CanvasElement::Freehand {
                             path,
                             thickness,
                             stroke_color,
                         });
                     }
-                    }
                 });
+                ctx.set_handled();
             }
             _ => {}
         }