Laravel-log-viewer: Solicitação: Divida a visualização em layout + conteúdo

Eu concordo. Que tal apenas adicionar seções?

Sim, essa seria a maneira usual de fornecer estrutura hierárquica às visualizações.

+1

Alguma sugestão sobre qual seção devo adicionar? Não preciso construir minha própria visualização, então não tenho ideia de onde devo adicionar seções. Estou aberto a RP, se não houver nenhum em alguns dias, farei minha própria seção esperando que esteja tudo bem.

talvez uma solução ainda melhor:

Rap2hpoutre \ LaravelLogViewer \ LogViewerController

• Crie variáveis ​​protegidas chamadas $ viewLogFile (string) e $ viewLayoutFile (string) que são usadas em View :: make ($ viewFile ...)

Isso permite que você simplesmente crie seu próprio LogViewerController que estende o Rap2hpoutre. Em seguida, apenas defina $ viewLogFile ou $ viewLayoutFile para usar seu próprio contêiner de visualização como pai.

Sim! Você poderia fazer uma solicitação pull para isso? Obrigado!! (Se você não quiser apenas me diga e eu posso fazer isso sozinho)

Como um ponto geral, strings embutidas em código e outros valores certamente se beneficiariam por serem propriedades do controlador, sejam elas alteradas ao substituir o controlador ou por meio de algum tipo de setter ou configuração injetada. Portanto, esta é certamente uma boa ideia de qualquer maneira, permitindo que o ponto de entrada principal para a página do visualizador de log seja alterado facilmente.

O ponto principal desse problema, porém, era dividir a visualização única em camadas incorporadas, que podem ser colocadas juntas de outras maneiras, ou ter camadas sobrescritas sem ter que substituir a visualização da página inteira. Além do wrapper da página, há a tabela de log principal e também a lista de arquivos de log da barra lateral. Todos estes seriam idealmente parciais (visualizações incluídas nas visualizações) IMO.

Devo voltar ao meu projeto que usa isso mais tarde no verão, então posso criar um PR então, se outra pessoa ainda não o fez.

https://github.com/rap2hpoutre/laravel-log-viewer/pull/65

Este problema é super antigo (3 anos) e eu apenas mantenho este pacote sem adicionar novos recursos. Eu encerro este problema. Sinta-se à vontade para pedir a reabertura.