This is an automatically generated e-mail. To reply, visit: http://reviewboard-tflink.rhcloud.com/r/42/ |
sass/app.scss (Diff revision 2) | |||
---|---|---|---|
87 | .foundicon-edit.blue { |
This doesn't scream "NEEDINFO" to me. What about using '?' bolded instead of the foundicon font? - ie, use a needinfo class to set the fontsize/color and just put a ? in the span.
sass/app.scss (Diff revision 2) | |||
---|---|---|---|
88 | color: $mainColor; |
using the main color here blends in too much. What do you think about using $fedoraPurple or the same yellow that the "new" indicator uses?
Overall, I like the addition but I would like to see some more visual difference from the surrounding text
- Tim Flink
On August 21st, 2013, 8:29 a.m. UTC, Martin Krizek wrote:
Review request for blockerbugs.
By Martin Krizek.
Updated Aug. 21, 2013, 8:29 a.m.
Bugs:
316
Repository:
blockerbugs
Description
Testing
Diffs
|