search

dianasaurus-debug / vue_youtube_keyword

vue project of finding texts in a youtube video
1 stars 0 forks source link

Menghapus Kode yg Tidak Perlu #1

Open jefrydco opened 4 years ago

jefrydco commented 4 years ago

Hi Mbak Diana, terima kasih telah mengirimkan kode belajar Vue.js Fundamental, saya sangat mengapresiasi kerja keras Mbak Diana.

Mbak juga sudah menggunakan kerangka kerja CSS Bootstrap dan menambahkan kustom Fon dari Google Fonts, luar biasa Mbak!

Namun saya mempunyai 1 saran kecil untuk kode yg Mbak kirim. Pada berkas index.html baris ke 10-12 berikut

https://github.com/dianasaurus-debug/dianasaurus-debug.github.io/blob/8966075bce718c42c3a2a9787f39dd0e5c31c223/index.html#L10-L12

Mbak menambahkan perpustakaan JavaScript eksternal jQuery, Popper dan Bootstrap. Menurut saya Mbak bisa menghapus ketiga perpustakaan JavaScript tersbeut karena pada aplikasi pencari teks pada video YouTube ini Mbak tidak menggunakannya. Bahkan untuk Bootstrap pun, Mbak hanya perlu menggunakan kerangka kerja CSS nya saja.

Semangat Mbak! 😄

dianasaurus-debug commented 4 years ago

Wahh terima kasih banyak mas untuk koreksinya 😊

On Wed, Aug 12, 2020 at 6:49 PM Jefry Dewangga notifications@github.com wrote:

Hi Mbak Diana, terima kasih telah mengirimkan kode belajar Vue.js Fundamental, saya sangat mengapresiasi kerja keras Mbak Diana.

Mbak juga sudah menggunakan kerangka kerja CSS Bootstrap dan menambahkan kustom Fon dari Google Fonts, luar biasa Mbak!

Namun saya mempunyai 1 saran kecil untuk kode yg Mbak kirim. Pada berkas index.html baris ke 10-12 berikut

https://github.com/dianasaurus-debug/dianasaurus-debug.github.io/blob/8966075bce718c42c3a2a9787f39dd0e5c31c223/index.html#L10-L12

Mbak menambahkan perpustakaan JavaScript eksternal jQuery, Popper dan Bootstrap. Menurut saya Mbak bisa menghapus ketiga perpustakaan JavaScript tersbeut karena pada aplikasi pencari teks pada video YouTube ini Mbak tidak menggunakannya. Bahkan untuk Bootstrap pun, Mbak hanya perlu menggunakan kerangka kerja CSS nya saja.

Semangat Mbak! 😄

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/dianasaurus-debug/dianasaurus-debug.github.io/issues/1, or unsubscribe https://github.com/notifications/unsubscribe-auth/AM5DML6Z72XZO3SQZJMN4CDSAJ6URANCNFSM4P4SDNHA .

jefrydco commented 4 years ago

Terima kasih kembali Mbak, saya biarkan isu ini terbuka ya Mbak, nanti kalau Mbak ada waktu dan berkenan memperbaiki, Mbak bisa menutup isu ini. Semangat selalu Mbak! 😄