Skip to content

Conversation

AdekeyeAdeniyi
Copy link
Contributor

Summary

This PR makes the main dashboard pie chart fully responsive across screen sizes.

Changes

  • Enabled responsive: true and maintainAspectRatio: false in chart options.
  • Commented out the thicknessPlugin option.
  • Adjusted chart styling to ensure adaptive resizing on different viewports.

Linked Issue

Closes #11525

@AdekeyeAdeniyi
Copy link
Contributor Author

@MilosPaunovic
Please review my PR when you're available.
Also, I’d appreciate your feedback on my PR message —I hope it looks okay.

@MilosPaunovic MilosPaunovic self-requested a review October 9, 2025 06:32
@MilosPaunovic MilosPaunovic moved this from To review to On hold in Pull Requests Oct 9, 2025
@MilosPaunovic
Copy link
Member

Hey @AdekeyeAdeniyi, are there any updates on this?

Copy link
Contributor Author

@AdekeyeAdeniyi AdekeyeAdeniyi left a comment

Choose a reason for hiding this comment

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

I have revert the docker file to it default confoguration

close #11525

@AdekeyeAdeniyi
Copy link
Contributor Author

@MilosPaunovic I have fixed the issue.

@MilosPaunovic MilosPaunovic moved this from On hold to To review in Pull Requests Oct 13, 2025
Copy link
Member

@MilosPaunovic MilosPaunovic left a comment

Choose a reason for hiding this comment

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

Hi @AdekeyeAdeniyi,

This is good progress, but we're not there yet.

If you make the field narrower, the chart will make itself smaller, but it will never get to the initial size again:

Screencast.from.2025-10-13.12-54-22.webm

If you create a custom dashboard by using this code:

id: mouse_224526
title: asdasd
description: Default Main Overview

timeWindow:
  default: P30D
  max: P365D

charts:
  - id: total_executions_timeseries
    type: io.kestra.plugin.core.dashboard.chart.TimeSeries
    chartOptions:
      displayName: Total Executions
      description: Executions duration and count per date
      legend:
        enabled: true
      column: date
      colorByColumn: state
      width: 9
    data:
      type: io.kestra.plugin.core.dashboard.data.Executions
      columns:
        date:
          field: START_DATE
          displayName: Date
        state:
          field: STATE
        total:
          displayName: Executions
          agg: COUNT
          graphStyle: BARS
        duration:
          field: DURATION
          displayName: Duration
          agg: SUM
          graphStyle: LINES

  - id: total_executions_pie
    type: io.kestra.plugin.core.dashboard.chart.Pie
    chartOptions:
      displayName: Total Executions
      graphStyle: DONUT
      tooltip: ALL
      width: 3
      legend:
        enabled: true
    data:
      type: io.kestra.plugin.core.dashboard.data.Executions
      columns:
        id:
          field: ID
          displayName: Execution Id
          agg: COUNT
        state:
          field: STATE
          displayName: State

Then you'll get weird behavior:
Screencast from 2025-10-13 12-53-51.webm

@MilosPaunovic MilosPaunovic moved this from To review to On hold in Pull Requests Oct 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: On hold

Development

Successfully merging this pull request may close these issues.

Make the main dashboard responsive

2 participants