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#1281: The Timeline goes to the bottom with long endpoints in WorkflowInstance page #2563

Merged
merged 5 commits into from
Oct 11, 2024

Conversation

kumaradityaraj
Copy link
Contributor

@kumaradityaraj kumaradityaraj commented Sep 6, 2024

Closes apache/incubator-kie-issues#1281

Description :
When a workflow instance was opened and the endpoint was too long then the timeline card moves below the details card because the full endpoint link was displayed. Now the truncated endpoint link is displayed.

How to test:
Open a workflow instance and try to change the endpoint url to a long string; it will display the truncated link.

Preview:

Screen.Recording.2024-09-27.at.12.02.32.PM.mov

@kumaradityaraj
Copy link
Contributor Author

@fantonangeli Please review this PR. Thank you.

@fantonangeli fantonangeli self-requested a review September 6, 2024 09:07
@fantonangeli fantonangeli added area:sonataflow pr: wip PR is still under development labels Sep 6, 2024
@tiagobento
Copy link
Contributor

@fantonangeli Can you please review this? Thanks!

@fantonangeli
Copy link
Contributor

@tiagobento I think there is a layout issue with the Flex elements which doesn't allow the Truncate to work well without expanding the Card.
File: packages/runtime-tools-swf-enveloped-components/src/workflowDetails/envelope/components/WorkflowDetails/WorkflowDetails.tsx
I'm trying to reserve some time for this PR.

@tiagobento
Copy link
Contributor

@kumaradityaraj Your screenshot didn't finish uploading to the description...

Copy link
Contributor

@fantonangeli fantonangeli left a comment

Choose a reason for hiding this comment

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

Tested on:

  • serverless-logic-web-tools
  • serverless-workflow-dev-ui-webapp
  • sonataflow-deployment-webapp
  • sonataflow-management-console
  • jbpm-compact-architecture-example
  • sonataflow-greeting-quarkus-example

LGTM. Please @kumaradityaraj update the video

@fantonangeli fantonangeli removed the pr: wip PR is still under development label Sep 26, 2024
@fantonangeli fantonangeli merged commit b905936 into apache:main Oct 11, 2024
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

The Timeline goes to the bottom with long endpoints in WorkflowInstance page
5 participants