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


##########
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-frontend/src/main/nifi/src/app/pages/flow-designer/ui/canvas/header/new-canvas-item/_new-canvas-item.component-theme.scss:
##########
@@ -26,20 +26,18 @@
 
     // Get the color palette from the color-config.
     $material-theme-primary-palette: map.get($material-theme-color-config, 
'primary');
+    $nifi-theme-surface-palette: map.get($nifi-theme-color-config, 'primary');
 
     // Get hues from palette
-    $surface-darker: utils.get-surface($nifi-theme-color-config, darker);
     $on-surface-highlight: utils.get-on-surface($nifi-theme-color-config, 
highlight);
     $on-surface-lighter: utils.get-on-surface($nifi-theme-color-config, 
lighter);
-
-    // Use the same logic as _navigation.component-theme.scss to determine the 
bg color for hover
-    $hover-bg: mat.get-color-from-palette($material-theme-primary-palette, 
'navbar');
-    @if ($hover-bg) {
-        // Nothing to do here, we have special color from the palette.
-    } @else {
-        // There was not a special value set for the navbar, so we use Angular 
Material behavior.
-        $hover-bg: mat.get-color-from-palette($material-theme-primary-palette, 
'default');
-    }
+    $is-dark: map-get($material-theme-color-config, is-dark);

Review Comment:
   I have added the 'not-allowed' cursor for the disabled state.



-- 
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-unsubscr...@nifi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to