Skip to content
Snippets Groups Projects

Put all sidebar icons in fixed width container

Merged Put all sidebar icons in fixed width container
18539-nav-alignment into master
Merged Annabel Gray requested to merge 18539-nav-alignment into master

What does this MR do?

Puts all icons (font awesome and custom) in fixed width container to fix weird alignment

Why was this MR needed?

Mix of font awesome icons & custom icons results in different icon widths

What are the relevant issue numbers?

Closes #18539 (closed)

Screenshots (if relevant)

Merge request reports

Pipeline #3516362 passed

Stage: prepare
Stage: test

Pipeline passed for d183e27e on 18539-nav-alignment

Merged by avatar (Apr 23, 2025 9:51pm UTC)

Loading

Pipeline #3517017 passed

Stage: prepare
Stage: test
Stage: post-test

Pipeline passed for c6cc7680 on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
app/views/layouts/nav/_dashboard.html.haml
1 1 %ul.nav.nav-sidebar
2 2 = nav_link(path: ['root#index', 'projects#trending', 'projects#starred', 'dashboard/projects#index'], html_options: {class: "#{project_tab_class} home"}) do
3 3 = link_to dashboard_projects_path, title: 'Projects', class: 'dashboard-shortcuts-projects' do
4 = navbar_icon('project')
  • Jacob Schatz Reassigned to @annabeldunstone

    Reassigned to @annabeldunstone

  • Annabel Gray Reassigned to @jschatz1

    Reassigned to @jschatz1

  • Jacob Schatz Status changed to merged

    Status changed to merged

  • Jacob Schatz mentioned in commit c6cc7680

    mentioned in commit c6cc7680

    • Please register or sign in to reply