996834 - CSS warning (require use of known properties) in /content/shared/css/common.css

Status: RESOLVED DUPLICATE of bug 1017882

(Hello (Loop) :: Client, defect)

Description

[Peter deHaan [:pdehaan]]

Found in sha:0141aee2225c9d2b2c04052932ff05189afebee8

### Steps to reproduce:
1. Validate https://github.com/mozilla/loop-client/blob/master/content/shared/css/common.css on http://csslint.net


### Actual results:
"CSS lint found 0 errors and 8 warnings."
Most seem harmless, but the last one seems a bit suspect:

https://github.com/mozilla/loop-client/blob/master/content/shared/css/common.css#L141

138	button.close {
139	  background: none;
140	  border: none;
141	  padding: none;
142	  cursor: pointer;
143	}

title: Require use of known properties
description: Expected (<padding-width> | inherit) but found 'none'.


### Expected results:
Not sure if we should change `padding: none;` to `padding: 0;`.

Comment 1

[Peter deHaan [:pdehaan]]

I'm not sure if this code is used by any front-end stuff or is just strictly for the readme.html file.

Comment 2

[Mark Banner (:standard8)]

This has been fixed by bug 1017882.