You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@nifi.apache.org by "dan-s1 (via GitHub)" <gi...@apache.org> on 2025/03/21 16:00:16 UTC
Re: [PR] NIFI-14385 allow button with underlying conditional options to be displayed for every FlowFile item on a list queue table [nifi]
dan-s1 commented on code in PR #9815:
URL: https://github.com/apache/nifi/pull/9815#discussion_r2007845976
##########
nifi-frontend/src/main/frontend/apps/nifi/src/app/pages/queue/ui/queue-listing/flowfile-table/flowfile-table.component.html:
##########
@@ -112,23 +112,17 @@ <h3 class="queue-listing-header primary-color">{{ selectedConnection?.label }}</
<th mat-header-cell *matHeaderCellDef></th>
<td mat-cell *matCellDef="let item">
<div class="flex items-center justify-end gap-x-2">
- @if (
- item.size > 0 ||
- (contentViewerAvailable && item.size > 0) ||
- currentUser.provenancePermissions.canRead
- ) {
- <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>
- }
+ <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">
<button mat-menu-item (click)="viewFlowFileClicked(item)">
<i class="fa fa-info-circle primary-color mr-2"></i>
- View Details
+ View details
Review Comment:
I assume there is no reason to change the case
```suggestion
View Details
```
--
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: issues-unsubscribe@nifi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org