Make WordPress Core

Opened 2 months ago

Closed 2 weeks ago

Last modified 2 weeks ago

#50927 closed enhancement (wontfix)

Grammatical mistake in sentence on image editor screen

Reported by: simrandeep Owned by: SergeyBiryukov
Milestone: Priority: normal
Severity: normal Version:
Component: Media Keywords: has-patch close
Focuses: ui-copy Cc:


There is a grammatical mistake in the following sentence on the image editor screen - "The aspect ratio is the relationship between the width and height." Here it must either be "The aspect ratio is the relationship between width and height." or "The aspect ratio is the relationship between the width and the height." Either 'the' should be removed from width or it must be added before height as well.

Attachments (2)

grammatical-mistake-image-editor.png (101.9 KB) - added by simrandeep 2 months ago.
50927.diff (1.2 KB) - added by simrandeep 2 months ago.

Download all attachments as: .zip

Change History (12)

2 months ago

#1 @simrandeep
2 months ago

  • Keywords has-patch added

#2 @johnbillion
2 months ago

  • Component changed from General to Media
  • Milestone changed from Awaiting Review to Future Release

Welcome @simrandeep! Thanks for the ticket and the patch.

#3 @SergeyBiryukov
2 months ago

  • Focuses ui-copy added
  • Milestone changed from Future Release to 5.6

#4 @SergeyBiryukov
2 months ago

  • Owner set to SergeyBiryukov
  • Status changed from new to reviewing

#5 @SergeyBiryukov
2 months ago

The existing string appears to be more commonly used than the proposed one:

#6 @ayeshrajans
6 weeks ago

This is super difficult to google due to "the" being a word often ignored in English searches. I'm not a native English speaker, and English is my second language. What we have sounds simple and correct to me.

https://ell.stackexchange.com/q/13307/1468 - This Q&A also suggests that it's OK to omit the repetitive "the" as long as it's in same context.

Last edited 6 weeks ago by ayeshrajans (previous) (diff)

#7 @SergeyBiryukov
6 weeks ago

  • Keywords close added

This ticket was mentioned in Slack in #core-media by antpb. View the logs.

2 weeks ago

#9 @johnbillion
2 weeks ago

  • Milestone 5.6 deleted
  • Resolution set to wontfix
  • Status changed from reviewing to closed

Thanks for opening this @simrandeep but I'll close this off as it doesn't appear to be incorrect, and if it is it's still understandable. Changing it would cause extra work for translators.


#10 @simrandeep
2 weeks ago

No problem @johnbillion :) It was my duty to inform as I found it wrong but if you think it's correct then it's totally fine to close it.

Note: See TracTickets for help on using tickets.