scottyaslan commented on code in PR #8766:
URL: https://github.com/apache/nifi/pull/8766#discussion_r1594284148


##########
nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-frontend/src/main/nifi/src/app/ui/common/controller-service/controller-service-table/controller-service-table.component.html:
##########
@@ -116,62 +116,73 @@
             <ng-container matColumnDef="actions">
                 <th mat-header-cell *matHeaderCellDef></th>
                 <td mat-cell *matCellDef="let item">
-                    @if (definedByCurrentGroup(item)) {
-                        <div class="flex items-center justify-end gap-x-2">
-                            <div
-                                class="pointer fa fa-cog primary-color"
-                                (click)="configureClicked(item, $event)"
-                                [title]="canConfigure(item) ? 'Edit' : 'View 
Configuration'"></div>
-                            @if (hasAdvancedUi(item)) {
-                                <div
-                                    class="pointer fa fa-cogs primary-color"
-                                    (click)="advancedClicked(item, $event)"
-                                    title="Advanced"></div>
+                    <div class="flex items-center justify-end gap-x-2">
+                        <button
+                            mat-icon-button
+                            type="button"
+                            [matMenuTriggerFor]="actionMenu"
+                            class="h-16 w-16 flex items-center justify-center 
icon global-menu">
+                            <i class="fa fa-ellipsis-v"></i>
+                        </button>
+                        <mat-menu #actionMenu="matMenu" xPosition="before">
+                            @if (definedByCurrentGroup(item)) {
+                                <button mat-menu-item 
(click)="configureClicked(item, $event)">
+                                    <i class="fa fa-cog primary-color 
mr-2"></i>
+                                    {{ canConfigure(item) ? 'Edit' : 'View 
Configuration' }}
+                                </button>
+                                @if (hasAdvancedUi(item)) {
+                                    <button mat-menu-item 
(click)="advancedClicked(item, $event)">
+                                        <i class="fa fa-cogs primary-color 
mr-2"></i>
+                                        Advanced
+                                    </button>
+                                }
+                                @if (canDisable(item)) {
+                                    <button mat-menu-item 
(click)="disableClicked(item)">
+                                        <i class="icon icon-enable-false 
primary-color mr-2"></i>
+                                        Disable
+                                    </button>
+                                }
+                                @if (canEnable(item)) {
+                                    <button mat-menu-item 
(click)="enabledClicked(item)">
+                                        <i class="fa fa-flash primary-color 
mr-2"></i>
+                                        Enable
+                                    </button>
+                                }
+                                @if (canChangeVersion(item)) {
+                                    <button mat-menu-item 
(click)="changeVersionClicked(item)">
+                                        <i class="fa fa-exchange primary-color 
mr-2"></i>
+                                        Change Version
+                                    </button>
+                                }
+                                @if (canDelete(item)) {
+                                    <button mat-menu-item 
(click)="deleteClicked(item, $event)">
+                                        <i class="fa fa-trash primary-color 
mr-2"></i>
+                                        Delete
+                                    </button>
+                                }
+                                @if (canViewState(item)) {
+                                    <button mat-menu-item 
(click)="viewStateClicked(item)">
+                                        <i class="fa fa-tasks primary-color 
mr-2"></i>
+                                        View State
+                                    </button>
+                                }
+                                @if (canManageAccessPolicies()) {
+                                    <button
+                                        mat-menu-item
+                                        (click)="$event.stopPropagation()"
+                                        [routerLink]="getPolicyLink(item)">
+                                        <i class="fa fa-key primary-color 
mr-2"></i>
+                                        Manage Access Policies

Review Comment:
   I will update all to "Manage Access Policies"



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to