bugfix: when upgrading and the limit was nil, errors would occur. The…

… negative class also wasn't having much effect and the light grey number were hardly visible
jamis · May 26, 2009 · c6f6dd1 · c6f6dd1
1 parent a42095f
commit c6f6dd1
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 6 deletions.
diff --git a/app/helpers/subscriptions_helper.rb b/app/helpers/subscriptions_helper.rb
@@ -14,7 +14,7 @@ def balance_cell(container, options={})
     classes << "negative" if balance < 0
     classes << "current_balance"
 
-    if container.is_a?(Account) && container.credit_card?
+    if container.is_a?(Account) && container.credit_card? && !container.limit.blank?
       percentage_used = container.limit.abs.to_i == 0 ? 100 :
         ((container.balance.abs.to_f / container.limit.abs.to_f) * 100).to_i
       classes << if percentage_used >= Account::DEFAULT_LIMIT_VALUES[:critical]:  "critical"

diff --git a/public/stylesheets/money.css b/public/stylesheets/money.css
@@ -266,10 +266,6 @@ td.number, th.number {
   white-space: nowrap;
 }
 
-td.current_balance, th.current_balance {
-  color: #999;
-}
-
 span.real_balance {
   font-size: 80%;
   color: #999;
@@ -281,7 +277,7 @@ span.check {
 }
 
 .negative {
-  color: red;
+  color: red !important;
 }
 
 .critical {