Merge origin/master into feat/migrate_to_compose (S4d-169: resolve PR #358 conflicts)

Unblock PR #358 pull_request CI while keeping it Draft. Resolve the 6 known conflicts:
- legacy View files kept deleted: MainActivity/AboutActivity/OpenSourceActivity/LaunchView
- .gitignore: union of both sides' additive entries
- pr_pre_check.yml: preserve the migration safety net (:app:assembleDebug, :shared:desktopTest,
  :app:testDebugUnitTest, strict WATERMARK_GOLDEN goldens); adopt master scaffolding (permissions,
  concurrency, paths-ignore, workflow_dispatch, wrapper validation, setup-android); DROP master's
  draft-skip guard so Draft PR #358 still runs the safety net; do NOT adopt master's compile-only -x test
- desktop_packaging.yml untouched
Verified: :app:assembleDebug + :shared:desktopTest BUILD SUCCESSFUL. Local commit only; not pushed.

Author: rosuH

.github/workflows/pr_pre_check.yml

@@ -1,55 +1,75 @@
-name: Basic Check
+name: PR Checks
 
 env:
   GRADLE_OPTS: -Dorg.gradle.jvmargs="-Xmx3072m -Dfile.encoding=UTF-8 -XX:+HeapDumpOnOutOfMemoryError -XX:MaxMetaspaceSize=1g" -Dkotlin.daemon.jvm.options="-Xmx2560m" -Dkotlin.incremental=false
-  CI_GRADLE_ARG_PROPERTIES: --stacktrace -PpreDexEnable=false --max-workers 2 --no-daemon
 
 on:
   pull_request:
-    types: [opened,reopened,synchronize]
+    types: [opened, reopened, synchronize]
     branches:
       - master
+    paths-ignore:
+      - '**/*.md'
+      - 'fastlane/**'
+      - 'static/**'
+      - 'captures/**'
+  workflow_dispatch:
+
+permissions:
+  contents: read
+
+concurrency:
+  group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }}
+  cancel-in-progress: true
+
 jobs:
-  check:
-    name: Check
-    runs-on: ubuntu-latest
-    steps:
-      - uses: actions/checkout@v4
-      - name: Skip duplicate actions
-        uses: fkirc/skip-duplicate-actions@v5
-        with:
-          cancel_others: true
-      - uses: actions/setup-java@v4
-        with:
-          distribution: 'zulu'
-          java-version: 17
-          cache: 'gradle'
-      - uses: gradle/actions/setup-gradle@v4
-        with:
-          gradle-version: wrapper
-          gradle-home-cache-cleanup: true
-#      - run: ./gradlew spotlessCheck
-  simple_compile:
+  build:
+    name: Compile + migration safety net (assembleDebug, :shared:desktopTest, strict goldens)
     runs-on: ubuntu-latest
+    timeout-minutes: 30
+    # S4d-169 (merge of origin/master): adopt master's scaffolding (paths-ignore, workflow_dispatch,
+    # permissions, concurrency, wrapper validation, setup-android) BUT keep the migration safety net
+    # (full tests, not master's compile-only `-x test`). The master `if: !github.event.pull_request.draft`
+    # guard is deliberately DROPPED so Draft PR #358 still runs :shared:desktopTest + the strict watermark
+    # goldens (the renderer pixel/dim gate the migration depends on).
     steps:
-      - uses: actions/checkout@v4
+      - name: Checkout
+        uses: actions/checkout@v4
         with:
           fetch-depth: 0
-      - uses: actions/setup-java@v4
+          persist-credentials: false
+
+      - name: Validate Gradle Wrapper
+        uses: gradle/wrapper-validation-action@v2
+
+      - name: Set up JDK 17
+        uses: actions/setup-java@v4
         with:
-          distribution: 'zulu'
-          java-version: 17
-          cache: 'gradle'
-      - uses: gradle/actions/setup-gradle@v4
+          distribution: 'temurin'
+          java-version: '17'
+
+      - name: Set up Android SDK
+        uses: android-actions/setup-android@v3
+
+      - name: Set up Gradle
+        uses: gradle/actions/setup-gradle@v4
         with:
           gradle-version: wrapper
+          cache-read-only: ${{ github.event_name == 'pull_request' }}
           gradle-home-cache-cleanup: true
+
+      - name: Make gradlew executable
+        run: chmod +x ./gradlew
+
       - name: Build apk with Gradle
         run: ./gradlew :app:assembleDebug
+
       - name: Run :shared multiplatform tests (JVM, CMP plan C1.8)
         run: ./gradlew :shared:desktopTest
+
       - name: Run :app unit tests incl. watermark golden (Robolectric NATIVE, CMP plan C1.7/C1.8)
         run: ./gradlew :app:testDebugUnitTest
+
       - name: Run STRICT watermark goldens (S4b renderer pixel/dim gate; pins FNV baselines)
         # WATERMARK_GOLDEN_STRICT is not a Gradle-tracked input, so a warm cache could otherwise
         # serve the prior non-strict :app:testDebugUnitTest result and skip the strict assertions.

.gitignore

@@ -114,3 +114,5 @@ hs_err_pid*
 .alma-snapshots
 .DS_Store
 .kotlin
+.weblate
+.weblate.ini

app/build.gradle.kts

@@ -41,8 +41,8 @@ android {
         applicationId = "me.rosuh.easywatermark"
         minSdk = (Apps.minSdk)
         targetSdk = (Apps.targetSdk)
-        versionCode = 20906
-        versionName = "2.9.6"
+        versionCode = 21000
+        versionName = "2.10.0"
         testInstrumentationRunner = "androidx.test.runner.AndroidJUnitRunner"
     }
 

app/src/main/AndroidManifest.xml

@@ -8,8 +8,18 @@
     <uses-permission
         android:name="android.permission.READ_EXTERNAL_STORAGE"
         android:maxSdkVersion="32" />
+    <uses-permission android:name="android.permission.VIBRATE" />
+
+    <!-- Devices running Android 12L (API level 32) or lower  -->
+    <uses-permission android:name="android.permission.READ_EXTERNAL_STORAGE" android:maxSdkVersion="32" />
+
+    <!-- Devices running Android 13 (API level 33) or higher -->
     <uses-permission android:name="android.permission.READ_MEDIA_IMAGES" />
 
+    <!-- To handle the reselection within the app on devices running Android 14
+         or higher if your app targets Android 14 (API level 34) or higher.  -->
+    <uses-permission android:name="android.permission.READ_MEDIA_VISUAL_USER_SELECTED" />
+
     <application
         android:name=".MyApp"
         android:allowBackup="true"
@@ -54,4 +64,4 @@
         </provider>
     </application>
 
-</manifest>
+</manifest>

app/src/main/java/me/rosuh/easywatermark/utils/FileUtils.kt

@@ -35,8 +35,33 @@ class FileUtils {
         }
 
         fun isImage(resolver: ContentResolver, uri: Uri?): Boolean {
-            val mimeType = getFileTypeFromUri(resolver, uri)
-            return isImage(mimeType)
+            return try {
+                val mimeType = getFileTypeFromUri(resolver, uri)
+                isImage(mimeType)
+            } catch (e: SecurityException) {
+                // Handle SecurityException on some Android systems (e.g., MIUI)
+                // where ContentResolver.getType() might be restricted
+                // Fall back to checking file extension
+                isImageByExtension(uri)
+            }
+        }
+
+        /**
+         * Fallback method to check if URI is an image by examining the file extension
+         * when MIME type detection fails due to security restrictions
+         */
+        private fun isImageByExtension(uri: Uri?): Boolean {
+            if (uri == null) return false
+
+            val extension = MimeTypeMap.getFileExtensionFromUrl(uri.toString())?.lowercase()
+            if (extension.isNullOrEmpty()) return false
+
+            val imageExtensions = setOf(
+                "jpg", "jpeg", "png", "gif", "bmp", "webp",
+                "tiff", "tif", "svg", "ico", "heic", "heif"
+            )
+
+            return imageExtensions.contains(extension)
         }
     }
 }

app/src/main/java/me/rosuh/easywatermark/utils/PickImageContract.kt

@@ -1,5 +1,6 @@
 package me.rosuh.easywatermark.utils
 
+import android.os.Build
 import android.view.HapticFeedbackConstants
 import android.view.View
 
@@ -9,18 +10,30 @@ class VibrateHelper private constructor() {
     private var cd: Long = 20L
 
     fun doVibrate(view: View) {
-        if (android.os.Build.VERSION.SDK_INT <= android.os.Build.VERSION_CODES.M) {
+        if (Build.VERSION.SDK_INT <= Build.VERSION_CODES.M) {
             return
         }
-        if (System.currentTimeMillis() - latestVibration <= cd) {
+        val now = System.currentTimeMillis()
+        if (now - latestVibration <= cd) {
             return
         }
-        view.performHapticFeedback(HapticFeedbackConstants.KEYBOARD_TAP)
+        latestVibration = now
+        performHapticFeedbackSafely {
+            view.performHapticFeedback(HapticFeedbackConstants.KEYBOARD_TAP)
+        }
     }
 
     companion object {
         fun get(): VibrateHelper {
             return VibrateHelper()
         }
+
+        internal fun performHapticFeedbackSafely(performFeedback: () -> Boolean): Boolean {
+            return try {
+                performFeedback()
+            } catch (_: SecurityException) {
+                false
+            }
+        }
     }
 }

app/src/main/java/me/rosuh/easywatermark/utils/VibrateHelper.kt

@@ -33,12 +33,12 @@ fun Activity.checkReadingPermission(
     },
     grant: () -> Unit,
 ) {
-    val permissions = if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.TIRAMISU) {
+    val permission = if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.TIRAMISU) {
         Manifest.permission.READ_MEDIA_IMAGES
     } else {
         Manifest.permission.READ_EXTERNAL_STORAGE
     }
-    preCheckStoragePermission(activityResultLauncher, permissions, failed, grant)
+    preCheckStoragePermission(activityResultLauncher, permission, failed, grant)
 }
 
 fun Activity.checkWritingPermission(

app/src/main/java/me/rosuh/easywatermark/utils/ktx/ContextExtension.kt

@@ -0,0 +1,34 @@
+package me.rosuh.easywatermark.utils.ktx
+
+import android.view.View
+import androidx.core.view.ViewCompat
+import androidx.core.view.WindowInsetsCompat
+
+data class InitialPadding(
+    val left: Int,
+    val top: Int,
+    val right: Int,
+    val bottom: Int
+)
+
+fun View.doOnApplyWindowInsets(
+    apply: (View, WindowInsetsCompat, InitialPadding) -> Unit
+) {
+    val initialPadding = InitialPadding(paddingLeft, paddingTop, paddingRight, paddingBottom)
+    ViewCompat.setOnApplyWindowInsetsListener(this) { view, insets ->
+        apply(view, insets, initialPadding)
+        WindowInsetsCompat.CONSUMED
+    }
+    if (isAttachedToWindow) {
+        ViewCompat.requestApplyInsets(this)
+    } else {
+        addOnAttachStateChangeListener(object : View.OnAttachStateChangeListener {
+            override fun onViewAttachedToWindow(v: View) {
+                v.removeOnAttachStateChangeListener(this)
+                ViewCompat.requestApplyInsets(v)
+            }
+
+            override fun onViewDetachedFromWindow(v: View) = Unit
+        })
+    }
+}

app/src/main/java/me/rosuh/easywatermark/utils/ktx/WindowInsetsExt.kt

@@ -0,0 +1,90 @@
+<?xml version="1.0" encoding="utf-8"?>
+<resources>
+    <string name="app_name">எளிதான வாட்டர்மார்க்</string>
+    <string name="action_pick">படத்தை தேர்வு செய்யவும்</string>
+    <string name="action_save">சேமிக்கவும்</string>
+    <string name="action_settings">பற்றி</string>
+    <string name="title_layout">மனையமைவு</string>
+    <string name="title_style">சூல் தண்டு</string>
+    <string name="title_content">உள்ளடக்கம்</string>
+    <string name="tips_do_not_choose_image">படம் எதுவும் தேர்ந்தெடுக்கப்படவில்லை</string>
+    <string name="tips_choose_color_dialog">வண்ணத்தைத் தேர்ந்தெடுங்கள்</string>
+    <string name="tips_confirm_dialog">உறுதிப்படுத்தவும்</string>
+    <string name="tips_cancel_dialog">ரத்துசெய்</string>
+    <string name="tips_error">பிழை</string>
+    <string name="title_text_style">சூல் தண்டு</string>
+    <string name="title_text_size">அளவு</string>
+    <string name="title_tile_mode">டைல்மோடு</string>
+    <string name="title_horizon_layout">கிடைமட்ட</string>
+    <string name="title_vertical_layout">செங்குத்து</string>
+    <string name="title_text_color">நிறம்</string>
+    <string name="title_text_rotate">கோணம்</string>
+    <string name="request_permission_failed">இசைவு பெற முடியவில்லை</string>
+    <string name="about_title_about">பற்றி</string>
+    <string name="about_title_info">தகவல்</string>
+    <string name="about_title_output">வெளியீட்டு வடிவம்</string>
+    <string name="about_title_version">பதிப்பு</string>
+    <string name="about_title_rating">எங்கள் பயன்பாட்டை மதிப்பிடவும்</string>
+    <string name="about_title_feed_back">பின்னூட்டம்</string>
+    <string name="about_title_update_log">சேஞ்ச்லாக்</string>
+    <string name="about_title_open_source">திறந்த மூல உரிமங்கள்</string>
+    <string name="error_not_img">தயவுசெய்து ஒரு படத்தைத் தேர்ந்தெடுக்கவும்</string>
+    <string name="error_file_not_found">படத்தை திறக்க முடியவில்லை அல்லது தேர்ந்தெடுக்கப்படவில்லை</string>
+    <string name="dialog_export_to_gallery">ஆல்பத்திற்கு ஏற்றுமதி செய்யவும்</string>
+    <string name="share">பகிரவும்</string>
+    <string name="water_mark_mode_text">உரை</string>
+    <string name="water_mark_mode_image">படவுரு</string>
+    <string name="tips_ok">உறுதிப்படுத்தவும்</string>
+    <string name="tips_cancel_output">ரத்து செய்</string>
+    <string name="tips_need_compress_img">போதுமான சேமிப்பிடம் இல்லை அல்லது படம் பெரிதாக உள்ளது</string>
+    <string name="tips_compressing">சுருக்குகிறது…</string>
+    <string name="tips_compress_images">படத்தை சுருக்கவும்</string>
+    <string name="tips_compress_ok">அமுக்கி முடிந்தது. ஏற்றுமதியைத் தொடர சரி என்பதைக் சொடுக்கு செய்யவும்.</string>
+    <string name="tips_compress_create_uri_failed">சுருக்கி முடிந்தது, ஆனால் யூரியை உருவாக்க முடியவில்லை. பயன்பாட்டை மறுதொடக்கம் செய்து மீண்டும் முயற்சிக்கவும். %1$s</string>
+    <string name="tips_tip_title">விபத்து கண்டறியப்பட்டது</string>
+    <string name="msg_crash">இந்த செயலிழப்பைப் புகாரளிக்க விரும்புகிறீர்களா? இது பயன்பாட்டை மேம்படுத்த எங்களுக்கு உதவும்! இது எந்த தனிப்பட்ட தகவலையும் சேகரிக்காது.</string>
+    <string name="crash_mail">மின்னஞ்சல் அனுப்பவும்</string>
+    <string name="tip_not_mail_found">மின்னஞ்சல் வாங்கி கண்டறியப்படவில்லை. மாற்றாக, நீங்கள் ஆப் கடை அல்லது GitHub இல் பிழை செய்தியை ஒட்டலாம்.</string>
+    <string name="config_default_water_mark_text">👋 மறுபகிர்வு செய்ய வேண்டாம்</string>
+    <string name="style_alpha">அகர</string>
+    <string name="tips_not_app_can_open_images">கோப்பு மேலாளர் கண்டறியப்படவில்லை.</string>
+    <string name="tips_choose_other_file_type">தவறான படம்</string>
+    <string name="error_save_oom">போதுமான நினைவகம் இல்லை, பின்புலத்தை தூய்மை செய்து மீண்டும் முயற்சிக்கவும்</string>
+    <string name="dev_comment">எந்த போதுமான மேம்பட்ட தொழில்நுட்பமும் மந்திரத்திலிருந்து பிரித்தறிய முடியாதது.</string>
+    <string name="dialog_title_exist_confirm">எச்சரிக்கை</string>
+    <string name="dialog_cancel_exist_confirm">ரத்துசெய்</string>
+    <string name="dialog_content_exist_confirm">மாற்றங்களை நிராகரிக்கவா?</string>
+    <string name="text_paint_fill">நிரப்பவும்</string>
+    <string name="text_paint_stroke">பக்கவாதம்</string>
+    <string name="text_typeface_normal">இயல்பானது</string>
+    <string name="text_typeface_bold">தடிமான</string>
+    <string name="text_typeface_italic">சாய்வு</string>
+    <string name="text_typeface_bold_italic">தடித்த சாய்வு</string>
+    <string name="dialog_save_export_list_title">ஏற்றுமதி பட்டியல்(%1$s)</string>
+    <string name="dialog_open_in_gallery">கேலரியில் பார்க்கவும்</string>
+    <string name="dialog_save_config_format">வடிவம்</string>
+    <string name="dialog_save_config_quality">தகுதி</string>
+    <string name="dialog_save_exporting">ஏற்றுமதி செய்கிறது</string>
+    <string name="tips_pick_image">படங்களை தேர்வு செய்யவும்</string>
+    <string name="pick_via_system">மேலும்…</string>
+    <string name="recovery_title">மீட்பு முறை</string>
+    <string name="copy_success">நகல் செய்</string>
+    <string name="copy_failed">நகலெடுக்க முடியவில்லை</string>
+    <string name="store_not_found">கடை எதுவும் நிறுவப்படவில்லையா?</string>
+    <string name="recovery_mode_closed">இது மூடப்பட்டது, மீண்டும் முயற்சிக்கவும்</string>
+    <string name="copy">பிழை செய்தியை நகலெடுக்கவும்</string>
+    <string name="recovery_mode_tips">பல செயலிழப்புகள் காரணமாக மீட்பு பயன்முறையில் நுழைந்தது. பின்வரும் முறைகள் மூலம் சாதனம் மற்றும் செயலிழப்பு தகவலை கீழே அனுப்புவது சிக்கலை தீர்க்கலாம். நன்றி.</string>
+    <string name="turn_off_recovery_mode">மீட்பு பயன்முறையை முடக்கு</string>
+    <string name="tips_input_text_can_not_be_empty">உள்ளடக்கம் காலியாக இருக்கக்கூடாது</string>
+    <string name="back">பின்</string>
+    <string name="dialog_title_template_edit">டெம்ப்ளேட்டைத் திருத்து</string>
+    <string name="dialog_title_edit_watermark">வாட்டர்மார்க் திருத்தவும்</string>
+    <string name="dialog_title_template_title">டெம்ப்ளேட் பட்டியல்</string>
+    <string name="tips_list_empty">வெறுமையாகத் தெரிகிறது</string>
+    <string name="dialog_button_add_template">கூட்டு</string>
+    <string name="tips_use_this_template">இந்த டெம்ப்ளேட்டைப் பயன்படுத்தவா?</string>
+    <string name="tips_delete_template">உள்ளீட்டை நீக்கவா?</string>
+    <string name="tips_database_init_error">தரவுத்தள துவக்கம் தோல்வியடைந்தது.</string>
+    <string name="tile_mode_title_repeat">மீண்டும் செய்யவும்</string>
+    <string name="tile_mode_title_decal">ஒற்றை</string>
+</resources>