From bb330c1771a41ffeabbde24cdcaf14c218c74e16 Mon Sep 17 00:00:00 2001 From: "Amir E. Aharoni" Date: Mon, 13 Oct 2025 04:57:27 -0400 Subject: [PATCH 01/13] Change "actioned" to "handled" in translatable strings (#6498) "actioned" is not so standard in English as a verb. "handled" sounds more appropriate. --- app/src/main/res/values/strings.xml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml index 0c0ae3a14..45e59cf3d 100644 --- a/app/src/main/res/values/strings.xml +++ b/app/src/main/res/values/strings.xml @@ -788,8 +788,8 @@ Upload your first media by tapping on the add button. Unmark as not for upload Marking as not for upload Unmarking as not for upload - Show already actioned pictures - Hiding already actioned pictures + Show already handled pictures + Hiding already handled pictures No more images found This image is already uploaded Can not select this image for upload From d0965206cd0c9217ffa7352ed0c1995265b2462d Mon Sep 17 00:00:00 2001 From: "Amir E. Aharoni" Date: Mon, 13 Oct 2025 05:05:26 -0400 Subject: [PATCH 02/13] Cleanup whitespace in the custom_selector_info_text2 string (#6496) In the current state, it appears confusingly on translatewiki, with a space in the beginning of a line. This patch changes it to just two linebreaks. --- app/src/main/res/values/strings.xml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml index 45e59cf3d..f9270a976 100644 --- a/app/src/main/res/values/strings.xml +++ b/app/src/main/res/values/strings.xml @@ -736,7 +736,7 @@ Upload your first media by tapping on the add button. Back Welcome to Custom Picture Selector This picker shows you which pictures you have already uploaded to Commons. - Unlike the picture on the left, the picture on the right has the Commons logo indicating it is already uploaded. \n Touch and hold for image preview. + Unlike the picture on the left, the picture on the right has the Commons logo indicating it is already uploaded.\n\nTouch and hold for image preview. Awesome This image has already been uploaded to Commons. For technical reasons, the app can\'t reliably upload more than %1$d pictures at once. The upload limit of %1$d has been exceeded by %2$d. From fbd28a05649b542c02e74198b2136cd5e5394151 Mon Sep 17 00:00:00 2001 From: "Amir E. Aharoni" Date: Mon, 13 Oct 2025 05:09:43 -0400 Subject: [PATCH 03/13] Change capitalization of "Add Location" (#6493) This makes it consistent with "Edit Location" and "Edit Image", which are used in the same screen. --- app/src/main/res/values/strings.xml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml index f9270a976..a0a4ca260 100644 --- a/app/src/main/res/values/strings.xml +++ b/app/src/main/res/values/strings.xml @@ -767,7 +767,7 @@ Upload your first media by tapping on the add button. Location data helps Wiki editors find your picture, making it much more useful.\nYour recent uploads have no location.\nWe suggest you turn on location in your camera app\'s settings.\nThank you for uploading! No location found How about adding the place where this image was taken?\nLocation data helps Wiki editors find your picture, making it much more useful.\nThank you! - Add location + Add Location Please remove from this email any information that you are not comfortable sharing publicly. Also, please be aware that your email address with which you are posting, and the associated name and profile picture, will be visible publicly. Details Achievements are only available in the prod flavor. Please check the developer documentation. From 1e51c4c5d02c6086b38bc71bbc9697e01adcb355 Mon Sep 17 00:00:00 2001 From: "Amir E. Aharoni" Date: Mon, 13 Oct 2025 05:13:22 -0400 Subject: [PATCH 04/13] Remove the arrow next to "Add location" (#6491) This resolves #6489 using the "remove arrow" method. --- app/src/main/res/drawable/ic_arrow_16dp.xml | 4 ---- .../res/layout/fragment_upload_media_detail_fragment.xml | 7 ------- 2 files changed, 11 deletions(-) delete mode 100644 app/src/main/res/drawable/ic_arrow_16dp.xml diff --git a/app/src/main/res/drawable/ic_arrow_16dp.xml b/app/src/main/res/drawable/ic_arrow_16dp.xml deleted file mode 100644 index d99c0b5e4..000000000 --- a/app/src/main/res/drawable/ic_arrow_16dp.xml +++ /dev/null @@ -1,4 +0,0 @@ - - - diff --git a/app/src/main/res/layout/fragment_upload_media_detail_fragment.xml b/app/src/main/res/layout/fragment_upload_media_detail_fragment.xml index cccf94c7b..e6f9e79e7 100644 --- a/app/src/main/res/layout/fragment_upload_media_detail_fragment.xml +++ b/app/src/main/res/layout/fragment_upload_media_detail_fragment.xml @@ -149,13 +149,6 @@ android:textColor="#2D8BA4" android:textSize="@dimen/normal_text" android:textStyle="bold" /> - - From 91ea4a6e7b3bba7cfd6bd1506d2614a31f97510c Mon Sep 17 00:00:00 2001 From: "Amir E. Aharoni" Date: Mon, 13 Oct 2025 05:43:59 -0400 Subject: [PATCH 05/13] Rephrase images_featured_explanation (#6484) Make the text of the panel consistent with its title. The title is "Featured images", so the text should use the same term. Also move this resource next to the title, to make it easier for the translators. --- app/src/main/res/values/strings.xml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml index a0a4ca260..183c683da 100644 --- a/app/src/main/res/values/strings.xml +++ b/app/src/main/res/values/strings.xml @@ -133,6 +133,7 @@ Settings Sign Up Featured Images + Featured images are contributions by highly skilled photographers and illustrators that the Wikimedia Commons community has chosen as some of the highest quality on the site. Custom Selector Category Peer Review @@ -529,7 +530,6 @@ Upload your first media by tapping on the add button. Error occurred while picking images Please wait… - Featured pictures are images from highly skilled photographers and illustrators that the Wikimedia Commons community has chosen as some of the highest quality on the site. Images Uploaded via Nearby places are the images which are uploaded by discovering places on the map. 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. Copy to the next items From b6014b017ca5a91fcee03fea7e785f3281d3d933 Mon Sep 17 00:00:00 2001 From: "translatewiki.net" Date: Mon, 13 Oct 2025 14:03:10 +0200 Subject: [PATCH 06/13] Localisation updates from https://translatewiki.net. --- app/src/main/res/values-az/error.xml | 2 +- app/src/main/res/values-qq/error.xml | 2 ++ 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/app/src/main/res/values-az/error.xml b/app/src/main/res/values-az/error.xml index 6e9503c43..53a971367 100644 --- a/app/src/main/res/values-az/error.xml +++ b/app/src/main/res/values-az/error.xml @@ -5,7 +5,7 @@ * Nemoralis --> - Nasazlıq + Commons çökdü Uups. Nəsə düzgün çalışmır! Nə etdiyinizi bizə deyin, sonra e-poçt vasitəsilə bizimlə paylaşın. Bu, bizə bunu düzəltməyə kömək edəcək! Təşəkkürlər! diff --git a/app/src/main/res/values-qq/error.xml b/app/src/main/res/values-qq/error.xml index e48ac7c9d..9fffe2e0f 100644 --- a/app/src/main/res/values-qq/error.xml +++ b/app/src/main/res/values-qq/error.xml @@ -1,9 +1,11 @@ Title of dialog to show when the app crashes + Shown when the application crashed. Prompt asking people to enter info about what they were doing when the app crashed Toast to be displayed once someone sends an error report thanking them.\n{{Identical|Thank you}} From 0c244f369c8b01889357994ac992a1819a240b56 Mon Sep 17 00:00:00 2001 From: "Amir E. Aharoni" Date: Mon, 13 Oct 2025 08:54:32 -0400 Subject: [PATCH 07/13] Replace android.R.string.* with R.string (#6499) All these messages are not really necessary because the app has its own localizations, and android.R.string sometimes doesn't display the localized string. Resolves #6470. --- .../nrw/commons/contributions/ContributionController.kt | 4 ++-- .../free/nrw/commons/description/DescriptionEditActivity.kt | 2 +- .../fr/free/nrw/commons/explore/map/ExploreMapFragment.kt | 4 ++-- .../explore/recentsearches/RecentSearchesFragment.kt | 6 +++--- .../free/nrw/commons/location/LocationPermissionsHelper.kt | 4 ++-- app/src/main/java/fr/free/nrw/commons/quiz/QuizChecker.kt | 2 +- .../java/fr/free/nrw/commons/quiz/QuizResultActivity.kt | 2 +- .../main/java/fr/free/nrw/commons/review/ReviewActivity.kt | 4 ++-- .../main/java/fr/free/nrw/commons/upload/UploadActivity.kt | 4 ++-- .../commons/upload/categories/UploadCategoriesFragment.kt | 2 +- .../fr/free/nrw/commons/upload/depicts/DepictsFragment.kt | 2 +- .../upload/mediaDetails/UploadMediaDetailFragment.kt | 4 ++-- .../main/java/fr/free/nrw/commons/utils/PermissionUtils.kt | 4 ++-- .../commons/description/DescriptionEditActivityUnitTest.kt | 4 ++-- 14 files changed, 24 insertions(+), 24 deletions(-) diff --git a/app/src/main/java/fr/free/nrw/commons/contributions/ContributionController.kt b/app/src/main/java/fr/free/nrw/commons/contributions/ContributionController.kt index 29267452b..b9532a12e 100644 --- a/app/src/main/java/fr/free/nrw/commons/contributions/ContributionController.kt +++ b/app/src/main/java/fr/free/nrw/commons/contributions/ContributionController.kt @@ -180,8 +180,8 @@ class ContributionController @Inject constructor(@param:Named("default_preferenc showAlertDialog( activity, activity.getString(R.string.location_permission_title), activity.getString(R.string.in_app_camera_location_permission_rationale), - activity.getString(android.R.string.ok), - activity.getString(android.R.string.cancel), + activity.getString(R.string.ok), + activity.getString(R.string.cancel), { createDialogsAndHandleLocationPermissions( activity, diff --git a/app/src/main/java/fr/free/nrw/commons/description/DescriptionEditActivity.kt b/app/src/main/java/fr/free/nrw/commons/description/DescriptionEditActivity.kt index 89d43845b..b1f1b7f9b 100644 --- a/app/src/main/java/fr/free/nrw/commons/description/DescriptionEditActivity.kt +++ b/app/src/main/java/fr/free/nrw/commons/description/DescriptionEditActivity.kt @@ -150,7 +150,7 @@ class DescriptionEditActivity : this, getString(titleStringID), getString(messageStringId), - getString(android.R.string.ok), + getString(R.string.ok), null ) } diff --git a/app/src/main/java/fr/free/nrw/commons/explore/map/ExploreMapFragment.kt b/app/src/main/java/fr/free/nrw/commons/explore/map/ExploreMapFragment.kt index cad6bd057..a1bae09fb 100644 --- a/app/src/main/java/fr/free/nrw/commons/explore/map/ExploreMapFragment.kt +++ b/app/src/main/java/fr/free/nrw/commons/explore/map/ExploreMapFragment.kt @@ -140,8 +140,8 @@ class ExploreMapFragment : CommonsDaggerSupportFragment(), ExploreMapContract.Vi requireActivity(), requireActivity().getString(R.string.location_permission_title), requireActivity().getString(R.string.location_permission_rationale_explore), - requireActivity().getString(android.R.string.ok), - requireActivity().getString(android.R.string.cancel), + requireActivity().getString(R.string.ok), + requireActivity().getString(R.string.cancel), { askForLocationPermission() }, null, null diff --git a/app/src/main/java/fr/free/nrw/commons/explore/recentsearches/RecentSearchesFragment.kt b/app/src/main/java/fr/free/nrw/commons/explore/recentsearches/RecentSearchesFragment.kt index c0f1bd5db..e7903c9ed 100644 --- a/app/src/main/java/fr/free/nrw/commons/explore/recentsearches/RecentSearchesFragment.kt +++ b/app/src/main/java/fr/free/nrw/commons/explore/recentsearches/RecentSearchesFragment.kt @@ -67,10 +67,10 @@ class RecentSearchesFragment : CommonsDaggerSupportFragment() { private fun showDeleteRecentAlertDialog(context: Context) { AlertDialog.Builder(context) .setMessage(getString(R.string.delete_recent_searches_dialog)) - .setPositiveButton(android.R.string.yes) { dialog: DialogInterface, _: Int -> + .setPositiveButton(R.string.yes) { dialog: DialogInterface, _: Int -> setDeleteRecentPositiveButton(context, dialog) } - .setNegativeButton(android.R.string.no, null) + .setNegativeButton(R.string.no, null) .setCancelable(false) .create() .show() @@ -102,7 +102,7 @@ class RecentSearchesFragment : CommonsDaggerSupportFragment() { setDeletePositiveButton(context, dialog, position) } ) - .setNegativeButton(android.R.string.cancel, null) + .setNegativeButton(R.string.cancel, null) .setCancelable(false) .create() .show() diff --git a/app/src/main/java/fr/free/nrw/commons/location/LocationPermissionsHelper.kt b/app/src/main/java/fr/free/nrw/commons/location/LocationPermissionsHelper.kt index fefb59adb..47b4165ad 100644 --- a/app/src/main/java/fr/free/nrw/commons/location/LocationPermissionsHelper.kt +++ b/app/src/main/java/fr/free/nrw/commons/location/LocationPermissionsHelper.kt @@ -64,8 +64,8 @@ class LocationPermissionsHelper( activity, activity.getString(dialogTitleResource), activity.getString(dialogTextResource), - activity.getString(android.R.string.ok), - activity.getString(android.R.string.cancel), + activity.getString(R.string.ok), + activity.getString(R.string.cancel), { ActivityCompat.requestPermissions( activity, diff --git a/app/src/main/java/fr/free/nrw/commons/quiz/QuizChecker.kt b/app/src/main/java/fr/free/nrw/commons/quiz/QuizChecker.kt index 0183056a6..5362f1542 100644 --- a/app/src/main/java/fr/free/nrw/commons/quiz/QuizChecker.kt +++ b/app/src/main/java/fr/free/nrw/commons/quiz/QuizChecker.kt @@ -151,7 +151,7 @@ class QuizChecker @Inject constructor( activity.getString(R.string.quiz), activity.getString(R.string.quiz_alert_message, revertPercentageForMessage), activity.getString(R.string.about_translate_proceed), - activity.getString(android.R.string.cancel), + activity.getString(R.string.cancel), { startQuizActivity(activity) }, null ) diff --git a/app/src/main/java/fr/free/nrw/commons/quiz/QuizResultActivity.kt b/app/src/main/java/fr/free/nrw/commons/quiz/QuizResultActivity.kt index 6979edd15..15884146d 100644 --- a/app/src/main/java/fr/free/nrw/commons/quiz/QuizResultActivity.kt +++ b/app/src/main/java/fr/free/nrw/commons/quiz/QuizResultActivity.kt @@ -193,7 +193,7 @@ class QuizResultActivity : AppCompatActivity() { alertadd.setPositiveButton(R.string.about_translate_proceed) { dialog, _ -> shareScreen(screenshot) } - alertadd.setNegativeButton(android.R.string.cancel) { dialog, _ -> + alertadd.setNegativeButton(R.string.cancel) { dialog, _ -> dialog.cancel() } alertadd.show() diff --git a/app/src/main/java/fr/free/nrw/commons/review/ReviewActivity.kt b/app/src/main/java/fr/free/nrw/commons/review/ReviewActivity.kt index 1d26eacae..dccb77af1 100644 --- a/app/src/main/java/fr/free/nrw/commons/review/ReviewActivity.kt +++ b/app/src/main/java/fr/free/nrw/commons/review/ReviewActivity.kt @@ -238,7 +238,7 @@ class ReviewActivity : BaseActivity() { this, getString(R.string.skip_image).uppercase(Locale.ROOT), getString(R.string.skip_image_explanation), - getString(android.R.string.ok), + getString(R.string.ok), null, null, null @@ -250,7 +250,7 @@ class ReviewActivity : BaseActivity() { this, getString(R.string.title_activity_review), getString(R.string.review_image_explanation), - getString(android.R.string.ok), + getString(R.string.ok), null, null, null diff --git a/app/src/main/java/fr/free/nrw/commons/upload/UploadActivity.kt b/app/src/main/java/fr/free/nrw/commons/upload/UploadActivity.kt index 799d5b0f1..66e0257f6 100644 --- a/app/src/main/java/fr/free/nrw/commons/upload/UploadActivity.kt +++ b/app/src/main/java/fr/free/nrw/commons/upload/UploadActivity.kt @@ -446,7 +446,7 @@ class UploadActivity : BaseActivity(), UploadContract.View, UploadBaseFragment.C this, getString(R.string.storage_permissions_denied), getString(R.string.unable_to_share_upload_item), - getString(android.R.string.ok) + getString(R.string.ok) ) { finish() } } else { showAlertDialog( @@ -455,7 +455,7 @@ class UploadActivity : BaseActivity(), UploadContract.View, UploadBaseFragment.C getString( R.string.write_storage_permission_rationale_for_image_share ), - getString(android.R.string.ok) + getString(R.string.ok) ) { checkStoragePermissions() } } } diff --git a/app/src/main/java/fr/free/nrw/commons/upload/categories/UploadCategoriesFragment.kt b/app/src/main/java/fr/free/nrw/commons/upload/categories/UploadCategoriesFragment.kt index 798ab74fe..ee6af7bb6 100644 --- a/app/src/main/java/fr/free/nrw/commons/upload/categories/UploadCategoriesFragment.kt +++ b/app/src/main/java/fr/free/nrw/commons/upload/categories/UploadCategoriesFragment.kt @@ -117,7 +117,7 @@ class UploadCategoriesFragment : UploadBaseFragment(), CategoriesContract.View { requireActivity(), getString(R.string.categories_activity_title), getString(R.string.categories_tooltip), - getString(android.R.string.ok), + getString(R.string.ok), null ) } diff --git a/app/src/main/java/fr/free/nrw/commons/upload/depicts/DepictsFragment.kt b/app/src/main/java/fr/free/nrw/commons/upload/depicts/DepictsFragment.kt index 484750410..5dcc2bf86 100644 --- a/app/src/main/java/fr/free/nrw/commons/upload/depicts/DepictsFragment.kt +++ b/app/src/main/java/fr/free/nrw/commons/upload/depicts/DepictsFragment.kt @@ -116,7 +116,7 @@ class DepictsFragment : UploadBaseFragment(), DepictsContract.View { requireActivity(), getString(R.string.depicts_step_title), getString(R.string.depicts_tooltip), - getString(android.R.string.ok), + getString(R.string.ok), null ) } diff --git a/app/src/main/java/fr/free/nrw/commons/upload/mediaDetails/UploadMediaDetailFragment.kt b/app/src/main/java/fr/free/nrw/commons/upload/mediaDetails/UploadMediaDetailFragment.kt index acf55670e..b3b067948 100644 --- a/app/src/main/java/fr/free/nrw/commons/upload/mediaDetails/UploadMediaDetailFragment.kt +++ b/app/src/main/java/fr/free/nrw/commons/upload/mediaDetails/UploadMediaDetailFragment.kt @@ -303,7 +303,7 @@ class UploadMediaDetailFragment : UploadBaseFragment(), UploadMediaDetailsContra requireActivity(), getString(titleStringID), getString(messageStringId), - getString(android.R.string.ok), + getString(R.string.ok), null ) } @@ -919,4 +919,4 @@ class UploadMediaDetailFragment : UploadBaseFragment(), UploadMediaDetailsContra const val UPLOADABLE_FILE: String = "uploadable_file" const val UPLOAD_MEDIA_DETAILS: String = "upload_media_detail_adapter" } -} \ No newline at end of file +} diff --git a/app/src/main/java/fr/free/nrw/commons/utils/PermissionUtils.kt b/app/src/main/java/fr/free/nrw/commons/utils/PermissionUtils.kt index df3b33bf6..5bcc9d1b2 100644 --- a/app/src/main/java/fr/free/nrw/commons/utils/PermissionUtils.kt +++ b/app/src/main/java/fr/free/nrw/commons/utils/PermissionUtils.kt @@ -209,8 +209,8 @@ object PermissionUtils { activity, activity.getString(rationaleTitle), activity.getString(rationaleMessage), - activity.getString(android.R.string.ok), - activity.getString(android.R.string.cancel), + activity.getString(R.string.ok), + activity.getString(R.string.cancel), { if (activity is UploadActivity) { activity.isShowPermissionsDialog = true diff --git a/app/src/test/kotlin/fr/free/nrw/commons/description/DescriptionEditActivityUnitTest.kt b/app/src/test/kotlin/fr/free/nrw/commons/description/DescriptionEditActivityUnitTest.kt index be3b7e8e3..5b5dfd7dd 100644 --- a/app/src/test/kotlin/fr/free/nrw/commons/description/DescriptionEditActivityUnitTest.kt +++ b/app/src/test/kotlin/fr/free/nrw/commons/description/DescriptionEditActivityUnitTest.kt @@ -193,8 +193,8 @@ class DescriptionEditActivityUnitTest { method.isAccessible = true method.invoke( activity, - android.R.string.ok, - android.R.string.ok, + R.string.ok, + R.string.ok, ) val dialog: AlertDialog = ShadowAlertDialog.getLatestDialog() as AlertDialog assertEquals(dialog.isShowing, true) From a1c5974e936c369e099232b8930732d057491e72 Mon Sep 17 00:00:00 2001 From: "Amir E. Aharoni" Date: Tue, 14 Oct 2025 04:54:54 -0400 Subject: [PATCH 08/13] Fix depicts and categories pickers for RTL languages (#6503) This fixes the layouts to work in both left to right (LTR) and right to left (RTL) languages. Also replace two hard-coded strings in the depicts picker with proper string resources. Fixes #6502. --- .../res/layout/layout_upload_categories_item.xml | 12 +++++++++--- .../res/layout/layout_upload_depicts_item.xml | 16 +++++++++++----- .../res/layout/upload_categories_fragment.xml | 3 --- .../main/res/layout/upload_depicts_fragment.xml | 3 --- app/src/main/res/values-qq/strings.xml | 2 ++ app/src/main/res/values/strings.xml | 2 ++ 6 files changed, 24 insertions(+), 14 deletions(-) diff --git a/app/src/main/res/layout/layout_upload_categories_item.xml b/app/src/main/res/layout/layout_upload_categories_item.xml index 1c432ca88..146d55af3 100644 --- a/app/src/main/res/layout/layout_upload_categories_item.xml +++ b/app/src/main/res/layout/layout_upload_categories_item.xml @@ -3,6 +3,7 @@ xmlns:app="http://schemas.android.com/apk/res-auto" xmlns:tools="http://schemas.android.com/tools" android:id="@+id/category_layout" + android:layoutDirection="locale" android:layout_width="match_parent" android:layout_height="wrap_content"> @@ -10,26 +11,31 @@ android:id="@+id/upload_category_checkbox" android:layout_width="wrap_content" android:layout_height="wrap_content" + android:minWidth="48dp" + android:minHeight="48dp" android:checkMark="?android:attr/textCheckMark" android:checked="false" android:gravity="center_vertical" android:padding="@dimen/tiny_gap" app:layout_constraintBottom_toBottomOf="parent" app:layout_constraintEnd_toStartOf="@+id/category_image" - app:layout_constraintLeft_toLeftOf="parent" + app:layout_constraintStart_toStartOf="parent" app:layout_constraintTop_toTopOf="parent" /> @@ -9,26 +10,31 @@ android:id="@+id/depict_checkbox" android:layout_width="wrap_content" android:layout_height="wrap_content" + android:minWidth="48dp" + android:minHeight="48dp" android:checkMark="?android:attr/textCheckMark" android:checked="false" android:gravity="center_vertical" android:padding="@dimen/tiny_gap" app:layout_constraintBottom_toBottomOf="parent" app:layout_constraintEnd_toStartOf="@+id/depicted_image" - app:layout_constraintLeft_toLeftOf="parent" + app:layout_constraintStart_toStartOf="parent" app:layout_constraintTop_toTopOf="parent" /> + android:text="@string/depicts_description" /> diff --git a/app/src/main/res/layout/upload_categories_fragment.xml b/app/src/main/res/layout/upload_categories_fragment.xml index 6ded4af87..cd053bce9 100644 --- a/app/src/main/res/layout/upload_categories_fragment.xml +++ b/app/src/main/res/layout/upload_categories_fragment.xml @@ -17,7 +17,6 @@ android:layout_width="wrap_content" android:layout_height="@dimen/half_standard_height" android:layout_marginEnd="@dimen/standard_gap" - android:layout_marginRight="@dimen/standard_gap" android:orientation="horizontal"> {{Optional}}\n<code>&amp;#169;</code> is the copyright symbol (©). {{Doc-commons-app-depicts}} {{Doc-commons-app-depicts}} + Label + Description Panning means moving the map left/right/up/down, typically by touching the screen with one finger and moving it.\n\nZooming means making the map\'s scale bigger or smaller, typically by pinching with two fingers.\n\nExample in other app:\nhttps://igss.schneider-electric.com/features/pan-and-zoom-in-definition/ A description of a visual element, location picker image shadow. Used for accesibility usually. {{Identical|Label}} diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml index 183c683da..5ae4ca5e8 100644 --- a/app/src/main/res/values/strings.xml +++ b/app/src/main/res/values/strings.xml @@ -710,6 +710,8 @@ Upload your first media by tapping on the add button. Please select the appropriate categories. Unlike depictions, categories are only in English. Commons makes your pictures reusable and adapted by everyone. Do you want to waive all rights? Do you want to be attributed? Do you want adaptations to use the same license? Depicts + Label + Description Media License Media Details View category page From c4cb65fc3c926c9ae854b8eb12e5ad52305c82ce Mon Sep 17 00:00:00 2001 From: "Amir E. Aharoni" Date: Tue, 14 Oct 2025 08:40:38 -0400 Subject: [PATCH 09/13] Improve the grammar of messages about GPX and KML files (#6497) Add articles, fix capitalization, add ellipses. --- app/src/main/res/values/strings.xml | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml index 5ae4ca5e8..05b3ee6f0 100644 --- a/app/src/main/res/values/strings.xml +++ b/app/src/main/res/values/strings.xml @@ -827,12 +827,12 @@ Upload your first media by tapping on the add button. Your log-in has expired. Please log in again. No application available to open GPX files File Saved Successfully - Do you want to open GPX file? - Do you want to open KML file? - Failed to save KML file. - Failed to save GPX file. - Saving KML File - Saving GPX File + Do you want to open the GPX file? + Do you want to open the KML file? + Failed to save the KML file. + Failed to save the GPX file. + Saving as a KML file... + Saving as a GPX file... %d image selected %d images selected From 41d95814c94e2fe8e08dc9e4e2646e5981ef0dd1 Mon Sep 17 00:00:00 2001 From: "Amir E. Aharoni" Date: Tue, 14 Oct 2025 08:41:31 -0400 Subject: [PATCH 10/13] Remove the string SingleWebViewActivity (#6494) Resolves issue #6492. Remove the title of a web activity and the accompanying string resource. This was not a real translatable message, but something that looks more like an identifier that shouldn't be translated. As far as I can tell, it's not seen anywhere in the interface because the actual title is set in the code that calls it. --- app/src/main/AndroidManifest.xml | 3 +-- app/src/main/res/values/strings.xml | 1 - 2 files changed, 1 insertion(+), 3 deletions(-) diff --git a/app/src/main/AndroidManifest.xml b/app/src/main/AndroidManifest.xml index e8215bd90..d83d645dd 100644 --- a/app/src/main/AndroidManifest.xml +++ b/app/src/main/AndroidManifest.xml @@ -57,8 +57,7 @@ tools:replace="android:appComponentFactory"> + android:exported="false" /> diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml index 05b3ee6f0..d193b9aa2 100644 --- a/app/src/main/res/values/strings.xml +++ b/app/src/main/res/values/strings.xml @@ -875,7 +875,6 @@ Upload your first media by tapping on the add button. Other wikis • File usages - SingleWebViewActivity Account Vanish Account Vanish account warning From 61c9de6fcc444596c890880b64568beb4ab0a93a Mon Sep 17 00:00:00 2001 From: "Amir E. Aharoni" Date: Tue, 14 Oct 2025 08:42:09 -0400 Subject: [PATCH 11/13] Add a missing comma to a message (#6477) There should be a comma before "etc." in a list, and there already is a comma before "etc." in the string depicts_search_text_hint, so it should be in this string to for consistency. --- app/src/main/res/values/strings.xml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml index d193b9aa2..064b197e4 100644 --- a/app/src/main/res/values/strings.xml +++ b/app/src/main/res/values/strings.xml @@ -619,7 +619,7 @@ Upload your first media by tapping on the add button. Exists Needs Photo Place type: - Bridge, museum, hotel etc. + Bridge, museum, hotel, etc. Something went wrong with log-in. You must reset your password! MEDIA CHILD CLASSES From 2e52adbef8b04a2b0c1187527b3271df849a263d Mon Sep 17 00:00:00 2001 From: "Amir E. Aharoni" Date: Tue, 14 Oct 2025 18:37:17 -0400 Subject: [PATCH 12/13] Clean up empty tags in XML files (#6505) This resolves all the "XML empty tags" lint errors. --- app/src/main/res/drawable/thumbnail_not_selected.xml | 2 +- app/src/main/res/layout/dialog_select_language.xml | 2 +- .../main/res/layout/fragment_upload_media_detail_fragment.xml | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/app/src/main/res/drawable/thumbnail_not_selected.xml b/app/src/main/res/drawable/thumbnail_not_selected.xml index 8ead4b377..50404cf47 100644 --- a/app/src/main/res/drawable/thumbnail_not_selected.xml +++ b/app/src/main/res/drawable/thumbnail_not_selected.xml @@ -1,5 +1,5 @@ - + diff --git a/app/src/main/res/layout/dialog_select_language.xml b/app/src/main/res/layout/dialog_select_language.xml index 6fa70dcdb..dcef302fc 100644 --- a/app/src/main/res/layout/dialog_select_language.xml +++ b/app/src/main/res/layout/dialog_select_language.xml @@ -16,7 +16,7 @@ android:padding="12dp" app:layout_constraintEnd_toEndOf="parent" app:layout_constraintStart_toStartOf="parent" - app:layout_constraintTop_toTopOf="parent"> + app:layout_constraintTop_toTopOf="parent" /> + android:orientation="horizontal" /> Date: Wed, 15 Oct 2025 00:52:05 -0400 Subject: [PATCH 13/13] Simplify android:gravity in two layouts (#6506) The "Inspect Code" linter complained that these two files had Right-to-left text compatibility issues. I couldn't really see any problems neither in English nor in Hebrew, but the linter's suggestion still made sense, so I cleaned it up. This fixes all the errors of the type "Android Lint: Internationalization / Right-to-left text compatibility issues". --- app/src/main/res/layout/fragment_review_image.xml | 6 ++---- app/src/main/res/layout/pic_of_day_app_widget.xml | 1 - 2 files changed, 2 insertions(+), 5 deletions(-) diff --git a/app/src/main/res/layout/fragment_review_image.xml b/app/src/main/res/layout/fragment_review_image.xml index c4e9e4f73..4a21f4df0 100644 --- a/app/src/main/res/layout/fragment_review_image.xml +++ b/app/src/main/res/layout/fragment_review_image.xml @@ -22,9 +22,8 @@ android:id="@+id/tv_review_question" android:layout_width="match_parent" android:layout_height="wrap_content" - android:gravity="center_vertical" + android:gravity="center" tools:text="testing1" - android:textAlignment="center" android:textColor="?attr/reviewHeading" android:textSize="32sp"/> @@ -33,9 +32,8 @@ android:layout_width="match_parent" android:layout_height="wrap_content" android:layout_marginBottom="@dimen/filter_padding" - android:gravity="center_vertical" + android:gravity="center" tools:text="testing2" - android:textAlignment="center" android:textSize="22sp"/> diff --git a/app/src/main/res/layout/pic_of_day_app_widget.xml b/app/src/main/res/layout/pic_of_day_app_widget.xml index 46e00a0f4..beb3e20d4 100644 --- a/app/src/main/res/layout/pic_of_day_app_widget.xml +++ b/app/src/main/res/layout/pic_of_day_app_widget.xml @@ -35,7 +35,6 @@ android:layout_height="wrap_content" android:id="@+id/appwidget_title" android:textAlignment="center" - android:layout_gravity="bottom" android:textColor="@color/white" android:layout_marginTop="@dimen/filter_padding" android:layout_marginStart="@dimen/tiny_padding"