add line numbers to make todo
refactoring to not pass entire sheet object around with selection