Skip to content
Snippets Groups Projects

Decrease gap between MR content and sidebar

Merged Annabel Dunstone Gray requested to merge decrease-mr-sidebar-gap into master
All threads resolved!

What does this MR do and why?

  • Decrease gap between MR content and sidebar
  • Remove ...to administrator from "Report abuse" button (it looks buggy when it wraps, plus it opens a drawer that says "Report abuse to administrator")
  • Increase space below "Activity" header
  • Use tertiary button for "Resolve conflicts"

Screenshots or screen recordings

Before After
Screenshot_2023-03-21_at_3.19.53_PM Screenshot_2023-03-21_at_3.18.50_PM
Screenshot_2023-03-21_at_3.19.43_PM Screenshot_2023-03-21_at_3.19.12_PM
Screenshot_2023-03-21_at_3.18.12_PM Screenshot_2023-03-21_at_3.17.50_PM

How to set up and validate locally

  • Look at MR page (with :moved_mr_sidebar enabled)

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

#394879 (closed)

Edited by Annabel Dunstone Gray

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
  • mentioned in issue #398659 (closed)

  • added 1 commit

    • d0c3cca8 - Fix small UI inconsistencies in MR view

    Compare with previous version

  • Annabel Dunstone Gray resolved all threads

    resolved all threads

  • requested review from @nradina

  • Nataliia Radina approved this merge request

    approved this merge request

  • :wave: @nradina, thanks for approving this merge request.

    This is the first time the merge request is approved. To ensure full test coverage, a new pipeline will be started shortly.

    For more info, please refer to the following links:

  • Nataliia Radina requested review from @zcuddy

    requested review from @zcuddy

  • added 1 commit

    • 1e00ad1c - Fix small UI inconsistencies in MR view

    Compare with previous version

  • Annabel Dunstone Gray added 1086 commits

    added 1086 commits

    Compare with previous version

  • Zack Cuddy approved this merge request

    approved this merge request

  • Zack Cuddy resolved all threads

    resolved all threads

  • Zack Cuddy enabled an automatic merge when the pipeline for 087cfe7c succeeds

    enabled an automatic merge when the pipeline for 087cfe7c succeeds

  • merged

  • Zack Cuddy mentioned in commit 071ee305

    mentioned in commit 071ee305

  • added workflowstaging label and removed workflowcanary label

  • Annabel Dunstone Gray changed the description

    changed the description

  • Please register or sign in to reply
    Loading