Laravel-log-viewer: Permintaan: Bagi tampilan menjadi tata letak + konten

Dibuat pada 14 Sep 2015  ·  9Komentar  ·  Sumber: rap2hpoutre/laravel-log-viewer

Setelah saran di sini:

https://github.com/rap2hpoutre/laravel-log-viewer/issues/39#issuecomment -139873059

Idenya adalah untuk memiliki tampilan terpisah - satu untuk tata letak, yaitu pembungkus halaman, dan satu untuk konten - tabel. Siapa pun yang ingin memasukkan konten ke dalam templat tata letak mereka sendiri di aplikasi mereka sendiri, perlu membagi tampilan seperti ini sebagai langkah pertama, jadi akan sangat membantu untuk membangunnya seperti ini sejak awal.

enhancement

Komentar yang paling membantu

mungkin solusi yang lebih baik:

Rap2hpoutre\LaravelLogViewer\LogViewerController

  • Buat variabel terproteksi bernama $viewLogFile(string) dan $viewLayoutFile(string) yang digunakan di View::make($viewFile...)

Ini memungkinkan Anda untuk membuat LogViewerController Anda sendiri yang memperluas yang Rap2hpoutre. Kemudian cukup atur $viewLogFile atau $viewLayoutFile untuk menggunakan wadah tampilan Anda sendiri sebagai induknya.

Semua 9 komentar

Saya setuju. Bagaimana dengan hanya menambahkan bagian?

Ya, ini akan menjadi cara biasa untuk memberikan struktur hierarkis ke tampilan.

+1

Ada saran tentang bagian apa yang harus saya tambahkan? Saya tidak perlu membangun tampilan saya sendiri jadi saya tidak tahu di mana saya harus menambahkan bagian. Saya terbuka untuk PR, jika tidak ada dalam beberapa hari saya akan membuat bagian saya sendiri berharap tidak apa-apa.

mungkin solusi yang lebih baik:

Rap2hpoutre\LaravelLogViewer\LogViewerController

  • Buat variabel terproteksi bernama $viewLogFile(string) dan $viewLayoutFile(string) yang digunakan di View::make($viewFile...)

Ini memungkinkan Anda untuk membuat LogViewerController Anda sendiri yang memperluas yang Rap2hpoutre. Kemudian cukup atur $viewLogFile atau $viewLayoutFile untuk menggunakan wadah tampilan Anda sendiri sebagai induknya.

Ya! Bisakah Anda membuat Permintaan Tarik untuk ini? Terima kasih!! (Jika Anda tidak mau, katakan saja dan saya bisa melakukannya sendiri)

Sebagai poin umum, string hard-coded dan nilai-nilai lain pasti akan mendapat manfaat dari menjadi properti controller, apakah mereka diubah dengan menimpa controller atau melalui semacam penyetel atau konfigurasi yang disuntikkan. Jadi ini tentu saja merupakan ide yang bagus, memungkinkan titik masuk utama ke halaman penampil log diubah dengan mudah.

Namun, poin utama dari masalah ini adalah untuk membagi tampilan tunggal menjadi lapisan yang disematkan, yang dapat disatukan dengan cara lain, atau memiliki lapisan yang diganti tanpa harus mengesampingkan seluruh tampilan halaman. Selain pembungkus halaman, ada tabel log utama, dan juga daftar bilah sisi file log. Semua ini idealnya sebagian (tampilan termasuk dalam tampilan) IMO.

Saya harus kembali ke proyek saya yang menggunakan ini nanti di Musim Panas, jadi saya bisa membuat PR, jika orang lain belum melakukannya.

Masalah ini sudah sangat lama (3 tahun) dan saya hanya memelihara paket ini tanpa menambahkan fitur baru. Saya menutup masalah ini. Jangan ragu untuk meminta dibuka kembali.

Apakah halaman ini membantu?
0 / 5 - 0 peringkat

Masalah terkait

akoepcke picture akoepcke  ·  9Komentar

hmrneves1 picture hmrneves1  ·  3Komentar

rap2hpoutre picture rap2hpoutre  ·  7Komentar

serdarsaygili picture serdarsaygili  ·  11Komentar

DevynCJohnson picture DevynCJohnson  ·  5Komentar