Skip to content

GitLab

  • Menu
Projects Groups Snippets
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • G gitlabhq1
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 21
    • Issues 21
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 12
    • Merge requests 12
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Packages & Registries
    • Packages & Registries
    • Package Registry
    • Infrastructure Registry
  • Analytics
    • Analytics
    • CI/CD
    • Repository
    • Value stream
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • gpt
  • large_projects
  • gitlabhq1
  • Issues
  • #8286

Closed
Open
Created Nov 11, 2014 by Administrator@rootOwner

Green "Accept merge request" is dangerous

Created by: coder-hugo

Since a few months we are using GitLab at work to mange our projects. For reviewing the code we use the merge request feature which is very helpful. But it happened more than once that the reviewer hit accidentally the button "Accept merge request" after modifying the assignee or the milestone. This happens because of several UX issues.

  1. You won't get notified that your changes regarding the assignee or the milestone have been saved automatically.
  2. There is no confirmation dialog before performing the auto-merge.
  3. The color of button is green. As the action behind this button usually will modify your master the button should be red or similar to give the user a fair waring that the action of this button is "dangerous".

IMHO the easiest solution for this issues would be the color change of the button. But I think that a confirmation dialog for such an action should be essential.

Assignee
Assign to
Time tracking