shamrickus commented on code in PR #7514:
URL: https://github.com/apache/trafficcontrol/pull/7514#discussion_r1202247219


##########
experimental/traffic-portal/src/app/core/users/roles/table/roles-table.component.html:
##########
@@ -25,4 +25,4 @@
 </mat-card>
 
 
-<a class="page-fab" mat-fab title="Create a new role" 
*ngIf="auth.hasPermission('ROLE:CREATE')" 
routerLink="new"><mat-icon>add</mat-icon></a>

Review Comment:
   The `tp-generic-table` has a the ability to double click to navigate to 
detail through the  property `[doubleClickLink]`, would you implement that? You 
can see an example in `servers-table`. Might need to rebase



-- 
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