modal now working - but