Make WordPress Core

Opened 4 years ago

Closed 3 years ago

Last modified 3 years ago

#50927 closed enhancement (wontfix)

Grammatical mistake in sentence on image editor screen

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

Description

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 4 years ago.
50927.diff (1.2 KB) - added by simrandeep 4 years ago.

Download all attachments as: .zip

Change History (12)

@simrandeep
4 years ago

#1 @simrandeep
4 years ago

  • Keywords has-patch added

#2 @johnbillion
4 years 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
4 years ago

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

#4 @SergeyBiryukov
4 years ago

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

#5 @SergeyBiryukov
4 years ago

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

#6 @ayeshrajans
4 years 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 4 years ago by ayeshrajans (previous) (diff)

#7 @SergeyBiryukov
4 years ago

  • Keywords close added

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


3 years ago

#9 @johnbillion
3 years 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.

Cheers!

#10 @simrandeep
3 years 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.