Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Menghilangkan Kode yg Tidak Perlu dan Mengklik Waktu Langsung Memulai Video yg Tersemat #15

Open
jefrydco opened this issue Aug 9, 2020 · 2 comments

Comments

@jefrydco
Copy link

jefrydco commented Aug 9, 2020

Hi Mas Arifin, terima kasih sudah mengirimkan kode belajar Vue.js Fundamental, saya sangat mengapresiasi kerja keras Mas Arifin.

Mas juga sudah menggunakan kerangka kerja Bootstrap pada aplikasi pencari teks pada video YouTube ini. Kemudian untuk menampilkan waktu agar terformat dengan baik juga Mas telah membuat fungsi utilitasnya sendiri, luar biasa Mas!

Namun, saya ada 2 saran Mas:

  1. Menghilangkan Kode yg Tidak Perlu
    Pada berkas index.html baris ke 210-212 Mas menggunakan perpustakaan JavaScript eksternal Bootstrap, Popper dan jQuery.

https://github.com/arifinizzah/subtify/blob/48169e4b8999c50675e7a7a66459a6c1a526cf32/index.html#L210-L212

Saya rasa Mas dapat menghilangkan perpustakaan JavaScript tersebut karena Mas hanya menggunakan Bootstrap sebagai pemercantik gaya. Mas juga tidak menggunakan ketiga perpustakaan JavaScript tersebut pada halaman.

  1. Mengklik Waktu Langsung Memulai Video yg Terembed
    Kalau yg ini mungkin saran fitur Mas. Sekarang pengguna jika ingin melihat teks pada videonya pengguna harus mengklik tautan waktu yg mengarahkannya ke tab baru untuk melihat videonya.

Menurut saya, mungkin akan lebih baik jika pengguna tidak perlu membuka tab baru untuk melihat videonya. Idenya, ketika pengguna mengklik tautan waktu, video yg tersemat akan memulai video dengan langsung mengarah ke detik yg bersangkutan.

Untuk mengarahkan video YouTube yg tersemat pada detik yg bersangkutan, caranya sama seperti tautan YouTube pada umumnya yakni dengan menambahkan parameter kueri t=<detik>s.

Semangat Mas! 😄

@arifineu
Copy link
Owner

arifineu commented Aug 9, 2020

Terimakasih mas Jefry atas koreksi dan masukannya 😄

Untuk poin yang nomer 2 yaitu "Mengklik Waktu Langsung Memulai Video yg Terembed" sebenarnya sudah masuk ke dalam rencana fitur mas. Tapi berhubung saya memiliki task lain yang harus saya kerjakan, jadi untuk saat ini saya masukkan dulu ke dalam list pengembangan selanjutnya.

@jefrydco
Copy link
Author

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

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

No branches or pull requests

2 participants