Update the Screenshots section of the default MR template
Proposal
Add that if a MR changes the UI then before
and after
screenshots must be added to match the recent documentation update !55791 (merged).
The MR must include Before and After screenshots if UI changes are made.
Problem
The current Screenshots
seems to suggest that it's entirely optional. I think we could make this more clear that it's required for changes to the UI.
Current message:
_These are strongly recommended to assist reviewers and reduce the time to merge your change._
<!--
Please include any relevant screenshots or screen recordings that will assist
reviewers and future readers. If you need help visually verifying the change,
please leave a comment and ping a GitLab reviewer, maintainer, or MR coach.
-->
Proposed changes:
_These are strongly recommended and must be included if your MR changes the UI._
<!--
Please include any relevant screenshots or screen recordings that will assist
reviewers and future readers. If you need help visually verifying the change,
please leave a comment and ping a GitLab reviewer, maintainer, or MR coach.
-->
Edited by Jiaan Louw