CSS clean-up.

CSS clean-up.

BUG=115350
TEST=Manual

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=125001

[Jamie, 2012-03-03 01:38:08]

The css had a lot of conflicting rules. I've tried to consolidate and simplify
them as much as possible.

https://chromiumcodereview.appspot.com/9582038/diff/1/remoting/webapp/connect...
File remoting/webapp/connection_history.css (right):

https://chromiumcodereview.appspot.com/9582038/diff/1/remoting/webapp/connect...
remoting/webapp/connection_history.css:29: border-spacing: 0;
Moved from main.css:<table> rule.

https://chromiumcodereview.appspot.com/9582038/diff/1/remoting/webapp/host_li...
File remoting/webapp/host_list.js (left):

https://chromiumcodereview.appspot.com/9582038/diff/1/remoting/webapp/host_li...
remoting/webapp/host_list.js:206: }
The 'collapsed' style was used to animate the host list when it appears. It's no
longer used.

https://chromiumcodereview.appspot.com/9582038/diff/1/remoting/webapp/main.css
File remoting/webapp/main.css (left):

https://chromiumcodereview.appspot.com/9582038/diff/1/remoting/webapp/main.cs...
remoting/webapp/main.css:301: button[disabled], .button[disabled]:hover,
.button[disabled]:active {
These dots were a bug.

https://chromiumcodereview.appspot.com/9582038/diff/1/remoting/webapp/main.css
File remoting/webapp/main.css (right):

https://chromiumcodereview.appspot.com/9582038/diff/1/remoting/webapp/main.cs...
remoting/webapp/main.css:53: button {
A dedicated button class? My buttons are classy enough already.

https://chromiumcodereview.appspot.com/9582038/diff/1/remoting/webapp/main.cs...
remoting/webapp/main.css:69: button:hover:not([disabled]) {
I think this is clearer than an explicit rule for hovering on disabled buttons.

https://chromiumcodereview.appspot.com/9582038/diff/1/remoting/webapp/main.cs...
remoting/webapp/main.css:138: .kd-modaldialog:not([hidden]) {
modaldialog has the odd dual definition because it's supposed to 'animate in'
when you show it. I haven't got that working yet, but I'll keep the CSS until I
do (or I give up).

https://chromiumcodereview.appspot.com/9582038/diff/1/remoting/webapp/main.html
File remoting/webapp/main.html (left):

https://chromiumcodereview.appspot.com/9582038/diff/1/remoting/webapp/main.ht...
remoting/webapp/main.html:68: <a id="clear-oauth" href="#"
i18n-content="SIGN_OUT_BUTTON"></a> |
No href means the links don't change colour if you click them, which is what we
want for 'action' links like this one.

[garykac, 2012-03-05 21:16:16]

lgtm