nodebugme/site — pull request #8 by twolfson (2014-11-01 05:26:56+00:00)
Title: Repaired broken label functionality for Firefox. Fixes #7

As reported in #7, Firefox dislikes having 2 `inputs` in 1 `label`. In this PR:

- Relocated second `input` outside of `label` thus fixing behavior
- Renamed `label` CSS to `.input-row` to make it more reusable and less conflict prone
- Updated all containers for `input's` to use `.input-row` to preserve previous styling

This has not been tested with a live server. It has been tested with copy/paste/replace HTML/CSS in the browser. I attempted to get a `vagrant` box running but got frustrated with `postgres` setup.

https://github.com/twolfson/vagrant-nodebugme

On another note, I noticed a lot of bad practices in the CSS. Using content semantic classes can lead to unnecessary issues and extra work like having to update all `label's` to `.input-row's`. From my experience, using [OOCSS][] conventions leads to less issues and more reusable classes. If you would like an explanation of how OOCSS could apply to this project, let me know and I will elaborate in another thread.

[OOCSS]: http://oocss.org/