Skip to content
Snippets Groups Projects

Remember user's inline/tabular diff view preference in a cookie

Merged username-removed-128633 requested to merge kkm/gitlab-ce-user-diff-view-pref-cookie into master

Originally opened at !1677 (closed) by @kkm.


As per #3071 (closed), some users (we have a local EE installation) prefer 2-column view in diff. In this MR I add an implementation for this feature, using a cookie.

Fixes #3071 (closed).

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Reassigned to @rymai

  • Added 2 commits:

    • cf1349ad - Remember user's inline/tabular diff view preference in a cookie
    • f6d816f9 - Add a controller test for the new 'diff_view' cookie
  • @dzaporozhets Changelog is good now! ;)

  • @rspeicher can you review please?

  • @rymai, @dzaporozhets: Thank you much guys for the help with the tests. I really know very little Ruby, and have near absolute zero experience writing tests for the Rails.

    Edited by username-removed-230683
  • Robert Speicher Added 2 commits:

    Added 2 commits:

    • 24f048be - Update CHANGELOG
    • 551ce0f2 - Add spec for assigning view param from cookie
  • Updated changelog entry and added a spec for assigning the view param from the cookie.

  • @rymai Rest LGTM, can merge when green.

  • mentioned in commit 9fdd605f

  • username-removed-128633 Status changed to merged

    Status changed to merged

  • Please register or sign in to reply
    Loading