shouldn't this be done in a JSX?