#388: Removed branch coverage columns if no branch coverage is available

danielpalme · Sep 8, 2020 · cea803b · cea803b
1 parent 5c54844
commit cea803b
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/src/AngularComponents/src/app/components/coverageinfo/coverage-info.component.ts b/src/AngularComponents/src/app/components/coverageinfo/coverage-info.component.ts
@@ -98,11 +98,11 @@ import { CodeElementViewModel } from "./viewmodels/codelement-viewmodel.class";
                   [ngClass]="{'icon-up-dir_active': settings.sortBy === 'coverage' && settings.sortOrder === 'desc',
                   'icon-down-dir_active': settings.sortBy === 'coverage' && settings.sortOrder === 'asc',
                   'icon-down-dir': settings.sortBy !== 'coverage'}"></i>{{translations.coverage}}</a></th>
-            <th class="right"><a href="#" (click)="updateSorting('covered_branches', $event)"><i class="icon-down-dir"
+            <th class="right" *ngIf="branchCoverageAvailable"><a href="#" (click)="updateSorting('covered_branches', $event)"><i class="icon-down-dir"
               [ngClass]="{'icon-up-dir_active': settings.sortBy === 'covered_branches' && settings.sortOrder === 'desc',
               'icon-down-dir_active': settings.sortBy === 'covered_branches' && settings.sortOrder === 'asc',
               'icon-down-dir': settings.sortBy !== 'covered_branches'}"></i>{{translations.covered}}</a></th>
-            <th class="right"><a href="#" (click)="updateSorting('total_branches', $event)"><i class="icon-down-dir"
+            <th class="right" *ngIf="branchCoverageAvailable"><a href="#" (click)="updateSorting('total_branches', $event)"><i class="icon-down-dir"
                 [ngClass]="{'icon-up-dir_active': settings.sortBy === 'total_branches' && settings.sortOrder === 'desc',
                 'icon-down-dir_active': settings.sortBy === 'total_branches' && settings.sortOrder === 'asc',
                 'icon-down-dir': settings.sortBy !== 'total_branches'}"></i>{{translations.total}}</a></th>

diff --git a/src/Readme.txt b/src/Readme.txt
@@ -66,6 +66,7 @@ CHANGELOG
 4.6.7.0
 
     * Fix: #363: Improved handling of HTML generation (several HTML reports at once)
+    * Fix: #388: Removed branch coverage columns if no branch coverage is available
 
 4.6.6.0
 

diff --git a/src/ReportGenerator.Core/Reporting/Builders/Rendering/resources/main.js b/src/ReportGenerator.Core/Reporting/Builders/Rendering/resources/main.js