Laravel-log-viewer: Demande : diviser la vue en mise en page + contenu

Je suis d'accord. Et si vous ajoutiez simplement des sections ?

Oui, ce serait la manière habituelle de fournir une structure hiérarchique aux vues.

+1

Une suggestion sur la section que je devrais ajouter? Je n'ai pas besoin de créer ma propre vue, donc je n'ai aucune idée de l'endroit où je dois ajouter des sections. Je suis ouvert aux relations publiques, s'il n'y en a pas dans quelques jours, je créerai ma propre section en espérant que ce soit OK.

peut-être une solution encore meilleure :

Rap2hpoutre\LaravelLogViewer\LogViewerController

• Créez des variables protégées nommées $viewLogFile(string) et $viewLayoutFile(string) qui sont utilisées dans View::make($viewFile...)

Cela vous permet de créer simplement votre propre LogViewerController qui étend celui de Rap2hpoutre. Ensuite, définissez simplement $viewLogFile ou $viewLayoutFile pour utiliser votre propre conteneur de vue comme parent.

Oui! Pourriez-vous s'il vous plaît faire une Pull Request pour cela? Merci!! (Si tu ne veux pas, dis-le moi et je peux le faire moi-même)

De manière générale, les chaînes codées en dur et les autres valeurs gagneraient certainement à être des propriétés du contrôleur, qu'elles soient modifiées en remplaçant le contrôleur ou via une sorte de setter ou de configuration injectée. C'est donc certainement une bonne idée de toute façon, permettant de modifier facilement le point d'entrée principal de la page de la visionneuse de journaux.

Cependant, le point principal de ce problème était de diviser la vue unique en couches intégrées, qui peuvent être assemblées d'autres manières, ou dont les couches sont remplacées sans avoir à remplacer la vue de la page entière. En plus du wrapper de page, il y a la table de journal principale, ainsi que la liste de la barre latérale des fichiers journaux. Tous ces éléments seraient idéalement partiels (vues incluses dans les vues) OMI.

Je devrais revenir à mon projet qui l'utilise plus tard dans l'été, donc je peux créer un PR alors, si quelqu'un d'autre ne l'a pas déjà fait.

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

Ce problème est très ancien (3 ans) et je ne maintiens que ce package sans ajouter de nouvelles fonctionnalités. Je clos ce sujet. N'hésitez pas à demander la réouverture.