Ick, it would be nice to have url as a constant at the top of the file and user getElementById and addEventListener instead of querySelector and onclick, but I see that's how it already was and is consistent with the rest of the file so fair enough.