Problem/Motivation

At the top of the page it says:

> Run the report to find out if there are detectable compatibility errors with the modules and themes installed on your site.Read more about preparing your site for Drupal 9.

There is a space missing between the two sentences.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#4 3277147-4.patch1.03 KBhmendes
#4 3277147-2_4_interdiff.txt1.04 KBhmendes
#2 3277147-2.patch1.03 KBhmendes

Comments

joachim created an issue. See original summary.

hmendes’s picture

hmendes commented
Status: Active » Needs review
StatusFileSize
new 3277147-2.patch1.03 KB

Sending a patch.

joachim’s picture

joachim commented
Status: Needs review » Needs work

Looks good, but I would put the space outside of the t() so that:

- translators don't accidentally mistake it for a typo and remove it
- existing translations don't break.

So:

... .= ' ' . t('...')
hmendes’s picture

hmendes commented
Status: Needs work » Needs review
StatusFileSize
new 3277147-2_4_interdiff.txt1.04 KB
new 3277147-4.patch1.03 KB

Oh, sure.
Thanks for the tip :D

joachim’s picture

joachim commented
Status: Needs review » Reviewed & tested by the community

Perfect, thanks!

  • 8c43244 committed on 8.x-3.x 
    Issue #3277147 by hmendes, joachim: Missing space in UI text
gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Status: Reviewed & tested by the community » Fixed

Looks good, agreed on the method of adding the space.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.