Added limit-container-width to breadcrumbs container

This fixes an issue where the content has a limited width but the breadcrumbs remained full width. Instead of creating a new variable specifically for the breadcrumbs, it just re-uses @content_class Closes #34573

Added limit-container-width to breadcrumbs container
aa0a6308 · Phil Hughes · 0b434daf · aa0a6308 · aa0a6308 · aa0a6308
Commit aa0a6308 authored Jul 11, 2017 by Phil Hughes
4 changed files
--- a/app/views/layouts/nav/_breadcrumbs.html.haml
+++ b/app/views/layouts/nav/_breadcrumbs.html.haml
@@ -2,7 +2,7 @@
 - hide_top_links = @hide_top_links || false

 %nav.breadcrumbs{ role: "navigation" }
-  .breadcrumbs-container{ class: container_class }
+  .breadcrumbs-container{ class: [container_class, @content_class] }
    .breadcrumbs-links.js-title-container
      - unless hide_top_links
        .title

--- a/app/views/projects/commit/show.html.haml
+++ b/app/views/projects/commit/show.html.haml
 - @no_container = true
 - container_class = !fluid_layout && diff_view == :inline ? 'container-limited' : ''
 - limited_container_width = fluid_layout ? '' : 'limit-container-width'
+- @content_class = limited_container_width
 - page_title        "#{@commit.title} (#{@commit.short_id})", "Commits"
 - page_description  @commit.description
 = render "projects/commits/head"

--- a/app/views/projects/show.html.haml
+++ b/app/views/projects/show.html.haml
 - @no_container = true
+- @content_class = "limit-container-width" unless fluid_layout
 - flash_message_container = show_new_nav? ? :new_global_flash : :flash_message

 = content_for :meta_tags do

--- a/app/views/projects/tree/show.html.haml
+++ b/app/views/projects/tree/show.html.haml
 - @no_container = true
+- @content_class = "limit-container-width" unless fluid_layout

 - page_title @path.presence || _("Files"), @ref
 = content_for :meta_tags do