Skip to content

Conversation

amire80
Copy link
Collaborator

@amire80 amire80 commented Oct 9, 2025

Description (required)

I was going over all the strings and documenting them (see #6457), and I had a very hard time understand what this message does. I read the code and finally figured it out. I added qq documentation for it so now it's clearer, but I also think that the English message can be clearer:

  • "subsequent" changed to "the next" - shorter, easier word.
  • "media" changed to "item" - "media" could mean a lot of things, and "item" is clearer in this context.

Tests performed (required)

Checked in the emulator in Android Studio.

<string name="images_via_nearby_explanation">Images Uploaded via Nearby places are the images which are uploaded by discovering places on the map.</string>
<string name="thanks_received_explanation">This feature allows editors to send a Thank you notification to users who make useful edits – by using a small thank link on the history page or diff page.</string>
<string name="copy_image_caption_description">Copy to subsequent media</string>
<string name="copy_image_caption_description">Copy to the next item</string>
Copy link
Member

@nicolas-raoul nicolas-raoul Oct 10, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The button actually copies to all of the remaining items after that one.
I feel that the meaning could be improved.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! I should have thought of that. I changed it to "items". Is it better?

I was going over all the strings and documenting them (see commons-app#6457),
and I had a very hard time understand what this message does.
I read the code and finally figured it out. I added qq documentation
for it so now it's clearer, but I also think that the English
message can be clearer:
* "subsequent" changed to "the next" - shorter, easier word.
* "media" changed to "item" - "media" could mean a lot of things,
  and "item" is clearer in this context.
Copy link

✅ Generated APK variants!

@nicolas-raoul nicolas-raoul merged commit b8d340f into commons-app:main Oct 11, 2025
1 check passed
@amire80 amire80 deleted the subsequent branch October 15, 2025 19:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants