Table row colour according to result

Question

I am new to AngularJs I am getting json data which is in format:

[
  {
    'StudentName':'abc',
    'maths':'0',
    'english':'0',
    'economics':'0',
  }
]

I want to calculate each students marks and if marks is less than 40% then table row should be red else else should be green. I have tried. HTML

<div ng-app="MyApp" ng-controller="con1"> 
  <table id="table1">
    <tr>
      <th>student Name</th>
      <th>History Marks</th>
      <th>Maths Marks</th>
      <th>Economics Marks</th>
      <th>Percentage</th>
    </tr>
    <tr ng-repeat="x in data" ng-if="{{co(per(x))}}" ng-class="pass">
      <td>{{x.StudentName}}</td>
      <td>{{x.maths}}</td>
      <td>{{x.economics}}</td>
      <td>{{x.english}}</td>
      <td>{{per(x)}}%</td>
    </tr>
  </table>

Script

var app = angular.module('MyApp', []);
app.controller('con1', function ($scope, $http) {
    $http.get('/ajax/data').success(function (res) { $scope.data = res; });
    $scope.per = function (x) {
        avg = ((parseInt(x.maths) + parseInt(x.economics) + parseInt(x.english)) * 100) / 300;
        return avg;
    };
    $scope.co = function (x) { (x > 40) ? k = 1 : k = 0; return (k); }
});

css

.pass{background:green;}
.fail{background:red;}

I am getting percentage but according to percentage I am not getting the row colour.

Can't Jquery and AngularJS co exist? :) Haven't started using Angular but I'm using Jquery — Jared Teng, Apr 21 '15 at 14:49
@JaredT, Yes, however, you will run into a lot of Angular purists that say you should NEVER use jQuery and Angular together. I disagree. In theory it's a great concept. In reality, it just makes some things way easier to use jQuery. — Scottie, Apr 21 '15 at 16:53
@Scottie Some of the "official" AngularJS plugins on GitHub are just wrappers for jQuery plugins, though (e.g. [Angular-DragDrop](https://github.com/codef0rmer/angular-dragdrop)) — RevanProdigalKnight, Apr 21 '15 at 17:20

Satpal · Accepted Answer · 2015-04-21T14:33:06.380

9

You need ngClass

The ngClass directive allows you to dynamically set CSS classes on an HTML element by databinding an expression that represents all classes to be added.

Code

ng-class='{ "pass" : co(per(x)) == 1, "fail" : co(per(x)) == 0 }'

OR, From @RevanProdigalKnight comment

ng-class="co(per(x)) ? 'pass' : 'fail'"

when calling function with ngIf, You don;t need string interpolation.

Use

ng-if="co(per(x))"

instead of

ng-if="{{co(per(x))}}"

edited Apr 21 '15 at 14:33

answered Apr 21 '15 at 14:22

Satpal

132,252
13
159
168

and if anything, if you want to change classes depending on a value.... you should be using ng-class. . . – Steve Apr 21 '15 at 14:26
3

You could also have `ng-class="co(per(x)) ? 'pass' : 'fail'"` – RevanProdigalKnight Apr 21 '15 at 14:27
1

@RevanProdigalKnight I learned something new today! I didn't know you could use ternary operators in ng-class. Thanks! – Scottie Apr 21 '15 at 14:31

score 4 · Answer 2 · edited May 23 '17 at 10:28

4

Use ng-class

ng-class="{pass: per(x) >= 40, fail: per(x) < 40}"

References:

edited May 23 '17 at 10:28

Community

1
1

answered Apr 21 '15 at 14:24

Tushar

85,780
21
159
179

Minor fix... The classes should be pass and fail, not green and red. – Scottie Apr 21 '15 at 14:30

score 1 · Answer 3 · answered Apr 21 '15 at 14:27

ng-if will actually remove the row from the DOM if it's below 40%.

You'll want

<tr ng-repeat="x in data" ng-class="{ 'pass' : per(x) > 40 }">

As a side note, it's recommended not to use functions in ng-repeats because performance can suffer significantly.

Table row colour according to result

3 Answers3