Merge branch...

Merge branch 'ee-47752-buttons-on-new-file-page-wrap-outside-of-container-for-long-branch-names' into 'master'

Resolve "Buttons on new file page wrap outside of container for long branch names" - EE

See merge request gitlab-org/gitlab-ee!6811

app/assets/stylesheets/pages/editor.scss

@@ -36,10 +36,7 @@
     line-height: 35px;
     padding-top: 7px;
     padding-bottom: 7px;
-
-    .float-right {
-      height: 20px;
-    }
+    display: flex;
   }
 
   .editor-ref {
@@ -60,20 +57,18 @@
 
   .new-file-name {
     display: inline-block;
-    max-width: 450px;
+    max-width: 420px;
     float: left;
 
     @media(max-width: map-get($grid-breakpoints, lg)-1) {
-      width: 280px;
-    }
-
-    @media(max-width: map-get($grid-breakpoints, md)-1) {
       width: 180px;
     }
   }
 
   .file-buttons {
-    font-size: 0;
+    display: flex;
+    flex: 1;
+    justify-content: flex-end;
   }
 
   .select2 {
@@ -110,12 +105,11 @@
   }
 }
 
-@include media-breakpoint-down(xs) {
+
+@include media-breakpoint-down(sm) {
   .file-editor {
     .file-title {
-      .float-right {
-        height: auto;
-      }
+      display: block;
     }
 
     .new-file-name {
@@ -143,6 +137,10 @@
         }
       }
     }
+
+    .editor-ref {
+      max-width: 250px;
+    }
   }
 }
 
@@ -205,7 +203,6 @@
   .template-type-selector {
     display: inline-block;
     vertical-align: top;
-    font-family: $regular-font;
     margin-top: -5px;
 
     @media(max-width: map-get($grid-breakpoints, md)-1) {

app/views/projects/blob/_editor.html.haml

@@ -2,13 +2,12 @@
 
 .file-holder-bottom-radius.file-holder.file.append-bottom-default
   .js-file-title.file-title.clearfix{ data: { current_action: action } }
-    .editor-ref
+    .editor-ref.block-truncated
       = sprite_icon('fork', size: 12)
       = ref
-    %span.editor-file-name
-      - if current_action?(:edit) || current_action?(:update)
-        = text_field_tag 'file_path', (params[:file_path] || @path),
-                                      class: 'form-control new-file-path js-file-path-name-input'
+    - if current_action?(:edit) || current_action?(:update)
+      = text_field_tag 'file_path', (params[:file_path] || @path),
+                                    class: 'form-control new-file-path js-file-path-name-input'
 
     - if current_action?(:new) || current_action?(:create)
       %span.editor-file-name
@@ -16,7 +15,7 @@
       = text_field_tag 'file_name', params[:file_name], placeholder: "File name",
         required: true, class: 'form-control new-file-name js-file-path-name-input'
 
-    .float-right.file-buttons
+    .file-buttons
       = button_tag class: 'soft-wrap-toggle btn', type: 'button', tabindex: '-1' do
         %span.no-wrap
           = custom_icon('icon_no_wrap')

changelogs/unreleased/47752-buttons-on-new-file-page-wrap-outside-of-container-for-long-branch-names.yml

+---
+title: Fix buttons on the new file page wrapping outside of the container
+merge_request: 21015
+author:
+type: fixed