mcgilman commented on code in PR #11224:
URL: https://github.com/apache/nifi/pull/11224#discussion_r3220808055


##########
nifi-frontend/src/main/frontend/libs/shared/src/components/connector-wizard/connector-wizard.component.html:
##########
@@ -90,16 +90,20 @@
                         </mat-step>
 
                         <ng-template matStepperIcon="number">
-                            <i class="fa fa-circle-o text-[24px] leading-none 
align-middle" aria-hidden="true"></i>
+                            <i class="fa fa-circle text-[16px] leading-none 
align-middle" aria-hidden="true"></i>
                         </ng-template>
                         <ng-template matStepperIcon="edit">
-                            <i class="fa fa-circle-o text-[24px] leading-none 
align-middle" aria-hidden="true"></i>
+                            <i
+                                class="fa fa-circle-o secondary-color 
text-[16px] leading-none align-middle"
+                                aria-hidden="true"></i>

Review Comment:
   I think the icons we're using for the steps is too thick and appear too 
strong.



##########
nifi-frontend/src/main/frontend/libs/shared/src/components/wizard/wizard.component.html:
##########


Review Comment:
   `step-disabled` was removed from this components styles. Is this still 
needed or was the style inadvertently removed.



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