popup move to constructor