Skip to content

Remove extra gap when 0 roadmap epics have children

Chad Lavimoniere requested to merge 464602-followup-epics-without-children-spacing into master

What does this MR do and why?

When none of the epics shown in a roadmap view have children, remove the unnecessary gap that is left for the show/hide children toggle button.

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Screenshots or screen recordings

Before After
gdk.test_3000_groups_flightjs_-_roadmap_state_all_sort_START_DATE_ASC_layout_QUARTERS_timeframe_range_type_THREE_YEARS_epic_iid_6_progress_WEIGHT_show_progress_true_show_milestones_true_milestones_type_ALL_show_labels_true gdk.test_3000_groups_flightjs_-roadmap_state_all_sort_START_DATE_ASC_layout_QUARTERS_timeframe_range_type_THREE_YEARS_epic_iid_6_progress_WEIGHT_show_progress_true_show_milestones_true_milestones_type_ALL_show_labels_true__2
gdk.test_3000_groups_flightjs_-roadmap_state_all_sort_START_DATE_ASC_layout_QUARTERS_timeframe_range_type_THREE_YEARS_epic_iid_6_progress_WEIGHT_show_progress_true_show_milestones_true_milestones_type_ALL_show_labels_true__1 gdk.test_3000_groups_flightjs_-roadmap_state_all_sort_START_DATE_ASC_layout_QUARTERS_timeframe_range_type_THREE_YEARS_epic_iid_6_progress_WEIGHT_show_progress_true_show_milestones_true_milestones_type_ALL_show_labels_true__3

(Unchanged)

How to set up and validate locally

  1. Check out this branch
  2. Ensure that you have a group that contains some epics that do not have child epics, and some epics that do have child epics
  3. Visit the Roadmap for that group
  4. Filter the roadmap on an epic that has no child epics

You will see that there is not a blank space where the child epic tree toggle button would go to the left of the epic name

Related to #464602 (closed)

Edited by Chad Lavimoniere

Merge request reports