Skip to content

Fix formatting on learn more link

What does this MR do and why?

Moves the full stop outside of "Learn More" links.

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

Before After
Squash message Screenshot_2023-06-19_at_16.24.40 Screenshot_2023-06-19_at_13.09.11
Squash and merge message Screenshot_2023-06-19_at_16.25.12 Screenshot_2023-06-19_at_16.22.13
Merge message Screenshot_2023-06-19_at_16.25.28 Screenshot_2023-06-19_at_16.22.36

How to set up and validate locally

  1. Navigate to a MR
  2. In app/assets/javascripts/vue_merge_request_widget/components/states/ready_to_merge.vue, change Line 548 to the follow so that the help messages are displayed.
              <div v-if="true" class="gl-w-full" data-testid="edit_commit_message">
  1. to test each of the messages, the easiest way to to change line 207 to if (true) { for "merge and squash" and then instead of 207, change 210 to be if (true) { for squash message

Numbered steps to set up and validate the change are strongly suggested.

MR acceptance checklist

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

Related to #414679 (closed)

Merge request reports

Loading