SRVKP-11383: Fix alignment of loading indicators in pipeline overview charts and pipelines metric charts ( Prometheus )#1022
Conversation
- Center loading indicators in Average Duration, Status, and Number of Runs charts. - Isolate Victory Chart wrappers from flex utilities to maintain correct SVG scaling. - Conditionally render search and toggle controls in the list page so they only appear after data is loaded. Signed-off-by: Aditya Shinde <adishind@redhat.com>
|
@adityavshinde: This pull request references SRVKP-11383 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the bug to target the "4.22.0" version, but no target version was set. DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
|
@adityavshinde: This pull request references SRVKP-11383 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the bug to target the "4.22.0" version, but no target version was set. DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: adityavshinde The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
There was a problem hiding this comment.
Similar loader issue here ( please refer to the screen recording below )
Additionally the font-size is different between prometheus and tekton results, if possible please update it, if required you can create a new Jira but we should ensure that the font-sizes are similar between these 2 views
Description
This PR is for Prometheus metrics which improves the UI of the Pipeline Overview page during the data-fetching phase. Previously, the loading spinners in the chart cards were misaligned. This update uses PatternFly 6 flex utilities to properly center the
<LoadingInline />components both vertically and horizontally within their respective chart areas.Additionally, it prevents the search bar and pipeline/repository toggle controls from rendering prematurely on the list page while the initial data is still loading.
Behaviour ( via Screen Recording )
Before
Prometheus-loader-before.mp4
After
Prometheus-loader-after.mp4
Changes Made
PipelinesAverageDurationK8s.tsx: Added PatternFly flex centering utilities (justify-content-center,align-items-center) to the loading wrapper.PipelinesRunsStatusCardK8s.tsx: Conditionally applied centering utilities for both the Donut chart and Line chart loaders to ensure perfect alignment without breaking the loaded chart layouts.PipelinerunsNumbersChartK8s.tsx: Centered the loading spinner for the numbers chart area.PipelineRunsListPageK8s.tsx: Wrapped the upper controls grid (SearchInputFieldandToggleGroup) in aloadedcondition so they only appear once the underlying list data is ready.How to Test