Some more comments, out of the blue: 1. if you create instances of a class that has a Dispose() method(such as Graphics, Font, Pen, Brush), you should call it when done with the objects. 2. you should avoid creating objects inside an event handler that is going to be called often; say you need a few brushes, just create them once and keep them around. 3. There are a whole lot of stock objects available at no extra charge, see classes such as Pens and Brushes. (and no, you don't dispose them, they aren't yours, you just borrow them). :)
Luc Pattyn [My Articles] Nil Volentibus Arduum
The quality and detail of your question reflects on the effectiveness of the help you are likely to get.
Please use <PRE> tags for code snippets, they improve readability.
CP Vanity has been updated to V2.3