Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

kie-issues#1595: Sandbox: Add "Open in Boxed Expression Editor" button in the DMN Runner form output cards #2720

Open
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

jomarko
Copy link
Contributor

@jomarko jomarko commented Nov 1, 2024

Closes: apache/incubator-kie-issues#1595

TODO

  • Check all EditorEnvelope.init occurrences

This PR adds a button for opening the boxed expression editor for the desired decision node from the DMN Runner form. See the button Open New Decision expression in the screenshot below.
Screenshot 2024-12-18 145214

@jomarko jomarko marked this pull request as ready for review December 18, 2024 14:19
@jomarko jomarko requested a review from tiagobento as a code owner December 18, 2024 14:19
@@ -51,6 +51,7 @@ export interface EditorInitArgs {

export interface KogitoEditorEnvelopeApi extends KeyboardShortcutsEnvelopeApi, I18nEnvelopeApi {
kogitoEditor_contentChanged(content: EditorContent, args: { showLoadingOverlay: boolean }): Promise<void>;
kogitoEditor_openBoxedExpressionEditor(nodeId: string): void;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This method is specific to the DMN Editor so shouldn't be here in the generic Editor's interface.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes, thank you @tiagobento , @ljmotta gave me a tip to look on:

  • YardChannelComponent
  • WebToolsEmbeddedEditor
  • yardTextEditor_moveCursorToPosition

working on it currently

@jomarko jomarko added the pr: wip PR is still under development label Dec 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:dmn pr: wip PR is still under development
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Sandbox: Add “Open in Boxed Expression Editor” button in the DMN Runner form output cards
2 participants