gitlab-org--gitlab-foss/app/views/profiles
Fatih Acet 88fa5916ff Merge branch '22343-honor-user-fixed-layout-pref' into 'master'
Ensure the 'fixed layout' preference is honored whenever possible

## What does this MR do?

Currently, when viewing any Merge Request the user's fixed-layout preference is overridden if they have set "Side-by-Side" view as their preference when viewing diffs.  This makes sense if they are currently viewing a diff, but this is confusing when their layout preference is overridden while they are on another tab (i.e. "Discussion" or "Builds").

This MR moves all responsibility for overriding the fixed layout from the Ruby page layout helper into frontend JavaScript where it is only applied when needed.

## Are there points in the code the reviewer needs to double check?

Check that nothing broke for users which have their layout preference set to "Fluid" as well as those which have it set to "Fixed".  I've already done this but double checking is always good idea 😄.

## Screenshots (if relevant)

![side-by-side-toggle-fixed](/uploads/033dc73e70b73da5692b75606733c938/side-by-side-toggle-fixed.gif)

## Does this MR meet the acceptance criteria?

- [x] [CHANGELOG](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CHANGELOG) entry added
- Tests
  - [ ] All builds are passing
- [x] Conform by the [style guides](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CONTRIBUTING.md#style-guides)
- [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)

## What are the relevant issue numbers?

Closes #22343 

## Other Notes

As @lbennett noted in #22343, this is something of a band-aid.  It fixed most of the issue, but we still need to have a discussion about whether or not is acceptable to override user layout preferences for "Side-by-Side" views in general.  At least this MR limits the scope of this behavior to a single tab within an MR or merge conflict page.

See merge request !6422
2016-10-04 11:58:15 +00:00
..
accounts Remove provider path, replace with dynamic path. 2016-08-03 11:50:38 -06:00
emails
keys Restore SSH Key title auto-population behavior 2016-09-13 11:49:55 -04:00
notifications
passwords
personal_access_tokens Refactor find_for_git_client method to not use assignment in conditionals and syntax fixes. 2016-08-17 17:21:18 -05:00
preferences ensure the 'fixed layout' preference is honored whenever possible 2016-09-20 00:51:00 -05:00
two_factor_auths Allow naming U2F devices. 2016-08-18 22:12:02 +05:30
_event_table.html.haml
_head.html.haml
audit_log.html.haml
show.html.haml Add organization field to user profile 2016-09-27 14:04:39 +03:00
update_username.js.haml Fix spelling: sucessfully -> successfully 2016-09-15 19:27:49 -07:00