Skip to content

Migrate issues and epics Bootstrap tabs to GlTabs

Miguel Rincon requested to merge 230748-tabs-haml-migrate-app-views-shared-issuable-_nav-html-haml into master

What does this MR do and why?

This change updates the classes used to render filter tabs for issues and epics filters.

Migrated tabs are compliant with Pajamas and don't use Bootstrap classes.

Ports 2 kind of elements:

  • %ul.nav-links.nav.nav-tabs to gl_tabs_nav
  • %li{ class: "active"} > link_to to gl_tab_link_to

Depends on !70919 (merged)

Screenshots or screen recordings

http://gdk.test:3000/groups/flightjs/-/epics

Before

old-epics

After

new-epics

http://gdk.test:3000/flightjs/flight/-/issues

Before

old-issues

After

new-issues

http://gdk.test:3000/flightjs/flight/-/merge_requests

Before

image

After

image

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

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 #230748 (closed)

Edited by Miguel Rincon

Merge request reports