Patch #44567
openModernize Gantt chart HTML structure and add row highlighting
Description
This patch modernizes the Gantt chart HTML structure as part of step 3 of #43397. It replaces the legacy table-based layout with a clearer structure that should make the chart easier to understand and extend.
It also adds row highlighting as an initial feature built on the new structure, making wide Gantt charts easier to read.
Changes¶
Refactor and modernize the Gantt chart layout¶
- Replace the table-based layout with a flex-and-pane layout.
- Replace
.gantt_hdrby splitting its multiple responsibilities—headers, borders, timeline periods, grid lines, and non-working-day backgrounds—into dedicated structural elements. - Introduce consistent logical rows across the chart.
- Move layout calculations out of the ERB template, making the overall HTML structure easier to see and understand.
Organize the Gantt chart DOM structure and semantics¶
- Clarify the roles of classes and
data-*attributes. - Use consistent class names and remove obsolete markup.
Highlight corresponding Gantt rows¶
- Highlight the corresponding row across the Gantt chart.
- Make wide charts easier to read and discuss during screen sharing.
Verification¶
Visual regression testing¶
No visual regressions were found through automated checks and manual review.
- 365 before-and-after image comparisons across 42 scenarios in 20 test groups.
- Coverage includes hierarchies, zoom levels, date ranges, display options, interactions, Arabic RTL rendering, printing, and PDF and PNG exports.
Results: https://redmine-gantt-comparison.hidakatsuya.chatgpt.site/
Performance¶
No performance regression was observed in a heavy test with 400 issues over 12 months at maximum zoom, three additional columns, 80 relations, and the progress line enabled.
| Metric | Before | After |
|---|---|---|
| Page ready time | 1,397.5 ms | 1,361.9 ms (-2.5%) |
| Ruby allocated memory per request | 108.98 MiB | 106.56 MiB (-2.22%) |
| Ruby retained memory per request | 14.96 MiB | 15.04 MiB (+0.53%) |
Manual browser testing¶
- Chrome, Firefox, and Safari on macOS.
- Chrome on Windows 11.
Next steps¶
I plan to continue focusing on making the Gantt chart easier to use for reviewing schedules and understanding project status.
- Remembering column widths, adding a sticky timeline header, and adopting the OpenColor palette.
- Improving the internal structure and logic: refactoring view data handling, simplifying absolute positioning calculations, and organizing the Stimulus controllers.
Appendix: DOM structure before and after¶
| Before | After |
|---|---|
table.gantt-table
└─ tr
├─ td.gantt_subjects_column
│ └─ div.gantt_subjects_container
│ ├─ div.gantt_hdr
│ │ absolute header background
│ ├─ div.gantt_hdr
│ │ absolute full-height overlay
│ └─ div.gantt_subjects
│ └─ form
│ ├─ div.project-name
│ │ individually positioned
│ ├─ div.version-name
│ │ individually positioned
│ └─ div.issue-subject × N
│ individually positioned
│
├─ td.gantt_selected_column × N
│ └─ div.gantt_selected_column_container
│ ├─ div.gantt_hdr
│ │ absolute full-height overlay
│ ├─ div.gantt_hdr
│ │ absolute column title
│ └─ div.gantt_selected_column_content
│ └─ div × N
│ individually positioned
│
└─ td
└─ div#gantt_area
├─ div.gantt_hdr
│ absolute header background
├─ div.gantt_hdr × months
│ absolute month cells
├─ div.gantt_hdr × weeks
│ absolute week cells
├─ div.gantt_hdr × days
│ absolute day cells and body grid
├─ form
│ ├─ div.task × N
│ │ individually positioned bars
│ ├─ div.task.label × N
│ │ individually positioned labels
│ └─ span.tooltip × N
├─ div#today_line
│ absolute
└─ div#gantt_draw_area
absolute
|
div.gantt-chart
├─ div.gantt-column
│ [data-gantt-column="subjects"]
│ └─ section.gantt-pane
│ ├─ header
│ └─ div.gantt-pane-body
│ └─ form
│ └─ div.gantt-row × N
│
├─ div.gantt-column
│ [data-gantt-column="status"]
│ └─ section.gantt-pane
│ ├─ header
│ └─ div.gantt-pane-body
│ └─ div.gantt-row × N
│
└─ div.gantt-timeline
└─ div.gantt-timeline-canvas
├─ header
│ └─ div.gantt-header-row
│ └─ div.gantt-period × N
└─ div.gantt-timeline-body
├─ form
│ └─ div.gantt-row × N
├─ div.gantt-today
└─ div.gantt-relations
|
The new structure removes the layout table and the generic .gantt_hdr, separates headers from body content, and introduces consistent pane and row structures across the chart.
Files
Related issues
Updated by Katsuya HIDAKA 1 day ago
You can also review the changes on GitHub: https://github.com/hidakatsuya/redmine/pull/57
To apply the patch locally, run the following from the Redmine source directory:
curl -L https://patch-diff.githubusercontent.com/raw/hidakatsuya/redmine/pull/57.patch | git am
Updated by Katsuya HIDAKA 1 day ago
Here is a screenshot of the updated Gantt chart. There are no visual changes except for the row highlighting.

Updated by Go MAEDA about 24 hours ago
Thank you for this patch.
Although the description does not mention it, I found that this patch also makes the Gantt chart render faster.
I tested it with a project that has 2,000 issues in my test environment. Rendering the Gantt tab became faster, and the improvement is far greater when collapsing and expanding a project.
| Operation | Before | After |
|---|---|---|
| Rendering the Gantt tab | 1,406 ms | 1,235 ms |
| Collapsing a project | 15,270 ms | 528 ms |
| Expanding a project | 19,061 ms | 1,507 ms |
Each figure is the average of three runs. The rendering time is the sum of the "Rendering" and "Scripting" times reported by the Performance panel of Chrome DevTools.
Updated by Go MAEDA about 10 hours ago
In addition to my previous comment, I found that resizing a column also became much faster.
With a project that has 2,000 issues, I dragged the right edge of the subject column and measured the time taken to process each mouse movement. It decreased from 6,746 ms to 90 ms. Before the patch, the chart froze for several seconds on every movement. After the patch, the column follows the mouse pointer smoothly.
| Operation | Before | After |
|---|---|---|
| Resizing the subject column (per mouse movement) | 6,746 ms | 90 ms |
Each figure is the median of three runs, measured with headless Chrome.
Updated by Go MAEDA about 9 hours ago
- Related to Feature #43397: Organize Gantt chart code by separating it into views and Stimulus controllers added
Updated by Katsuya HIDAKA about 9 hours ago
Thank you for testing the patch and for the detailed performance measurements.
These improvements are benefits of the structural refactoring. In particular, collapsing and expanding now use gantt-parent-row-key to identify the hierarchy instead of inferring it from indentation. Improving performance here was also part of the intent.
Updated by Katsuya HIDAKA about 9 hours ago
- File 0001-Refactor-and-modernize-Gantt-chart-layout.patch 0001-Refactor-and-modernize-Gantt-chart-layout.patch added
I am replacing 0001-Refactor-and-modernize-Gantt-chart-layout with an updated version that:
- Adds tests for the
ChartLayouthelper class. - Refactors its layout calculations to make them easier to understand.
You can review the changes in this GitHub comparison.
If you want to apply the full updated patch series, you can follow the instructions in #note-1.