Skip to content

Conversation

@TheArKaID
Copy link

@TheArKaID TheArKaID commented Aug 16, 2025

Halo Mas @nafiesl ,
Merujuk ke issue buku-masjid/buku-masjid#127 tentang "clipboard img paste & drag drop", sepertinya penambahannya perlu dilakukan sekalian dari sini.

Contoh implementasi nantinya akan seperti ini:

image

Contoh hasilnya:
image

Hanya saja, saya lihat buku-masjid menggunakan versi 2.* (BS4), sedangkan PR ini based on master (3., BS5).
Perbedaan dari 2.
ke 3.* ada di laravelcollective/html dan konekt/html, dan juga mostly penambahan mb-3 dan fw-bold pada sebagian class, tapi saya coba versi 3.* (beserta fitur ini) ke buku-masjid sepertinya no issue Mas, apa ada concern tertentu ya?

Salam,
Arka.

@nafiesl
Copy link
Owner

nafiesl commented Aug 17, 2025

Maasyaallah, kayanya mantap ini mas @TheArKaID. Kalau methodnya dikasih nama FormField::fileDrop() gimana mas? Sepertinya cukup mewakili fungsinya (utk dropzone).

@TheArKaID
Copy link
Author

Maasyaallah, kayanya mantap ini mas @TheArKaID. Kalau methodnya dikasih nama FormField::fileDrop() gimana mas? Sepertinya cukup mewakili fungsinya (utk dropzone).

Bisa Mas @nafiesl saya sesuaikan ke ::fileDrop.

@TheArKaID TheArKaID changed the title Add FormField::fileModern Add FormField::fileDrop Aug 17, 2025
@nafiesl nafiesl self-requested a review August 17, 2025 06:40
@nafiesl
Copy link
Owner

nafiesl commented Aug 17, 2025

Bisa Mas @nafiesl saya sesuaikan ke ::fileDrop.

Makasih mas @TheArKaID nanti kalau ready for review, infokan mas, kita review sama2. Terima kasih mas

@TheArKaID
Copy link
Author

Makasih mas @TheArKaID nanti kalau ready for review, infokan mas, kita review sama2. Terima kasih mas

Saat ini sudah ready for review Mas @nafiesl .

@nafiesl
Copy link
Owner

nafiesl commented Aug 17, 2025

Saat ini sudah ready for review Mas @nafiesl .

Terima kasih mas @TheArKaID, ijin saya review dan uji coba 1-2 hari ke depan ya mas.

Hanya saja, saya lihat buku-masjid menggunakan versi 2.* (BS4), sedangkan PR ini based on master (3., BS5).
Perbedaan dari 2. ke 3.* ada di laravelcollective/html dan konekt/html, dan juga mostly penambahan mb-3 dan fw-bold pada sebagian class, tapi saya coba versi 3.* (beserta fitur ini) ke buku-masjid sepertinya no issue Mas, apa ada concern tertentu ya?

Tidak apa mas terkait hal ini, nanti setelah PRnya merged ke master, saya bisa cherry-pick commitnya dan update sesuai kebutuhan untuk versi bootstrap 4. Jadi nanti yang di buku masjid, bisa tetap pakai yang v2.x

Makasih banyak ini mas Arka.

@TheArKaID
Copy link
Author

Makasih banyak ini mas Arka.

Sama-sama Mas Nafies, infokan saja Mas bila ada penyesuaian yang diperlukan.

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