diff options
author | Jacob Schatz <jschatz@gitlab.com> | 2016-08-25 19:12:33 +0000 |
---|---|---|
committer | Ruben Davila <rdavila84@gmail.com> | 2016-09-13 20:55:23 -0500 |
commit | ab4b65a85eaddb830d5c6f9f302a283c6f5f24a8 (patch) | |
tree | f49672b3d3b9404000f4b678e0847f8b938010bc | |
parent | b015ff3938a5ebc3d931f84ed55385d5de485ee9 (diff) | |
download | gitlab-ce-ab4b65a85eaddb830d5c6f9f302a283c6f5f24a8.tar.gz |
Merge branch 'fix-pipeline-horizontal-scrolling' into 'master'
Fix unnecessary horizontal scroll area in pipeline visualizations
## What does this MR do?
Removes a min-width attribute for the build pipeline visualization wrapper
## Are there points in the code the reviewer needs to double check?
I am curious why this line was there in the first place. Perhaps I'm just not seeing it.
## Why was this MR needed?
I was noticing an unnecessary horizontal scroll bar when viewing pipeline visualizations embedded in merge request builds tab:
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/5627/builds
## What are the relevant issue numbers?
Line appears to have been introduced in !5742
## Screenshots (if relevant)
![Screen_Shot_2016-08-24_at_11.15.13_PM](/uploads/e395ed8d4415187735d06b8386fe4633/Screen_Shot_2016-08-24_at_11.15.13_PM.png)
## Does this MR meet the acceptance criteria?
As this is a very simple cosmetic change, I don't think it warrants inclusion in the changelog, documentation, or tests.
- [x] Branch has no merge conflicts with `master` (if you do - rebase it please)
- [x] [Squashed related commits together](https://git-scm.com/book/en/Git-Tools-Rewriting-History#Squashing-Commits)
See merge request !6005
-rw-r--r-- | app/assets/stylesheets/pages/pipelines.scss | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/app/assets/stylesheets/pages/pipelines.scss b/app/assets/stylesheets/pages/pipelines.scss index a5d4ebc404b..0dcf61dd2dd 100644 --- a/app/assets/stylesheets/pages/pipelines.scss +++ b/app/assets/stylesheets/pages/pipelines.scss @@ -264,7 +264,6 @@ .pipeline-visualization { position: relative; - min-width: 1220px; ul { padding: 0; |