cobyism/dciy — pull request #25 by jwilm (2013-12-14 23:17:28+00:00)
Title: Homogenize the User Interface

Several of the frequently visited views in DCIY have browser default styling which results in a non uniform UI. The current UI is shown below and followed by the UI updates in this PR.

## Unstyled Views

## UI Updates in this PR
This PR updates those views with styling consistent with the rest of DCIY. A button-danger class is added to support this.

### new_project_path
All of the form labels had **bold** set as the `font-weight`. The `https://github.com/` URL snippet was placed in a span so that the input field could have the same font and size without disrupting the rest of the UI. The `

` tag was nested as in other pages to provide a uniform style. The submit input had ".button" class added.

### projects_path
Here is where most of the effort was input on this PR. As noted above, a `.button-danger` class was added to support a visual cue as to the nature of the `Remove` button. Also note the "Manage Projects" link to this page on the side. The project list was formerly set up as a table. Here we use a `

` to organize the data. The controls are nested in a span which is floated to the right.

You may want to review how the SCSS is set up for the red button style - My CSS-Fu is a bit weak, and I don't have a good field for how to manage variables for a bunch of related but non-identical colors. There's also an issue with the buttons descending below the text on the left.
![projects_path](https://f.cloud.github.com/assets/4285147/1749405/9fc14814-6511-11e3-8135-2dba6e3defaa.png)

### new_build_path
Here the `

` text is nested in the `page-header` and `page-title` divs as on other pages. The "Back" link is now inline with the form actions, and the submit button has the `.button` class.

### build_path
The `

` and `` tags here are replaced with a description list. The lines are slightly separated (vertically and horizontally) to make it feel less cluttered. You will also notice that the verbose "SHA for build.." text is replaced with "SHA". An SHA placeholder was added for when a build is first started - I figured "HEAD" was appropriate since DCIY just lets you specify a repository and a branch.

For completed builds, the commit hash display normally.

## Other comments
I built this changes on top of my last PR which has not yet been merged. If there is a better method for submitting multiple PRs, I would love to read about it.

Cheers!