[Message Deleted]

[ABitSmart]

When are you calling OnMouseUp ? Taking a guess, I think you do not need it. Looks like OnMouseDown and OnMouseMove are doing what you want ??

[Prajeesh]

[Message Deleted]

[ABitSmart]

Prajeesh wrote:

StartPoint = Args.GetPosition(PhotoBox);

In the MoveDown event, what is PhotoBox? shouldn't it be DrawingCanvas ? I did the same thing you are doing, on the MouseMove and MouseLeftButtonDown event of the Canvas, and I never got any extra lines.

[Prajeesh]

[Message Deleted]

[ABitSmart]

private void OnMouseMove(object Sender, MouseEventArgs Args) { if (Args.LeftButton == MouseButtonState.Pressed) { EndPoint = Args.GetPosition(DrawingCanvas); DrawLine(StartPoint, EndPoint, DrawingCanvas); //do not reset the startpoint //StartPoint = EndPoint; } } And in your DrawLine method, delete the old line before adding the new line.

[Prajeesh]

[Message Deleted]

[ABitSmart]

Searching for the control and removing it from the Children collection. Something like this,

Line previousLine = null;
foreach (UIElement child in TargetCanvas.Children )
{
if( child is Line)
{
previousLine = child as Line;
break;
}
}

if (null != previousLine)
{
TargetCanvas.Children.Remove(previousLine);
}

please update it to fit your needs.

[Prajeesh]

[Message Deleted]

[ABitSmart]

Yes, I hope you can build ahead from the help you have been provided. What you could do is generate a new line name everytime you start drawing and use that line name in the MoveMouse for adding the new line and also for deleting the existing line.

[Pete OHanlon]

I'd leave it if I were you. It looks like he wants you to give him the complete code.

"WPF has many lovers. It's a veritable porn star!" - Josh Smith

As Braveheart once said, "You can take our freedom but you'll never take our Hobnobs!" - Martin Hughes.

My blog | My articles | MoXAML PowerToys

[ABitSmart]

I reminded him about the advise you had provided him few posts ago, but he doesn't seem to pay any heed to it.