diff options
| author | Fatih Acet <acetfatih@gmail.com> | 2017-09-18 22:54:43 +0000 |
|---|---|---|
| committer | Fatih Acet <acetfatih@gmail.com> | 2017-09-18 22:54:43 +0000 |
| commit | f0b089cf7881a0eed7f3aefe22d5b3006a39023d (patch) | |
| tree | 0b4ec0f23e3ef25a158d084003dbac1ec27405bc | |
| parent | 0223d84e797c914542882646c4faa97760c1a21c (diff) | |
| parent | 42d2dfad8f8c4c82d40ad5a300ebb8d0ab9d6a7a (diff) | |
| download | gitlab-ce-f0b089cf7881a0eed7f3aefe22d5b3006a39023d.tar.gz | |
Merge branch '37259-some-mr-ready-mobile-fixes' into 'master'
Fix MR widget with ready to merge buttons/controls at mobile breakpoint
Closes #37259
See merge request gitlab-org/gitlab-ce!14242
4 files changed, 16 insertions, 3 deletions
diff --git a/app/assets/javascripts/vue_merge_request_widget/components/states/mr_widget_ready_to_merge.js b/app/assets/javascripts/vue_merge_request_widget/components/states/mr_widget_ready_to_merge.js index 65187754009..1fb7150d20e 100644 --- a/app/assets/javascripts/vue_merge_request_widget/components/states/mr_widget_ready_to_merge.js +++ b/app/assets/javascripts/vue_merge_request_widget/components/states/mr_widget_ready_to_merge.js @@ -202,8 +202,8 @@ export default { <div class="mr-widget-body media"> <status-icon status="success" /> <div class="media-body"> - <div class="media space-children"> - <span class="btn-group"> + <div class="mr-widget-body-controls media space-children"> + <span class="btn-group append-bottom-5"> <button @click="handleMergeButtonClick()" :disabled="isMergeButtonDisabled" @@ -260,7 +260,7 @@ export default { </li> </ul> </span> - <div class="media-body space-children"> + <div class="media-body-wrap space-children"> <template v-if="isMergeAllowed()"> <label> <input diff --git a/app/assets/stylesheets/framework/media_object.scss b/app/assets/stylesheets/framework/media_object.scss index b573052c14a..89c561479cc 100644 --- a/app/assets/stylesheets/framework/media_object.scss +++ b/app/assets/stylesheets/framework/media_object.scss @@ -6,3 +6,7 @@ .media-body { flex: 1; } + +.media-body-wrap { + flex-grow: 1; +} diff --git a/app/assets/stylesheets/pages/merge_requests.scss b/app/assets/stylesheets/pages/merge_requests.scss index 439636fe026..09a14578dd3 100644 --- a/app/assets/stylesheets/pages/merge_requests.scss +++ b/app/assets/stylesheets/pages/merge_requests.scss @@ -356,6 +356,10 @@ } } +.mr-widget-body-controls { + flex-wrap: wrap; +} + .mr_source_commit, .mr_target_commit { margin-bottom: 0; diff --git a/changelogs/unreleased/37259-some-mr-ready-mobile-fixes.yml b/changelogs/unreleased/37259-some-mr-ready-mobile-fixes.yml new file mode 100644 index 00000000000..a00a41f567f --- /dev/null +++ b/changelogs/unreleased/37259-some-mr-ready-mobile-fixes.yml @@ -0,0 +1,5 @@ +--- +title: Fix MR ready to merge buttons/controls at mobile breakpoint +merge_request: 14242 +author: +type: fixed |