This feels weird. Shouldn't it use attr() instead?