OnError is a very old Basic statement. I already used it about 30 years ago. It looks like OnClick but it is different. OnClick appeared later and is used in a different way.

OnError should be used with Resume something at some place. Resume is necessary to end the OnError event, and maybe you forgot it. With OnClick, you don't need to end the event.

See http://www.cpearson.com/excel/errorhandling.htm , http://www.your-save-time-and-improv...error-vba.html