Is there a reason why we didn't do this using event delegation/attachment instead? `onclick` handlers aren't great.