sirilius / watermarkktp

Watermark KTP is a free and secure web-based tool that allows you to add a watermark to your scanned KTP (Indonesian ID card).
https://watermarkktp.com
MIT License
384 stars 79 forks source link

Confusing behavior on auto-watermarking #64

Closed codenoid closed 2 years ago

codenoid commented 2 years ago

Issue

Setelah update terbaru, saya melihat ada fitur baru yang otomatis menambahkan text Verifikasi, 26-7-2022 secara dinamis ke caption watermark, ini bagus, tapi pengaplikasiannya mungkin masih kurang tepat:

beberapa bug(?) yang ditemukan:

1. CTRL+R
    -> tulis caption watermark
    -> tambah gambar 
    -> watermark yang sudah ditulis otomatis terganti menjadi *Verifikasi, 26-7-2022*

2. CTRL+R
    -> klik "reset"
    -> caption watermark otomatis terganti menjadi *Verifikasi, 26-7-2022*, padahal awalnya kosong

Related code:

https://github.com/sirilius/watermarkktp/blob/2eaa0de5b3a1679ceac596319a947ad23a413591/src/_includes/js/watermark.js#L215-L236

Suggestion

Yang pertama, bagaimana kalau kita menambahkan field baru diatas field Font, nama field nya Template Watermark, contoh sebagai berikut ini:

image

Kedua, by default, biarkan caption watermark kosong, supaya pengguna2 yang sebelumnya tidak bingung dengan perubahan terbaru ini, apalagi pengguna2 tua seperti saya yang kadang masih minta tolong orang untuk mempelajari aplikasi

jadi semisal ada perubahan value tetap atas dasar consent pengguna

sirilius commented 2 years ago

Hallo @codenoid , Thank you sudah membuka issue ini.

Sepertinya ini akan bagus untuk diaplikasikan pada Watermark KTP. Akan aku coba untuk mengeksperimenkannya.

sirilius commented 2 years ago

Done!

codenoid commented 2 years ago

Mantab, yang ini (aefcc0315fe1b03a7779ddcc319b7364bd3b1fd7) much2 simpler tanpa mengorbankan clarity 🦖 👍 👍 👍

Alright, this issue should be closed ig