Skip to content

Remove "currently" from some UI text in MRs

Thomas Randolph requested to merge tor/maintenance/line-comment-text into master

What does this MR do and why?

For #376744 (closed)

  • Removes the word "currently" from some tooltips on the MR diff view.
  • Also removes some periods from the tooltips, so that all of the modified tooltips remain unpunctuated.

Screenshots or screen recordings

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

How to set up and validate locally

It's not very easy to validate this locally because of the special-case symlink tooltip.

  1. Create an MR where the changes diff includes:
    • A file with no newline at the end
    • AND a removal of a normal file and the addition of a symlink of the same name
    • OR a removal of a symlink and the addition of a file of the same name
  2. Hover over:
    1. The automatically inserted line in the file without a newline at the end
    2. Any line in your symlink/file pair

MR acceptance checklist

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

Edited by Thomas Randolph

Merge request reports