search

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

Refactor, memperbaiki kualitas kode javascript, dan menghapus jquery #43

Closed reacto11mecha closed 3 years ago

reacto11mecha commented 3 years ago

Halo, saya membuat beberapa perubahan perihal code quality. Kalo diliat liat ada beberapa variabel yang masih antara pake let, const sama var dan ada juga variabel yang ganda gegera pake var. Usahakan jangan pake var lagi karena dia bisa di declare berulang kali padahal sebenernya udah ada variabel yang dideclare sebelumnya.

Untuk file html sama css semuanya kena format sama prettier supaya kodenya keliatan lebih rapih. Semua html yang ada juga dihapus js document.write karena ga disaranin pake itu lagi, sebagai gantinya dibuat file js yang bisa ngeganti itu.

Untuk jquery dihapus karena emang sebenernya kalo diliat liat project ini tuh ga perlu jquery bisa jalan sebenernya. Jadi pas ku refactor sekalian dihapus jquerynya. Jquery dihapus juga bisa impact ke kuota internet pengguna biar ga gede banget ngambil filenya.

Terima Kasih

netlify[bot] commented 3 years ago

✔️ Deploy Preview for watermarkktp ready!

🔨 Explore the source changes: 40504dae8006a618dccdc672fe6e18413a931483

🔍 Inspect the deploy log: https://app.netlify.com/sites/watermarkktp/deploys/6197a12d65391b00072ac4b4

😎 Browse the preview: https://deploy-preview-43--watermarkktp.netlify.app

sirilius commented 3 years ago

Halo kak, terima kasih sudah mengirimkan PR ini.

Setelah aku cek, ternyata fitur draggable file engga berfungsi, mungkin ada saran untuk memperbaikinya?

reacto11mecha commented 3 years ago

Halo kak, terima kasih sudah mengirimkan PR ini.

Setelah aku cek, ternyata fitur draggable file engga berfungsi, mungkin ada saran untuk memperbaikinya?

Sebentar, saya coba benerin dulu

reacto11mecha commented 3 years ago

Sudah kak, ternyata memang bug dari lama. Jadinya make for loop biasa lagi sebagai fixnya, coba di cek apakah udah bener atau belom https://stackoverflow.com/questions/11573710/event-datatransfer-files-is-empty-when-ondrop-is-fired

sirilius commented 3 years ago

Sudah kak, ternyata memang bug dari lama. Jadinya make for loop biasa lagi sebagai fixnya, coba di cek apakah udah bener atau belom https://stackoverflow.com/questions/11573710/event-datatransfer-files-is-empty-when-ondrop-is-fired

Oke kak, sudah bisa :+1:

sirilius commented 3 years ago

Terima kasih banyak kak :pray:

reacto11mecha commented 3 years ago

Sama sama