mcmurrayre/systems_check_3 — pull request #1 by hchood (2014-09-02 20:20:36+00:00)
Title: Indentation & HTML edits

Hi Ryan,

Nice work. The systems check looked good :)

I made some edits on the PR but here's a high-level summary:

**Modify query for `/recipes/:id` page to retrieve info for a single recipe.** You can add a `WHERE` clause in your query to retrieve only the listings for a particular recipe.

**Indentation.** Make sure to follow conventions on indentation! The sooner you get used to indenting things properly the better. It'll become super important once we get to bigger apps and employers see sloppy indentation as a mark of inexperience/lack of professionalism. This is especially important in your views. It can be hard to catch a missing closing tag, for example, if indentation is messed up.

**Tables vs. unordered lists.** In these contexts, I'd probably use unordered lists instead of tables. Tables are good for showing numerous pieces of information about multiple records. For the index page, however, we're only displaying a single piece of information (recipe name) and on the show page we're only showing one piece of information about each ingredient.

**Wrap any text in your views inside an HTML tag.** You had a couple of erb tags floating around inside the body, not wrapped in a `

`, `

`, or other tag. It's generally a good idea to wrap those in HTML tags.

**I wouldn't use the `` tag.** You won't see it used much -- layout stuff is usually done through CSS.

Let me know if you have any questions :)