Skip to content

Commit

Permalink
Merge pull request #238 from stuartwoodman/AUS-3888-Remove-Opacity-Sl…
Browse files Browse the repository at this point in the history
…ider-For-Unsupported-Layers

AUS-3888 Remove opacity slider for unsupported layers
  • Loading branch information
vjf authored Mar 9, 2023
2 parents a395ce7 + 643f2d9 commit 4e1d03a
Show file tree
Hide file tree
Showing 4 changed files with 20 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
</u>
<i *ngIf="getUILayerModel(layer.id).statusMap.getRenderStarted()" class="fa fa-trash float-right red layerLoadClickable" (click)="removeLayer(layer.id);$event.stopPropagation()"></i>
</div>
<div *ngIf="getUILayerModel(layer.id).statusMap.getRenderStarted()" class="opacity-slider-panel d-flex" (click)="$event.stopPropagation()">
<div *ngIf="showOpacitySlider(layer) && getUILayerModel(layer.id).statusMap.getRenderStarted()" class="opacity-slider-panel d-flex" (click)="$event.stopPropagation()">
<div class="opacity-label">Opacity {{ getUILayerModel(layer.id).opacity }}%&nbsp;</div>
<mat-slider [min]="0" [max]="100" class="opacity-slider flex-grow-1" [(ngModel)]="getUILayerModel(layer.id).opacity" (input)="layerOpacityChange($event, layer)"></mat-slider>
<div *ngIf="hasLegend(layer)">
Expand Down
9 changes: 9 additions & 0 deletions src/app/menupanel/activelayers/activelayerspanel.component.ts
Original file line number Diff line number Diff line change
Expand Up @@ -65,6 +65,15 @@ export class ActiveLayersPanelComponent {
*/
}

/**
* Determine if a layer hsould have an opacity slider
* @param layer the layer
* @returns true if a layer should have an opacity slider, false otherwise
*/
showOpacitySlider(layer: LayerModel): boolean {
return this.csMapService.layerHasOpacity(layer);
}

/**
* Layer opacity slider change event
*/
Expand Down
2 changes: 1 addition & 1 deletion src/app/menupanel/layerpanel/layerpanel.component.html
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@
<div class="progress-bar bg-gradient-blue-purple" [style.width]="getUILayerModel(layer.id).statusMap.getCompletePercentage()" role="progressbar"></div>
</div>
-->
<div *ngIf="getUILayerModel(layer.id) && (getUILayerModel(layer.id).statusMap.getRenderStarted() || getUILayerModel(layer.id).statusMap.getRenderComplete())" class="opacity-slider-panel d-flex" (click)="$event.stopPropagation()">
<div *ngIf="showOpacitySlider(layer) && getUILayerModel(layer.id) && (getUILayerModel(layer.id).statusMap.getRenderStarted() || getUILayerModel(layer.id).statusMap.getRenderComplete())" class="opacity-slider-panel d-flex" (click)="$event.stopPropagation()">
<div class="opacity-label">Opacity {{ getUILayerModel(layer.id).opacity }}%&nbsp;</div>
<mat-slider [min]="0" [max]="100" class="opacity-slider flex-grow-1" [(ngModel)]="getUILayerModel(layer.id).opacity" (input)="layerOpacityChange($event, layer)"></mat-slider>
</div>
Expand Down
9 changes: 9 additions & 0 deletions src/app/menupanel/layerpanel/layerpanel.component.ts
Original file line number Diff line number Diff line change
Expand Up @@ -221,6 +221,15 @@ export class LayerPanelComponent implements OnInit {
this.legendUiService.removeLegend(layer.id);
}

/**
* Determine if a layer hsould have an opacity slider
* @param layer the layer
* @returns true if a layer should have an opacity slider, false otherwise
*/
showOpacitySlider(layer: LayerModel): boolean {
return this.csMapService.layerHasOpacity(layer);
}

/**
* Layer opacity slider change event
*/
Expand Down

0 comments on commit 4e1d03a

Please sign in to comment.