Skip to content

fix(orchestrator): fix responsiveness of the WorkflowPage#554

Merged
mareklibra merged 1 commit into
redhat-developer:mainfrom
LiorSoffer:runButton
Apr 3, 2025
Merged

fix(orchestrator): fix responsiveness of the WorkflowPage#554
mareklibra merged 1 commit into
redhat-developer:mainfrom
LiorSoffer:runButton

Conversation

@LiorSoffer
Copy link
Copy Markdown
Contributor

@LiorSoffer LiorSoffer commented Mar 24, 2025

resolve FLPATH-2232

Before:

Screencast.from.2025-03-25.15-04-57.mp4

After:

Screencast.from.2025-03-25.15-12-43.mp4

@rhdh-gh-app
Copy link
Copy Markdown

rhdh-gh-app Bot commented Mar 24, 2025

Changed Packages

Package Name Package Path Changeset Bump Current Version
@red-hat-developer-hub/backstage-plugin-orchestrator workspaces/orchestrator/plugins/orchestrator patch v2.6.2

@mareklibra
Copy link
Copy Markdown
Member

@LiorSoffer Can you please add screenshots? Before and after?

@LiorSoffer
Copy link
Copy Markdown
Contributor Author

@mareklibra this small change is about making the code clearer, my old solution was ugly and over complicated. Along the way, I also fixed how the workflow runs tab looks when it's shrunk (screenshots added).

@christoph-jerolimov christoph-jerolimov changed the title fix(orchestrator):rename WorkflowPageTabContent to RunButton fix(orchestrator): rename WorkflowPageTabContent to RunButton Mar 31, 2025
@LiorSoffer LiorSoffer closed this Mar 31, 2025
@LiorSoffer LiorSoffer deleted the runButton branch March 31, 2025 18:29
@LiorSoffer LiorSoffer restored the runButton branch March 31, 2025 18:44
@LiorSoffer LiorSoffer reopened this Mar 31, 2025
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This text should be written from the user's point of view.
IIUC the change is in fixing responsiveness of the WorkflowPage.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Otherwise it LGTM

@LiorSoffer LiorSoffer changed the title fix(orchestrator): rename WorkflowPageTabContent to RunButton fix(orchestrator): fixing responsiveness of the WorkflowPage Apr 1, 2025
@LiorSoffer LiorSoffer changed the title fix(orchestrator): fixing responsiveness of the WorkflowPage fix(orchestrator): fix responsiveness of the WorkflowPage Apr 1, 2025
@LiorSoffer LiorSoffer requested a review from mareklibra April 2, 2025 10:20
Signed-off-by: Lior Soffer <liorsoffer1@gmail.com>
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Apr 3, 2025

@mareklibra mareklibra merged commit 28c900b into redhat-developer:main Apr 3, 2025
9 checks passed
ishanjogi89 added a commit to ishanjogi89/rhdh-plugins that referenced this pull request Apr 7, 2025
…ugins into FLPATH-2224

* 'FLPATH-2224' of https://github.com/ijogi-redhat/rhdh-plugins:
  feat(lightspeed): Adopt road-core service APIs as backend upstream  (redhat-developer#516)
  Version Packages (redhat-developer#606)
  fix(global-header): remove search bar underline (redhat-developer#603)
  Version Packages (redhat-developer#563)
  fix(orchestrator): Add Workflow Status to WorkflowsTable (redhat-developer#564)
  chore(lightspeed): upgrade PF chatbot in lightspeed (redhat-developer#602)
  Version Packages (redhat-developer#600)
  Version Packages (redhat-developer#588)
  feat(marketplace): removing view plugins button (redhat-developer#599)
  fix(ai-integrations): improve responsiveness and layout (redhat-developer#596)
  fix(orchestrator): fix responsiveness of the WorkflowPage (redhat-developer#554)
  fix(orchestrator): fix view variables screen (redhat-developer#595)
  feat(ai-experience): Integrate new ai-experience backend plugin (redhat-developer#594)
@LiorSoffer LiorSoffer deleted the runButton branch August 28, 2025 12:14
elai-shalev pushed a commit to elai-shalev/rhdh-plugins that referenced this pull request Jan 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants