search

timber / starter-theme

The "_s" for Timber: a dead-simple theme that you can build anything from
MIT License
812 stars 274 forks source link

Update comment-form.twig with improved standards #75

Closed stefenphelps closed 5 years ago

stefenphelps commented 5 years ago

Since multiple forms could display on the same post, I removed all IDs and instead wrapped the label around each input to keep things accessible. I also updated each input with the proper type attribute for better mobile keyboard support.

jarednova commented 5 years ago

Thanks @stefenphelps, didn't realize that wrapping elements in <label> was the new way of doing things. But Google confirms it — merging!