starting to work on modals