Narrow the meaning and visibility of "Lock <object>"
Purpose
Issues and MRs currently provide an action called "Lock ". The actual outcome of this action is only that discussions are locked to only project members. By presenting this in an object-level placement, and referencing the entire object, it's implied that the entire object would be locked, possibly including editing data. We should be more specific about what this action does.
Current
In the revised sidebar (behind a FF) this is in the top actions menu, in the legacy sidebar this is in the sidebar.
Sidebar | Top actions |
---|---|
![]() |
![]() |
Once applied the lock status appears in both the header and in the activity section, replacing the comment box:
Header (MR) | Header (Issue) | Activity | Activity (w/ access) |
---|---|---|---|
![]() |
![]() |
![]() |
![]() |
Proposal
MVC1: Rename this action "Lock discussion". This mirrors how we already reference this in the docs.
MVC2: For MR code comments update the text Please register or sign in to reply
to Discussions are locked. Please register or sign in to reply
Additional consideration: Remove the header "Lock" badge. The lock status remains visible in the activity area, where it's most applicable and more clearly communicates the locked meaning.