Skip to content

Migrated branches_dropdown and projects_dropdown to GlCollapsibleListbox

Ross Byrne requested to merge 384280-migrate-dropdown into master

What does this MR do and why?

This migrates GlDropdown in app/assets/javascripts/projects/commit/components/branches_dropdown.vue to GlCollapsibleListbox (see #384280 (closed)).

Migrates GlDropdown in app/assets/javascripts/projects/commit/components/projects_dropdown.vue to GlCollapsibleListbox (see #386027 (closed)).

Screenshots or screen recordings

Screenshots for branches_dropdown.vue for #384280 (closed)

Before After
Screenshot_2022-12-12_at_14.46.44 Screenshot_2022-12-13_at_16.44.39
Screenshot_2022-12-14_at_11.36.47 Screenshot_2022-12-14_at_11.26.18
Screenshot_2022-12-12_at_14.46.21 Screenshot_2023-01-11_at_13.44.06

Screenshots for projects_dropdown.vue for #386027 (closed)

Before After
Screenshot_2022-12-16_at_16.16.36 Screenshot_2022-12-16_at_16.11.06
Screenshot_2022-12-16_at_16.16.26 Screenshot_2023-01-11_at_13.44.56

How to set up and validate locally

  1. Navigate to project > commits
  2. Select a commit
  3. Select dropdown called "Options" in top right
  4. Select option "Cherry-pick"
  5. Migrated dropdown is the branch dropdown under heading "Pick into branch"

Details

As can be seen in the screen shots, there are some visual differences between the original and migrated dropdowns. I would love some feedback as to whether this is acceptable or not.

I reached out to @gitlab-org/manage/foundations/engineering with questions about this and that discussion can be seen in #384280 (comment 1207143572).

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #384280 (closed), #386027 (closed)

Edited by Ross Byrne

Merge request reports