Stop using deprecated dojo/_base/event::stop().  This leaves lots of places calling both evt.stopPropagation() and evt.preventDefault(), somewhat bloating the code, but I think many of those cases can be changed to just calling either evt.stopPropagation() or evt. preventDefault().

Also doing coding style fixes.

Refs #16585 !strict.