Add break statement to for loop after moving ColorPalette proxy
Adds a break statement to a loop in the pqSettingsDialog constructor that moves Color Palette to the end of the settings list. Once the proxy is found and moved, there is no need to continue looping. This change will avoid traversing the Color Palette twice, but more importantly, it also resolves an invalid pointer exception that gets thrown by CMB/ModelBuilder Windows executables.