Make WordPress Core

Opened 5 years ago

Closed 5 years ago

#49235 closed defect (bug) (fixed)

Sentence correction in wp-admin/options-general.php file

Reported by: kinjaldalwadi's profile kinjaldalwadi Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 5.4 Priority: normal
Severity: normal Version: 5.4
Component: Administration Keywords: has-patch 2nd-opinion
Focuses: ui-copy Cc:

Description

Need to make correction in below sentence by adding "then":
Sentence : If you change this we will send you an email at your new address to confirm it.
Need correction : If you change this then we will send you an email at your new address to confirm it.

Screenshot : https://prnt.sc/qpvkqu

Attachments (4)

47087.patch (1.2 KB) - added by kinjaldalwadi 5 years ago.
wp-admin/options-general.php file
47104.patch (668 bytes) - added by kinjaldalwadi 5 years ago.
wp-admin/user-edit.php file
47104.2.patch (960 bytes) - added by kinjaldalwadi 5 years ago.
wp-admin/network/settings.php file
49235.4.diff (2.9 KB) - added by donmhico 5 years ago.

Download all attachments as: .zip

Change History (9)

@kinjaldalwadi
5 years ago

wp-admin/options-general.php file

#1 @SergeyBiryukov
5 years ago

  • Component changed from General to Administration
  • Focuses ui-copy added
  • Keywords has-patch 2nd-opinion added; Sentence correction settings options-general.php removed

Hi there, welcome to WordPress Trac! Thanks for the ticket and the patch.

This sentence looks correct to me as is, though perhaps would be better with a comma.

Adding ui-copy focus to get some more eyes on it.

#2 @SergeyBiryukov
5 years ago

#49267 was marked as a duplicate.

#3 @SergeyBiryukov
5 years ago

  • Milestone changed from Awaiting Review to 5.4
  • Owner set to SergeyBiryukov
  • Status changed from new to reviewing

@kinjaldalwadi
5 years ago

wp-admin/user-edit.php file

@kinjaldalwadi
5 years ago

wp-admin/network/settings.php file

@donmhico
5 years ago

#4 @donmhico
5 years ago

Thanks for the patch @kinjaldalwadi. I combined all of the patches in a single patch 49235.4.diff.

#5 @SergeyBiryukov
5 years ago

  • Resolution set to fixed
  • Status changed from reviewing to closed

In 47143:

Administration: Add punctuation to "If you change this, we will send you an email" message on General Settings, Network Settings, and Profile screens.

Props kinjaldalwadi, donmhico.
Fixes #49235.

Note: See TracTickets for help on using tickets.