Remove "currently" from some UI text in MRs
The following discussion from !100102 (merged) should be addressed:
-
@aqualls started a discussion: @thomasrandolph This works, and matches what's around it. If it were in the documentation, we'd need to handle this statement differently - I'd ask you to take out
currently
- but I think it's more important here to 1) stay consistent and 2) keep you moving.Approving.
Additionally, there is inconsistency in punctuation use in these tooltips.
-
@pgascouvaillancourt started a discussion: nitpick(non-blocking): The punctuation is inconsistent. Some tooltips' text ends with a dot and other don't. It'd be neat if we could settle on one format and apply it everywhere.
Proposed Solution
We should update the text for commenting on lines to not include "currently."
We should also pick a punctuation format for tooltips (I believe the standard is no trailing punctuation) and update them all to match.