yangzhang75 commented on code in PR #8516:
URL: https://github.com/apache/texera/pull/8516#discussion_r3993095023


##########
frontend/src/app/workspace/component/workflow-form/workflow-form.component.ts:
##########
@@ -539,14 +570,51 @@ export class WorkflowFormComponent implements OnInit, 
OnDestroy {
     return ["TEXTAREA", "SELECT"].includes(active.tagName) || 
active.isContentEditable;
   }
 
+  private operators(): OperatorPredicate[] {
+    return this.workflowActionService.getTexeraGraph().getAllOperators();

Review Comment:
   Right. Fixed in a007da5b5: the picker iterates the enabled operators only, 
and the shown results apply the same rule to a viewed step (its eye survives 
disabling, but the compiled plan leaves the step out). The spec adds a disabled 
step with the eye and one already picked: neither is offered, and the picked 
one is not shown.



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