From 26a3cb8b7e36a421c4491ad987e053d96ee84075 Mon Sep 17 00:00:00 2001 From: Dan Brown Date: Thu, 6 Aug 2026 15:32:17 +0100 Subject: [PATCH 01/21] Views: Started implementation of new sidebar back-end --- .../Providers/ViewTweaksServiceProvider.php | 2 + app/Entities/Controllers/BookController.php | 2 - app/View/SidebarSection.php | 16 ++++++ app/View/SidebarSectionInterface.php | 20 ++++++++ app/View/SidebarSectionManager.php | 51 +++++++++++++++++++ .../SidebarSections/BooksIndexRecents.php | 26 ++++++++++ resources/views/books/index.blade.php | 4 +- 7 files changed, 118 insertions(+), 3 deletions(-) create mode 100644 app/View/SidebarSection.php create mode 100644 app/View/SidebarSectionInterface.php create mode 100644 app/View/SidebarSectionManager.php create mode 100644 app/View/SidebarSections/BooksIndexRecents.php diff --git a/app/App/Providers/ViewTweaksServiceProvider.php b/app/App/Providers/ViewTweaksServiceProvider.php index 6771e513f..d2d45abcc 100644 --- a/app/App/Providers/ViewTweaksServiceProvider.php +++ b/app/App/Providers/ViewTweaksServiceProvider.php @@ -4,6 +4,7 @@ namespace BookStack\App\Providers; use BookStack\Entities\BreadcrumbsViewComposer; use BookStack\Util\DateFormatter; +use BookStack\View\SidebarSectionManager; use Illuminate\Pagination\Paginator; use Illuminate\Support\Facades\Blade; use Illuminate\Support\Facades\View; @@ -33,6 +34,7 @@ class ViewTweaksServiceProvider extends ServiceProvider // View Globals View::share('dates', $this->app->make(DateFormatter::class)); + View::share('sidebar', $this->app->make(SidebarSectionManager::class)); // Custom blade view directives Blade::directive('icon', function ($expression) { diff --git a/app/Entities/Controllers/BookController.php b/app/Entities/Controllers/BookController.php index 98470d91c..9fa64d293 100644 --- a/app/Entities/Controllers/BookController.php +++ b/app/Entities/Controllers/BookController.php @@ -53,7 +53,6 @@ class BookController extends Controller $books = $this->queries->visibleForListWithCover() ->orderBy($listOptions->getSort(), $listOptions->getOrder()) ->paginate(setting()->getInteger('lists-page-count-books', 18, 1, 1000)); - $recents = $this->isSignedIn() ? $this->queries->recentlyViewedForCurrentUser()->take(4)->get() : false; $popular = $this->queries->popularForList()->take(4)->get(); $new = $this->queries->visibleForList()->orderBy('created_at', 'desc')->take(4)->get(); @@ -63,7 +62,6 @@ class BookController extends Controller return view('books.index', [ 'books' => $books, - 'recents' => $recents, 'popular' => $popular, 'new' => $new, 'view' => $view, diff --git a/app/View/SidebarSection.php b/app/View/SidebarSection.php new file mode 100644 index 000000000..68e1aa291 --- /dev/null +++ b/app/View/SidebarSection.php @@ -0,0 +1,16 @@ +view; + } +} diff --git a/app/View/SidebarSectionInterface.php b/app/View/SidebarSectionInterface.php new file mode 100644 index 000000000..84032ab61 --- /dev/null +++ b/app/View/SidebarSectionInterface.php @@ -0,0 +1,20 @@ + + */ + public function withData(array $viewData, Request $request): array; +} diff --git a/app/View/SidebarSectionManager.php b/app/View/SidebarSectionManager.php new file mode 100644 index 000000000..82af3721e --- /dev/null +++ b/app/View/SidebarSectionManager.php @@ -0,0 +1,51 @@ +[]>> + */ + protected array $sectionByLocation = [ + 'books-index' => [ + 'left' => [ + BooksIndexRecents::class, + ] + ], + ]; + + /** + * Register a sidebar section to be displayed at the given location and position. + * @param class-string $sectionClass + */ + public function register(string $location, string $position, string $sectionClass): void + { + if (!isset($this->sectionByLocation[$location])) { + $this->sectionByLocation[$location] = []; + } + + if (!isset($this->sectionByLocation[$location][$position])) { + $this->sectionByLocation[$location][$position] = []; + } + + $this->sectionByLocation[$location][$position][] = $sectionClass; + } + + /** + * Get all sidebar sections registered for a given location. + * + * @return SidebarSectionInterface[] + * @throws BindingResolutionException + */ + public function getSectionsForLocation(string $location, string $position): array + { + $sections = array_unique($this->sectionByLocation[$location][$position] ?? []); + return array_map(function (string $className) { + return app()->make($className); + }, $sections); + } +} diff --git a/app/View/SidebarSections/BooksIndexRecents.php b/app/View/SidebarSections/BooksIndexRecents.php new file mode 100644 index 000000000..485e6d431 --- /dev/null +++ b/app/View/SidebarSections/BooksIndexRecents.php @@ -0,0 +1,26 @@ +isGuest(); + $recents = $userSignedIn ? $this->queries->recentlyViewedForCurrentUser()->take(4)->get() : null; + return [ + 'recents' => $recents, + ]; + } +} diff --git a/resources/views/books/index.blade.php b/resources/views/books/index.blade.php index 660c008df..60152410c 100644 --- a/resources/views/books/index.blade.php +++ b/resources/views/books/index.blade.php @@ -5,7 +5,9 @@ @stop @section('left') - @include('books.parts.index-sidebar-section-recents', ['recents' => $recents]) + @foreach($sidebar->getSectionsForLocation('books-index', 'left') as $section) + @include($section->getView(), $section->withData(get_defined_vars(), request())) + @endforeach @include('books.parts.index-sidebar-section-popular', ['popular' => $popular]) @include('books.parts.index-sidebar-section-new', ['new' => $new]) @stop From dead0bfb5fd5b725230a5020973ddd4fa9560a95 Mon Sep 17 00:00:00 2001 From: Dan Brown Date: Fri, 7 Aug 2026 08:32:27 +0100 Subject: [PATCH 02/21] Views: Extracted books index/show sidebar sections --- app/Entities/Controllers/BookController.php | 15 +-------- app/View/SidebarSectionManager.php | 28 +++++++++++++++- .../SidebarSections/BooksIndexActions.php | 18 +++++++++++ app/View/SidebarSections/BooksIndexNew.php | 29 +++++++++++++++++ .../SidebarSections/BooksIndexPopular.php | 24 ++++++++++++++ app/View/SidebarSections/BooksShowActions.php | 24 ++++++++++++++ .../SidebarSections/BooksShowActivity.php | 28 ++++++++++++++++ app/View/SidebarSections/BooksShowDetails.php | 32 +++++++++++++++++++ .../SidebarSections/BooksShowSearchForm.php | 18 +++++++++++ app/View/SidebarSections/BooksShowShelves.php | 23 +++++++++++++ app/View/SidebarSections/BooksShowTags.php | 18 +++++++++++ resources/views/books/index.blade.php | 8 ++--- .../show-sidebar-section-details.blade.php | 2 +- .../show-sidebar-section-shelves.blade.php | 4 +-- resources/views/books/show.blade.php | 13 +++----- .../views/common/sidebar-sections.blade.php | 11 +++++++ 16 files changed, 262 insertions(+), 33 deletions(-) create mode 100644 app/View/SidebarSections/BooksIndexActions.php create mode 100644 app/View/SidebarSections/BooksIndexNew.php create mode 100644 app/View/SidebarSections/BooksIndexPopular.php create mode 100644 app/View/SidebarSections/BooksShowActions.php create mode 100644 app/View/SidebarSections/BooksShowActivity.php create mode 100644 app/View/SidebarSections/BooksShowDetails.php create mode 100644 app/View/SidebarSections/BooksShowSearchForm.php create mode 100644 app/View/SidebarSections/BooksShowShelves.php create mode 100644 app/View/SidebarSections/BooksShowTags.php create mode 100644 resources/views/common/sidebar-sections.blade.php diff --git a/app/Entities/Controllers/BookController.php b/app/Entities/Controllers/BookController.php index 9fa64d293..aa4f99daa 100644 --- a/app/Entities/Controllers/BookController.php +++ b/app/Entities/Controllers/BookController.php @@ -2,10 +2,8 @@ namespace BookStack\Entities\Controllers; -use BookStack\Activity\ActivityQueries; use BookStack\Activity\ActivityType; use BookStack\Activity\Models\View; -use BookStack\Activity\Tools\UserEntityWatchOptions; use BookStack\Entities\Queries\BookQueries; use BookStack\Entities\Queries\BookshelfQueries; use BookStack\Entities\Queries\EntityQueries; @@ -19,7 +17,6 @@ use BookStack\Exceptions\NotFoundException; use BookStack\Facades\Activity; use BookStack\Http\Controller; use BookStack\Permissions\Permission; -use BookStack\References\ReferenceFetcher; use BookStack\Util\DatabaseTransaction; use BookStack\Util\SimpleListOptions; use Illuminate\Http\Request; @@ -34,7 +31,6 @@ class BookController extends Controller protected BookQueries $queries, protected EntityQueries $entityQueries, protected BookshelfQueries $shelfQueries, - protected ReferenceFetcher $referenceFetcher, ) { } @@ -53,8 +49,6 @@ class BookController extends Controller $books = $this->queries->visibleForListWithCover() ->orderBy($listOptions->getSort(), $listOptions->getOrder()) ->paginate(setting()->getInteger('lists-page-count-books', 18, 1, 1000)); - $popular = $this->queries->popularForList()->take(4)->get(); - $new = $this->queries->visibleForList()->orderBy('created_at', 'desc')->take(4)->get(); $this->shelfContext->clearShelfContext(); @@ -62,8 +56,6 @@ class BookController extends Controller return view('books.index', [ 'books' => $books, - 'popular' => $popular, - 'new' => $new, 'view' => $view, 'listOptions' => $listOptions, ]); @@ -125,7 +117,7 @@ class BookController extends Controller /** * Display the specified book. */ - public function show(Request $request, ActivityQueries $activities, string $slug) + public function show(Request $request, string $slug) { try { $book = $this->queries->findVisibleBySlugOrFail($slug); @@ -138,7 +130,6 @@ class BookController extends Controller } $bookChildren = (new BookContents($book))->getTree(true); - $bookParentShelves = $book->shelves()->scopes('visible')->get(); View::incrementFor($book); if ($request->has('shelf')) { @@ -151,10 +142,6 @@ class BookController extends Controller 'book' => $book, 'current' => $book, 'bookChildren' => $bookChildren, - 'bookParentShelves' => $bookParentShelves, - 'watchOptions' => new UserEntityWatchOptions(user(), $book), - 'activity' => $activities->entityActivity($book, 20, 1), - 'referenceCount' => $this->referenceFetcher->getReferenceCountToEntity($book), ]); } diff --git a/app/View/SidebarSectionManager.php b/app/View/SidebarSectionManager.php index 82af3721e..d83b7afb0 100644 --- a/app/View/SidebarSectionManager.php +++ b/app/View/SidebarSectionManager.php @@ -2,7 +2,16 @@ namespace BookStack\View; +use BookStack\View\SidebarSections\BooksIndexActions; +use BookStack\View\SidebarSections\BooksIndexNew; +use BookStack\View\SidebarSections\BooksIndexPopular; use BookStack\View\SidebarSections\BooksIndexRecents; +use BookStack\View\SidebarSections\BooksShowActions; +use BookStack\View\SidebarSections\BooksShowActivity; +use BookStack\View\SidebarSections\BooksShowDetails; +use BookStack\View\SidebarSections\BooksShowSearchForm; +use BookStack\View\SidebarSections\BooksShowShelves; +use BookStack\View\SidebarSections\BooksShowTags; use Illuminate\Contracts\Container\BindingResolutionException; class SidebarSectionManager @@ -14,7 +23,24 @@ class SidebarSectionManager 'books-index' => [ 'left' => [ BooksIndexRecents::class, - ] + BooksIndexPopular::class, + BooksIndexNew::class, + ], + 'right' => [ + BooksIndexActions::class, + ], + ], + 'books-show' => [ + 'left' => [ + BooksShowSearchForm::class, + BooksShowTags::class, + BooksShowShelves::class, + BooksShowActivity::class, + ], + 'right' => [ + BooksShowDetails::class, + BooksShowActions::class, + ], ], ]; diff --git a/app/View/SidebarSections/BooksIndexActions.php b/app/View/SidebarSections/BooksIndexActions.php new file mode 100644 index 000000000..62f856a65 --- /dev/null +++ b/app/View/SidebarSections/BooksIndexActions.php @@ -0,0 +1,18 @@ + $viewData['view'], + ]; + } +} diff --git a/app/View/SidebarSections/BooksIndexNew.php b/app/View/SidebarSections/BooksIndexNew.php new file mode 100644 index 000000000..33a57f9d9 --- /dev/null +++ b/app/View/SidebarSections/BooksIndexNew.php @@ -0,0 +1,29 @@ +queries->visibleForList() + ->orderBy('created_at', 'desc') + ->take(4) + ->get(); + + return [ + 'new' => $new, + ]; + } +} diff --git a/app/View/SidebarSections/BooksIndexPopular.php b/app/View/SidebarSections/BooksIndexPopular.php new file mode 100644 index 000000000..9f5cc613f --- /dev/null +++ b/app/View/SidebarSections/BooksIndexPopular.php @@ -0,0 +1,24 @@ + $this->queries->popularForList()->take(4)->get(), + ]; + } +} diff --git a/app/View/SidebarSections/BooksShowActions.php b/app/View/SidebarSections/BooksShowActions.php new file mode 100644 index 000000000..0f97e625c --- /dev/null +++ b/app/View/SidebarSections/BooksShowActions.php @@ -0,0 +1,24 @@ + $book, + 'watchOptions' => new UserEntityWatchOptions(user(), $book), + ]; + } +} diff --git a/app/View/SidebarSections/BooksShowActivity.php b/app/View/SidebarSections/BooksShowActivity.php new file mode 100644 index 000000000..d892c5ea7 --- /dev/null +++ b/app/View/SidebarSections/BooksShowActivity.php @@ -0,0 +1,28 @@ + $this->activityQueries->entityActivity($book, 20, 1), + ]; + } +} diff --git a/app/View/SidebarSections/BooksShowDetails.php b/app/View/SidebarSections/BooksShowDetails.php new file mode 100644 index 000000000..bd91663b2 --- /dev/null +++ b/app/View/SidebarSections/BooksShowDetails.php @@ -0,0 +1,32 @@ +referenceFetcher->getReferenceCountToEntity($book); + + return [ + 'book' => $book, + 'watchOptions' => new UserEntityWatchOptions(user(), $book), + 'referenceCount' => $referenceCount, + ]; + } +} diff --git a/app/View/SidebarSections/BooksShowSearchForm.php b/app/View/SidebarSections/BooksShowSearchForm.php new file mode 100644 index 000000000..8fa24134b --- /dev/null +++ b/app/View/SidebarSections/BooksShowSearchForm.php @@ -0,0 +1,18 @@ + trans('entities.books_search_this'), + ]; + } +} diff --git a/app/View/SidebarSections/BooksShowShelves.php b/app/View/SidebarSections/BooksShowShelves.php new file mode 100644 index 000000000..28d2d8aa9 --- /dev/null +++ b/app/View/SidebarSections/BooksShowShelves.php @@ -0,0 +1,23 @@ +shelves()->scopes('visible')->get(); + + return [ + 'shelves' => $shelves, + ]; + } +} diff --git a/app/View/SidebarSections/BooksShowTags.php b/app/View/SidebarSections/BooksShowTags.php new file mode 100644 index 000000000..351686c7f --- /dev/null +++ b/app/View/SidebarSections/BooksShowTags.php @@ -0,0 +1,18 @@ + $viewData['book'], + ]; + } +} diff --git a/resources/views/books/index.blade.php b/resources/views/books/index.blade.php index 60152410c..020cb0966 100644 --- a/resources/views/books/index.blade.php +++ b/resources/views/books/index.blade.php @@ -5,13 +5,9 @@ @stop @section('left') - @foreach($sidebar->getSectionsForLocation('books-index', 'left') as $section) - @include($section->getView(), $section->withData(get_defined_vars(), request())) - @endforeach - @include('books.parts.index-sidebar-section-popular', ['popular' => $popular]) - @include('books.parts.index-sidebar-section-new', ['new' => $new]) + @include('common.sidebar-sections', ['location' => 'books-index', 'position' => 'left']) @stop @section('right') - @include('books.parts.index-sidebar-section-actions', ['view' => $view]) + @include('common.sidebar-sections', ['location' => 'books-index', 'position' => 'right']) @stop diff --git a/resources/views/books/parts/show-sidebar-section-details.blade.php b/resources/views/books/parts/show-sidebar-section-details.blade.php index 709d0ffd9..2c3d6d141 100644 --- a/resources/views/books/parts/show-sidebar-section-details.blade.php +++ b/resources/views/books/parts/show-sidebar-section-details.blade.php @@ -1,7 +1,7 @@
{{ trans('common.details') }}
- @include('entities.meta', ['entity' => $book, 'watchOptions' => $watchOptions]) + @include('entities.meta', ['entity' => $book, 'watchOptions' => $watchOptions, 'referenceCount' => $referenceCount]) @if($book->hasPermissions())
@if(userCan(\BookStack\Permissions\Permission::RestrictionsManage, $book)) diff --git a/resources/views/books/parts/show-sidebar-section-shelves.blade.php b/resources/views/books/parts/show-sidebar-section-shelves.blade.php index 9de9b95c6..9704c180c 100644 --- a/resources/views/books/parts/show-sidebar-section-shelves.blade.php +++ b/resources/views/books/parts/show-sidebar-section-shelves.blade.php @@ -1,6 +1,6 @@ -@if(count($bookParentShelves) > 0) +@if(count($shelves) > 0)
{{ trans('entities.shelves') }}
- @include('entities.list', ['entities' => $bookParentShelves, 'style' => 'compact']) + @include('entities.list', ['entities' => $shelves, 'style' => 'compact'])
@endif \ No newline at end of file diff --git a/resources/views/books/show.blade.php b/resources/views/books/show.blade.php index 4eb83164f..48a951132 100644 --- a/resources/views/books/show.blade.php +++ b/resources/views/books/show.blade.php @@ -66,15 +66,10 @@ @stop -@section('right') - @include('books.parts.show-sidebar-section-details', ['book' => $book, 'watchOptions' => $watchOptions]) - @include('books.parts.show-sidebar-section-actions', ['book' => $book, 'watchOptions' => $watchOptions]) -@stop - @section('left') - @include('entities.search-form', ['label' => trans('entities.books_search_this')]) - @include('books.parts.show-sidebar-section-tags', ['book' => $book]) - @include('books.parts.show-sidebar-section-shelves', ['bookParentShelves' => $bookParentShelves]) - @include('books.parts.show-sidebar-section-activity', ['activity' => $activity]) + @include('common.sidebar-sections', ['location' => 'books-show', 'position' => 'left']) @stop +@section('right') + @include('common.sidebar-sections', ['location' => 'books-show', 'position' => 'right']) +@stop diff --git a/resources/views/common/sidebar-sections.blade.php b/resources/views/common/sidebar-sections.blade.php new file mode 100644 index 000000000..a4a055e6a --- /dev/null +++ b/resources/views/common/sidebar-sections.blade.php @@ -0,0 +1,11 @@ +{{-- +$location - string +$position - string +--}} +@php + $sectionVars = get_defined_vars(); + $sectionRequest = request(); +@endphp +@foreach($sidebar->getSectionsForLocation($location, $position) as $section) + @include($section->getView(), $section->withData($sectionVars, $sectionRequest)) +@endforeach \ No newline at end of file From b462c67e2ff7a44c1942486967888d48b0d06380 Mon Sep 17 00:00:00 2001 From: Dan Brown Date: Fri, 7 Aug 2026 12:54:38 +0100 Subject: [PATCH 03/21] Views: Extracted sidebar blocks for main entity pages --- .../Controllers/BookshelfController.php | 16 +---- .../Controllers/ChapterController.php | 11 +--- app/Entities/Controllers/PageController.php | 13 +--- app/View/SidebarSectionManager.php | 62 +++++++++++++++++++ .../SidebarSections/BooksIndexRecents.php | 7 ++- .../SidebarSections/ChaptersShowActions.php | 24 +++++++ .../SidebarSections/ChaptersShowBookTree.php | 23 +++++++ .../SidebarSections/ChaptersShowDetails.php | 37 +++++++++++ .../ChaptersShowSearchForm.php | 18 ++++++ app/View/SidebarSections/ChaptersShowTags.php | 22 +++++++ app/View/SidebarSections/PagesShowActions.php | 24 +++++++ .../SidebarSections/PagesShowAttachments.php | 22 +++++++ .../SidebarSections/PagesShowBookTree.php | 23 +++++++ app/View/SidebarSections/PagesShowDetails.php | 37 +++++++++++ app/View/SidebarSections/PagesShowPageNav.php | 26 ++++++++ app/View/SidebarSections/PagesShowTags.php | 22 +++++++ .../SidebarSections/ShelvesIndexActions.php | 18 ++++++ app/View/SidebarSections/ShelvesIndexNew.php | 29 +++++++++ .../SidebarSections/ShelvesIndexPopular.php | 24 +++++++ .../SidebarSections/ShelvesIndexRecents.php | 29 +++++++++ .../SidebarSections/ShelvesShowActions.php | 22 +++++++ .../SidebarSections/ShelvesShowActivity.php | 28 +++++++++ .../SidebarSections/ShelvesShowDetails.php | 30 +++++++++ app/View/SidebarSections/ShelvesShowTags.php | 18 ++++++ resources/views/chapters/show.blade.php | 13 ++-- resources/views/entities/book-tree.blade.php | 2 +- resources/views/pages/show.blade.php | 8 +-- resources/views/shelves/index.blade.php | 12 ++-- resources/views/shelves/show.blade.php | 10 +-- 29 files changed, 564 insertions(+), 66 deletions(-) create mode 100644 app/View/SidebarSections/ChaptersShowActions.php create mode 100644 app/View/SidebarSections/ChaptersShowBookTree.php create mode 100644 app/View/SidebarSections/ChaptersShowDetails.php create mode 100644 app/View/SidebarSections/ChaptersShowSearchForm.php create mode 100644 app/View/SidebarSections/ChaptersShowTags.php create mode 100644 app/View/SidebarSections/PagesShowActions.php create mode 100644 app/View/SidebarSections/PagesShowAttachments.php create mode 100644 app/View/SidebarSections/PagesShowBookTree.php create mode 100644 app/View/SidebarSections/PagesShowDetails.php create mode 100644 app/View/SidebarSections/PagesShowPageNav.php create mode 100644 app/View/SidebarSections/PagesShowTags.php create mode 100644 app/View/SidebarSections/ShelvesIndexActions.php create mode 100644 app/View/SidebarSections/ShelvesIndexNew.php create mode 100644 app/View/SidebarSections/ShelvesIndexPopular.php create mode 100644 app/View/SidebarSections/ShelvesIndexRecents.php create mode 100644 app/View/SidebarSections/ShelvesShowActions.php create mode 100644 app/View/SidebarSections/ShelvesShowActivity.php create mode 100644 app/View/SidebarSections/ShelvesShowDetails.php create mode 100644 app/View/SidebarSections/ShelvesShowTags.php diff --git a/app/Entities/Controllers/BookshelfController.php b/app/Entities/Controllers/BookshelfController.php index 1e8b26b51..a918c48f1 100644 --- a/app/Entities/Controllers/BookshelfController.php +++ b/app/Entities/Controllers/BookshelfController.php @@ -2,7 +2,6 @@ namespace BookStack\Entities\Controllers; -use BookStack\Activity\ActivityQueries; use BookStack\Activity\Models\View; use BookStack\Entities\Queries\BookQueries; use BookStack\Entities\Queries\BookshelfQueries; @@ -13,7 +12,6 @@ use BookStack\Exceptions\ImageUploadException; use BookStack\Exceptions\NotFoundException; use BookStack\Http\Controller; use BookStack\Permissions\Permission; -use BookStack\References\ReferenceFetcher; use BookStack\Util\SimpleListOptions; use Exception; use Illuminate\Http\Request; @@ -27,7 +25,6 @@ class BookshelfController extends Controller protected EntityQueries $entityQueries, protected BookQueries $bookQueries, protected ShelfContext $shelfContext, - protected ReferenceFetcher $referenceFetcher, ) { } @@ -46,21 +43,12 @@ class BookshelfController extends Controller $shelves = $this->queries->visibleForListWithCover() ->orderBy($listOptions->getSort(), $listOptions->getOrder()) ->paginate(setting()->getInteger('lists-page-count-shelves', 18, 1, 1000)); - $recents = $this->isSignedIn() ? $this->queries->recentlyViewedForCurrentUser()->get() : false; - $popular = $this->queries->popularForList()->get(); - $new = $this->queries->visibleForList() - ->orderBy('created_at', 'desc') - ->take(4) - ->get(); $this->shelfContext->clearShelfContext(); $this->setPageTitle(trans('entities.shelves')); return view('shelves.index', [ 'shelves' => $shelves, - 'recents' => $recents, - 'popular' => $popular, - 'new' => $new, 'view' => $view, 'listOptions' => $listOptions, ]); @@ -105,7 +93,7 @@ class BookshelfController extends Controller * * @throws NotFoundException */ - public function show(Request $request, ActivityQueries $activities, string $slug) + public function show(Request $request, string $slug) { try { $shelf = $this->queries->findVisibleBySlugOrFail($slug); @@ -144,9 +132,7 @@ class BookshelfController extends Controller 'shelf' => $shelf, 'sortedVisibleShelfBooks' => $sortedVisibleShelfBooks, 'view' => $view, - 'activity' => $activities->entityActivity($shelf, 20, 1), 'listOptions' => $listOptions, - 'referenceCount' => $this->referenceFetcher->getReferenceCountToEntity($shelf), ]); } diff --git a/app/Entities/Controllers/ChapterController.php b/app/Entities/Controllers/ChapterController.php index db2391599..c089d3575 100644 --- a/app/Entities/Controllers/ChapterController.php +++ b/app/Entities/Controllers/ChapterController.php @@ -3,7 +3,6 @@ namespace BookStack\Entities\Controllers; use BookStack\Activity\Models\View; -use BookStack\Activity\Tools\UserEntityWatchOptions; use BookStack\Entities\Models\Book; use BookStack\Entities\Queries\ChapterQueries; use BookStack\Entities\Queries\EntityQueries; @@ -18,7 +17,6 @@ use BookStack\Exceptions\NotifyException; use BookStack\Exceptions\PermissionsException; use BookStack\Http\Controller; use BookStack\Permissions\Permission; -use BookStack\References\ReferenceFetcher; use BookStack\Util\DatabaseTransaction; use Illuminate\Http\Request; use Illuminate\Validation\ValidationException; @@ -30,7 +28,6 @@ class ChapterController extends Controller protected ChapterRepo $chapterRepo, protected ChapterQueries $queries, protected EntityQueries $entityQueries, - protected ReferenceFetcher $referenceFetcher, ) { } @@ -87,10 +84,10 @@ class ChapterController extends Controller return redirect($chapter->getUrl()); } - $sidebarTree = (new BookContents($chapter->book))->getTree(); $pages = $this->entityQueries->pages->visibleForChapterList($chapter->id)->get(); - $nextPreviousLocator = new NextPreviousContentLocator($chapter, $sidebarTree); + $bookTree = (new BookContents($chapter->book))->getTree(); + $nextPreviousLocator = new NextPreviousContentLocator($chapter, $bookTree); View::incrementFor($chapter); $this->setPageTitle($chapter->getShortName()); @@ -99,12 +96,10 @@ class ChapterController extends Controller 'book' => $chapter->book, 'chapter' => $chapter, 'current' => $chapter, - 'sidebarTree' => $sidebarTree, - 'watchOptions' => new UserEntityWatchOptions(user(), $chapter), 'pages' => $pages, 'next' => $nextPreviousLocator->getNext(), 'previous' => $nextPreviousLocator->getPrevious(), - 'referenceCount' => $this->referenceFetcher->getReferenceCountToEntity($chapter), + 'bookTree' => $bookTree, ]); } diff --git a/app/Entities/Controllers/PageController.php b/app/Entities/Controllers/PageController.php index 82edfbc27..ec3affc60 100644 --- a/app/Entities/Controllers/PageController.php +++ b/app/Entities/Controllers/PageController.php @@ -4,7 +4,6 @@ namespace BookStack\Entities\Controllers; use BookStack\Activity\Models\View; use BookStack\Activity\Tools\CommentTree; -use BookStack\Activity\Tools\UserEntityWatchOptions; use BookStack\Entities\Models\Book; use BookStack\Entities\Models\Chapter; use BookStack\Entities\Queries\EntityQueries; @@ -20,7 +19,6 @@ use BookStack\Exceptions\NotFoundException; use BookStack\Exceptions\PermissionsException; use BookStack\Http\Controller; use BookStack\Permissions\Permission; -use BookStack\References\ReferenceFetcher; use BookStack\Util\HtmlContentFilter; use BookStack\Util\HtmlContentFilterConfig; use Exception; @@ -35,7 +33,6 @@ class PageController extends Controller protected PageRepo $pageRepo, protected PageQueries $queries, protected EntityQueries $entityQueries, - protected ReferenceFetcher $referenceFetcher ) { } @@ -151,11 +148,10 @@ class PageController extends Controller $pageContent = (new PageContent($page)); $page->html = $pageContent->render(); - $pageNav = $pageContent->getNavigation($page->html); - $sidebarTree = (new BookContents($page->book))->getTree(); + $bookTree = (new BookContents($page->book))->getTree(); $commentTree = (new CommentTree($page)); - $nextPreviousLocator = new NextPreviousContentLocator($page, $sidebarTree); + $nextPreviousLocator = new NextPreviousContentLocator($page, $bookTree); View::incrementFor($page); $this->setPageTitle($page->getShortName()); @@ -164,13 +160,10 @@ class PageController extends Controller 'page' => $page, 'book' => $page->book, 'current' => $page, - 'sidebarTree' => $sidebarTree, + 'bookTree' => $bookTree, 'commentTree' => $commentTree, - 'pageNav' => $pageNav, - 'watchOptions' => new UserEntityWatchOptions(user(), $page), 'next' => $nextPreviousLocator->getNext(), 'previous' => $nextPreviousLocator->getPrevious(), - 'referenceCount' => $this->referenceFetcher->getReferenceCountToEntity($page), ]); } diff --git a/app/View/SidebarSectionManager.php b/app/View/SidebarSectionManager.php index d83b7afb0..d0105fef3 100644 --- a/app/View/SidebarSectionManager.php +++ b/app/View/SidebarSectionManager.php @@ -12,6 +12,25 @@ use BookStack\View\SidebarSections\BooksShowDetails; use BookStack\View\SidebarSections\BooksShowSearchForm; use BookStack\View\SidebarSections\BooksShowShelves; use BookStack\View\SidebarSections\BooksShowTags; +use BookStack\View\SidebarSections\ChaptersShowActions; +use BookStack\View\SidebarSections\ChaptersShowBookTree; +use BookStack\View\SidebarSections\ChaptersShowDetails; +use BookStack\View\SidebarSections\ChaptersShowSearchForm; +use BookStack\View\SidebarSections\ChaptersShowTags; +use BookStack\View\SidebarSections\PagesShowActions; +use BookStack\View\SidebarSections\PagesShowAttachments; +use BookStack\View\SidebarSections\PagesShowBookTree; +use BookStack\View\SidebarSections\PagesShowDetails; +use BookStack\View\SidebarSections\PagesShowPageNav; +use BookStack\View\SidebarSections\PagesShowTags; +use BookStack\View\SidebarSections\ShelvesIndexActions; +use BookStack\View\SidebarSections\ShelvesIndexNew; +use BookStack\View\SidebarSections\ShelvesIndexPopular; +use BookStack\View\SidebarSections\ShelvesIndexRecents; +use BookStack\View\SidebarSections\ShelvesShowActions; +use BookStack\View\SidebarSections\ShelvesShowActivity; +use BookStack\View\SidebarSections\ShelvesShowDetails; +use BookStack\View\SidebarSections\ShelvesShowTags; use Illuminate\Contracts\Container\BindingResolutionException; class SidebarSectionManager @@ -20,6 +39,26 @@ class SidebarSectionManager * @var array[]>> */ protected array $sectionByLocation = [ + 'shelves-index' => [ + 'left' => [ + ShelvesIndexRecents::class, + ShelvesIndexPopular::class, + ShelvesIndexNew::class, + ], + 'right' => [ + ShelvesIndexActions::class, + ], + ], + 'shelves-show' => [ + 'left' => [ + ShelvesShowTags::class, + ShelvesShowDetails::class, + ShelvesShowActivity::class, + ], + 'right' => [ + ShelvesShowActions::class, + ], + ], 'books-index' => [ 'left' => [ BooksIndexRecents::class, @@ -42,6 +81,29 @@ class SidebarSectionManager BooksShowActions::class, ], ], + 'chapters-show' => [ + 'left' => [ + ChaptersShowSearchForm::class, + ChaptersShowTags::class, + ChaptersShowBookTree::class, + ], + 'right' => [ + ChaptersShowDetails::class, + ChaptersShowActions::class, + ], + ], + 'pages-show' => [ + 'left' => [ + PagesShowTags::class, + PagesShowAttachments::class, + PagesShowPageNav::class, + PagesShowBookTree::class, + ], + 'right' => [ + PagesShowDetails::class, + PagesShowActions::class, + ], + ], ]; /** diff --git a/app/View/SidebarSections/BooksIndexRecents.php b/app/View/SidebarSections/BooksIndexRecents.php index 485e6d431..dcc5c94de 100644 --- a/app/View/SidebarSections/BooksIndexRecents.php +++ b/app/View/SidebarSections/BooksIndexRecents.php @@ -17,8 +17,11 @@ class BooksIndexRecents extends SidebarSection public function withData(array $viewData, Request $request): array { - $userSignedIn = !user()->isGuest(); - $recents = $userSignedIn ? $this->queries->recentlyViewedForCurrentUser()->take(4)->get() : null; + $recents = null; + if (!user()->isGuest()) { + $recents = $this->queries->recentlyViewedForCurrentUser()->take(4)->get(); + } + return [ 'recents' => $recents, ]; diff --git a/app/View/SidebarSections/ChaptersShowActions.php b/app/View/SidebarSections/ChaptersShowActions.php new file mode 100644 index 000000000..91e990d2f --- /dev/null +++ b/app/View/SidebarSections/ChaptersShowActions.php @@ -0,0 +1,24 @@ + $chapter, + 'watchOptions' => new UserEntityWatchOptions(user(), $chapter), + ]; + } +} diff --git a/app/View/SidebarSections/ChaptersShowBookTree.php b/app/View/SidebarSections/ChaptersShowBookTree.php new file mode 100644 index 000000000..d0de183f9 --- /dev/null +++ b/app/View/SidebarSections/ChaptersShowBookTree.php @@ -0,0 +1,23 @@ + $book, + 'bookTree' => $viewData['bookTree'], + ]; + } +} diff --git a/app/View/SidebarSections/ChaptersShowDetails.php b/app/View/SidebarSections/ChaptersShowDetails.php new file mode 100644 index 000000000..1453ceb9d --- /dev/null +++ b/app/View/SidebarSections/ChaptersShowDetails.php @@ -0,0 +1,37 @@ +referenceFetcher->getReferenceCountToEntity($chapter); + + return [ + 'chapter' => $chapter, + 'book' => $book, + 'watchOptions' => new UserEntityWatchOptions(user(), $chapter), + 'referenceCount' => $referenceCount, + ]; + } +} diff --git a/app/View/SidebarSections/ChaptersShowSearchForm.php b/app/View/SidebarSections/ChaptersShowSearchForm.php new file mode 100644 index 000000000..2ae71bc05 --- /dev/null +++ b/app/View/SidebarSections/ChaptersShowSearchForm.php @@ -0,0 +1,18 @@ + trans('entities.chapters_search_this'), + ]; + } +} diff --git a/app/View/SidebarSections/ChaptersShowTags.php b/app/View/SidebarSections/ChaptersShowTags.php new file mode 100644 index 000000000..df806d191 --- /dev/null +++ b/app/View/SidebarSections/ChaptersShowTags.php @@ -0,0 +1,22 @@ + $chapter, + ]; + } +} diff --git a/app/View/SidebarSections/PagesShowActions.php b/app/View/SidebarSections/PagesShowActions.php new file mode 100644 index 000000000..de832e775 --- /dev/null +++ b/app/View/SidebarSections/PagesShowActions.php @@ -0,0 +1,24 @@ + $page, + 'watchOptions' => new UserEntityWatchOptions(user(), $page), + ]; + } +} diff --git a/app/View/SidebarSections/PagesShowAttachments.php b/app/View/SidebarSections/PagesShowAttachments.php new file mode 100644 index 000000000..ef1dec4a0 --- /dev/null +++ b/app/View/SidebarSections/PagesShowAttachments.php @@ -0,0 +1,22 @@ + $page, + ]; + } +} diff --git a/app/View/SidebarSections/PagesShowBookTree.php b/app/View/SidebarSections/PagesShowBookTree.php new file mode 100644 index 000000000..3de85db49 --- /dev/null +++ b/app/View/SidebarSections/PagesShowBookTree.php @@ -0,0 +1,23 @@ + $book, + 'bookTree' => $viewData['bookTree'], + ]; + } +} diff --git a/app/View/SidebarSections/PagesShowDetails.php b/app/View/SidebarSections/PagesShowDetails.php new file mode 100644 index 000000000..e8ddfd341 --- /dev/null +++ b/app/View/SidebarSections/PagesShowDetails.php @@ -0,0 +1,37 @@ +referenceFetcher->getReferenceCountToEntity($page); + + return [ + 'page' => $page, + 'book' => $book, + 'watchOptions' => new UserEntityWatchOptions(user(), $page), + 'referenceCount' => $referenceCount, + ]; + } +} diff --git a/app/View/SidebarSections/PagesShowPageNav.php b/app/View/SidebarSections/PagesShowPageNav.php new file mode 100644 index 000000000..6f9d96652 --- /dev/null +++ b/app/View/SidebarSections/PagesShowPageNav.php @@ -0,0 +1,26 @@ +getNavigation($page->html); + + return [ + 'pageNav' => $pageNav, + ]; + } +} diff --git a/app/View/SidebarSections/PagesShowTags.php b/app/View/SidebarSections/PagesShowTags.php new file mode 100644 index 000000000..5fa46dc17 --- /dev/null +++ b/app/View/SidebarSections/PagesShowTags.php @@ -0,0 +1,22 @@ + $page, + ]; + } +} diff --git a/app/View/SidebarSections/ShelvesIndexActions.php b/app/View/SidebarSections/ShelvesIndexActions.php new file mode 100644 index 000000000..f15feecbe --- /dev/null +++ b/app/View/SidebarSections/ShelvesIndexActions.php @@ -0,0 +1,18 @@ + $viewData['view'], + ]; + } +} diff --git a/app/View/SidebarSections/ShelvesIndexNew.php b/app/View/SidebarSections/ShelvesIndexNew.php new file mode 100644 index 000000000..90fda4985 --- /dev/null +++ b/app/View/SidebarSections/ShelvesIndexNew.php @@ -0,0 +1,29 @@ +queries->visibleForList() + ->orderBy('created_at', 'desc') + ->take(4) + ->get(); + + return [ + 'new' => $new, + ]; + } +} diff --git a/app/View/SidebarSections/ShelvesIndexPopular.php b/app/View/SidebarSections/ShelvesIndexPopular.php new file mode 100644 index 000000000..4860b8c71 --- /dev/null +++ b/app/View/SidebarSections/ShelvesIndexPopular.php @@ -0,0 +1,24 @@ + $this->queries->popularForList()->take(4)->get(), + ]; + } +} diff --git a/app/View/SidebarSections/ShelvesIndexRecents.php b/app/View/SidebarSections/ShelvesIndexRecents.php new file mode 100644 index 000000000..72b6430eb --- /dev/null +++ b/app/View/SidebarSections/ShelvesIndexRecents.php @@ -0,0 +1,29 @@ +isGuest()) { + $recents = $this->queries->recentlyViewedForCurrentUser()->take(4)->get(); + } + + return [ + 'recents' => $recents, + ]; + } +} diff --git a/app/View/SidebarSections/ShelvesShowActions.php b/app/View/SidebarSections/ShelvesShowActions.php new file mode 100644 index 000000000..b28abab6f --- /dev/null +++ b/app/View/SidebarSections/ShelvesShowActions.php @@ -0,0 +1,22 @@ + $shelf, + ]; + } +} diff --git a/app/View/SidebarSections/ShelvesShowActivity.php b/app/View/SidebarSections/ShelvesShowActivity.php new file mode 100644 index 000000000..90de335e2 --- /dev/null +++ b/app/View/SidebarSections/ShelvesShowActivity.php @@ -0,0 +1,28 @@ + $this->activityQueries->entityActivity($shelf, 20, 1), + ]; + } +} diff --git a/app/View/SidebarSections/ShelvesShowDetails.php b/app/View/SidebarSections/ShelvesShowDetails.php new file mode 100644 index 000000000..62dc9bab2 --- /dev/null +++ b/app/View/SidebarSections/ShelvesShowDetails.php @@ -0,0 +1,30 @@ +referenceFetcher->getReferenceCountToEntity($shelf); + + return [ + 'shelf' => $shelf, + 'referenceCount' => $referenceCount, + ]; + } +} diff --git a/app/View/SidebarSections/ShelvesShowTags.php b/app/View/SidebarSections/ShelvesShowTags.php new file mode 100644 index 000000000..7c02c50fc --- /dev/null +++ b/app/View/SidebarSections/ShelvesShowTags.php @@ -0,0 +1,18 @@ + $viewData['book'], + ]; + } +} diff --git a/resources/views/chapters/show.blade.php b/resources/views/chapters/show.blade.php index 7ef877661..569d901f6 100644 --- a/resources/views/chapters/show.blade.php +++ b/resources/views/chapters/show.blade.php @@ -62,15 +62,10 @@ @stop -@section('right') - @include('chapters.parts.show-sidebar-section-details', ['chapter' => $chapter, 'book' => $book, 'watchOptions' => $watchOptions]) - @include('chapters.parts.show-sidebar-section-actions', ['chapter' => $chapter, 'watchOptions' => $watchOptions]) -@stop - @section('left') - @include('entities.search-form', ['label' => trans('entities.chapters_search_this')]) - @include('chapters.parts.show-sidebar-section-tags', ['chapter' => $chapter]) - @include('entities.book-tree', ['book' => $book, 'sidebarTree' => $sidebarTree]) + @include('common.sidebar-sections', ['location' => 'chapters-show', 'position' => 'left']) @stop - +@section('right') + @include('common.sidebar-sections', ['location' => 'chapters-show', 'position' => 'right']) +@stop diff --git a/resources/views/entities/book-tree.blade.php b/resources/views/entities/book-tree.blade.php index c95b1eebc..81dbd302e 100644 --- a/resources/views/entities/book-tree.blade.php +++ b/resources/views/entities/book-tree.blade.php @@ -11,7 +11,7 @@ @endif - @foreach($sidebarTree as $bookChild) + @foreach($bookTree as $bookChild)
  • @include('entities.list-item-basic', ['entity' => $bookChild, 'classes' => $current->matches($bookChild)? 'selected' : '']) diff --git a/resources/views/pages/show.blade.php b/resources/views/pages/show.blade.php index d34bfc6ae..08d01973d 100644 --- a/resources/views/pages/show.blade.php +++ b/resources/views/pages/show.blade.php @@ -36,13 +36,9 @@ @stop @section('left') - @include('pages.parts.show-sidebar-section-tags', ['page' => $page]) - @include('pages.parts.show-sidebar-section-attachments', ['page' => $page]) - @include('pages.parts.show-sidebar-section-page-nav', ['pageNav' => $pageNav]) - @include('entities.book-tree', ['book' => $book, 'sidebarTree' => $sidebarTree]) + @include('common.sidebar-sections', ['location' => 'pages-show', 'position' => 'left']) @stop @section('right') - @include('pages.parts.show-sidebar-section-details', ['page' => $page, 'watchOptions' => $watchOptions, 'book' => $book]) - @include('pages.parts.show-sidebar-section-actions', ['page' => $page, 'watchOptions' => $watchOptions]) + @include('common.sidebar-sections', ['location' => 'pages-show', 'position' => 'right']) @stop diff --git a/resources/views/shelves/index.blade.php b/resources/views/shelves/index.blade.php index 70357068d..2fc78fa50 100644 --- a/resources/views/shelves/index.blade.php +++ b/resources/views/shelves/index.blade.php @@ -4,12 +4,10 @@ @include('shelves.parts.list', ['shelves' => $shelves, 'view' => $view, 'listOptions' => $listOptions]) @stop -@section('right') - @include('shelves.parts.index-sidebar-section-actions', ['view' => $view]) +@section('left') + @include('common.sidebar-sections', ['location' => 'shelves-index', 'position' => 'left']) @stop -@section('left') - @include('shelves.parts.index-sidebar-section-recents', ['recents' => $recents]) - @include('shelves.parts.index-sidebar-section-popular', ['popular' => $popular]) - @include('shelves.parts.index-sidebar-section-new', ['new' => $new]) -@stop \ No newline at end of file +@section('right') + @include('common.sidebar-sections', ['location' => 'shelves-index', 'position' => 'right']) +@stop diff --git a/resources/views/shelves/show.blade.php b/resources/views/shelves/show.blade.php index 9d07e5da0..320e1066c 100644 --- a/resources/views/shelves/show.blade.php +++ b/resources/views/shelves/show.blade.php @@ -69,15 +69,9 @@ @stop @section('left') - @include('shelves.parts.show-sidebar-section-tags', ['shelf' => $shelf]) - @include('shelves.parts.show-sidebar-section-details', ['shelf' => $shelf]) - @include('shelves.parts.show-sidebar-section-activity', ['activity' => $activity]) + @include('common.sidebar-sections', ['location' => 'shelves-show', 'position' => 'left']) @stop @section('right') - @include('shelves.parts.show-sidebar-section-actions', ['shelf' => $shelf, 'view' => $view]) + @include('common.sidebar-sections', ['location' => 'shelves-show', 'position' => 'right']) @stop - - - - From d4d90cc39c803f10f27bf8501088b116bcfc8d6c Mon Sep 17 00:00:00 2001 From: Dan Brown Date: Fri, 7 Aug 2026 16:00:04 +0100 Subject: [PATCH 04/21] Views: Renamed SidebarSections to ViewBlocks --- .../Providers/ViewTweaksServiceProvider.php | 4 +- app/View/SidebarSectionManager.php | 139 ------------------ .../{SidebarSection.php => ViewBlock.php} | 2 +- ...onInterface.php => ViewBlockInterface.php} | 4 +- app/View/ViewBlockManager.php | 139 ++++++++++++++++++ .../BooksIndexActions.php | 6 +- .../BooksIndexNew.php | 6 +- .../BooksIndexPopular.php | 6 +- .../BooksIndexRecents.php | 6 +- .../BooksShowActions.php | 6 +- .../BooksShowActivity.php | 6 +- .../BooksShowDetails.php | 6 +- .../BooksShowSearchForm.php | 6 +- .../BooksShowShelves.php | 6 +- .../BooksShowTags.php | 6 +- .../ChaptersShowActions.php | 6 +- .../ChaptersShowBookTree.php | 6 +- .../ChaptersShowDetails.php | 6 +- .../ChaptersShowSearchForm.php | 6 +- .../ChaptersShowTags.php | 6 +- .../PagesShowActions.php | 6 +- .../PagesShowAttachments.php | 6 +- .../PagesShowBookTree.php | 6 +- .../PagesShowDetails.php | 6 +- .../PagesShowPageNav.php | 6 +- .../PagesShowTags.php | 6 +- .../ShelvesIndexActions.php | 6 +- .../ShelvesIndexNew.php | 6 +- .../ShelvesIndexPopular.php | 6 +- .../ShelvesIndexRecents.php | 6 +- .../ShelvesShowActions.php | 6 +- .../ShelvesShowActivity.php | 6 +- .../ShelvesShowDetails.php | 6 +- .../ShelvesShowTags.php | 6 +- .../views/common/sidebar-sections.blade.php | 2 +- 35 files changed, 232 insertions(+), 232 deletions(-) delete mode 100644 app/View/SidebarSectionManager.php rename app/View/{SidebarSection.php => ViewBlock.php} (73%) rename app/View/{SidebarSectionInterface.php => ViewBlockInterface.php} (82%) create mode 100644 app/View/ViewBlockManager.php rename app/View/{SidebarSections => ViewBlocks}/BooksIndexActions.php (68%) rename app/View/{SidebarSections => ViewBlocks}/BooksIndexNew.php (81%) rename app/View/{SidebarSections => ViewBlocks}/BooksIndexPopular.php (77%) rename app/View/{SidebarSections => ViewBlocks}/BooksIndexRecents.php (81%) rename app/View/{SidebarSections => ViewBlocks}/BooksShowActions.php (79%) rename app/View/{SidebarSections => ViewBlocks}/BooksShowActivity.php (81%) rename app/View/{SidebarSections => ViewBlocks}/BooksShowDetails.php (86%) rename app/View/{SidebarSections => ViewBlocks}/BooksShowSearchForm.php (67%) rename app/View/{SidebarSections => ViewBlocks}/BooksShowShelves.php (77%) rename app/View/{SidebarSections => ViewBlocks}/BooksShowTags.php (68%) rename app/View/{SidebarSections => ViewBlocks}/ChaptersShowActions.php (80%) rename app/View/{SidebarSections => ViewBlocks}/ChaptersShowBookTree.php (75%) rename app/View/{SidebarSections => ViewBlocks}/ChaptersShowDetails.php (88%) rename app/View/{SidebarSections => ViewBlocks}/ChaptersShowSearchForm.php (67%) rename app/View/{SidebarSections => ViewBlocks}/ChaptersShowTags.php (75%) rename app/View/{SidebarSections => ViewBlocks}/PagesShowActions.php (79%) rename app/View/{SidebarSections => ViewBlocks}/PagesShowAttachments.php (74%) rename app/View/{SidebarSections => ViewBlocks}/PagesShowBookTree.php (75%) rename app/View/{SidebarSections => ViewBlocks}/PagesShowDetails.php (87%) rename app/View/{SidebarSections => ViewBlocks}/PagesShowPageNav.php (80%) rename app/View/{SidebarSections => ViewBlocks}/PagesShowTags.php (74%) rename app/View/{SidebarSections => ViewBlocks}/ShelvesIndexActions.php (68%) rename app/View/{SidebarSections => ViewBlocks}/ShelvesIndexNew.php (81%) rename app/View/{SidebarSections => ViewBlocks}/ShelvesIndexPopular.php (78%) rename app/View/{SidebarSections => ViewBlocks}/ShelvesIndexRecents.php (81%) rename app/View/{SidebarSections => ViewBlocks}/ShelvesShowActions.php (75%) rename app/View/{SidebarSections => ViewBlocks}/ShelvesShowActivity.php (81%) rename app/View/{SidebarSections => ViewBlocks}/ShelvesShowDetails.php (83%) rename app/View/{SidebarSections => ViewBlocks}/ShelvesShowTags.php (68%) diff --git a/app/App/Providers/ViewTweaksServiceProvider.php b/app/App/Providers/ViewTweaksServiceProvider.php index d2d45abcc..0c68aa1c0 100644 --- a/app/App/Providers/ViewTweaksServiceProvider.php +++ b/app/App/Providers/ViewTweaksServiceProvider.php @@ -4,7 +4,7 @@ namespace BookStack\App\Providers; use BookStack\Entities\BreadcrumbsViewComposer; use BookStack\Util\DateFormatter; -use BookStack\View\SidebarSectionManager; +use BookStack\View\ViewBlockManager; use Illuminate\Pagination\Paginator; use Illuminate\Support\Facades\Blade; use Illuminate\Support\Facades\View; @@ -34,7 +34,7 @@ class ViewTweaksServiceProvider extends ServiceProvider // View Globals View::share('dates', $this->app->make(DateFormatter::class)); - View::share('sidebar', $this->app->make(SidebarSectionManager::class)); + View::share('viewBlocks', $this->app->make(ViewBlockManager::class)); // Custom blade view directives Blade::directive('icon', function ($expression) { diff --git a/app/View/SidebarSectionManager.php b/app/View/SidebarSectionManager.php deleted file mode 100644 index d0105fef3..000000000 --- a/app/View/SidebarSectionManager.php +++ /dev/null @@ -1,139 +0,0 @@ -[]>> - */ - protected array $sectionByLocation = [ - 'shelves-index' => [ - 'left' => [ - ShelvesIndexRecents::class, - ShelvesIndexPopular::class, - ShelvesIndexNew::class, - ], - 'right' => [ - ShelvesIndexActions::class, - ], - ], - 'shelves-show' => [ - 'left' => [ - ShelvesShowTags::class, - ShelvesShowDetails::class, - ShelvesShowActivity::class, - ], - 'right' => [ - ShelvesShowActions::class, - ], - ], - 'books-index' => [ - 'left' => [ - BooksIndexRecents::class, - BooksIndexPopular::class, - BooksIndexNew::class, - ], - 'right' => [ - BooksIndexActions::class, - ], - ], - 'books-show' => [ - 'left' => [ - BooksShowSearchForm::class, - BooksShowTags::class, - BooksShowShelves::class, - BooksShowActivity::class, - ], - 'right' => [ - BooksShowDetails::class, - BooksShowActions::class, - ], - ], - 'chapters-show' => [ - 'left' => [ - ChaptersShowSearchForm::class, - ChaptersShowTags::class, - ChaptersShowBookTree::class, - ], - 'right' => [ - ChaptersShowDetails::class, - ChaptersShowActions::class, - ], - ], - 'pages-show' => [ - 'left' => [ - PagesShowTags::class, - PagesShowAttachments::class, - PagesShowPageNav::class, - PagesShowBookTree::class, - ], - 'right' => [ - PagesShowDetails::class, - PagesShowActions::class, - ], - ], - ]; - - /** - * Register a sidebar section to be displayed at the given location and position. - * @param class-string $sectionClass - */ - public function register(string $location, string $position, string $sectionClass): void - { - if (!isset($this->sectionByLocation[$location])) { - $this->sectionByLocation[$location] = []; - } - - if (!isset($this->sectionByLocation[$location][$position])) { - $this->sectionByLocation[$location][$position] = []; - } - - $this->sectionByLocation[$location][$position][] = $sectionClass; - } - - /** - * Get all sidebar sections registered for a given location. - * - * @return SidebarSectionInterface[] - * @throws BindingResolutionException - */ - public function getSectionsForLocation(string $location, string $position): array - { - $sections = array_unique($this->sectionByLocation[$location][$position] ?? []); - return array_map(function (string $className) { - return app()->make($className); - }, $sections); - } -} diff --git a/app/View/SidebarSection.php b/app/View/ViewBlock.php similarity index 73% rename from app/View/SidebarSection.php rename to app/View/ViewBlock.php index 68e1aa291..a835e7075 100644 --- a/app/View/SidebarSection.php +++ b/app/View/ViewBlock.php @@ -2,7 +2,7 @@ namespace BookStack\View; -abstract class SidebarSection implements SidebarSectionInterface +abstract class ViewBlock implements ViewBlockInterface { protected string $view; diff --git a/app/View/SidebarSectionInterface.php b/app/View/ViewBlockInterface.php similarity index 82% rename from app/View/SidebarSectionInterface.php rename to app/View/ViewBlockInterface.php index 84032ab61..b9c25e9f6 100644 --- a/app/View/SidebarSectionInterface.php +++ b/app/View/ViewBlockInterface.php @@ -4,10 +4,10 @@ namespace BookStack\View; use Illuminate\Http\Request; -interface SidebarSectionInterface +interface ViewBlockInterface { /** - * Get the view to render for this section. + * Get the view to render for this block. */ public function getView(): string; diff --git a/app/View/ViewBlockManager.php b/app/View/ViewBlockManager.php new file mode 100644 index 000000000..88d866135 --- /dev/null +++ b/app/View/ViewBlockManager.php @@ -0,0 +1,139 @@ +[]>> + */ + protected array $blocksByLocationAndPosition = [ + 'shelves-index' => [ + 'left' => [ + ShelvesIndexRecents::class, + ShelvesIndexPopular::class, + ShelvesIndexNew::class, + ], + 'right' => [ + ShelvesIndexActions::class, + ], + ], + 'shelves-show' => [ + 'left' => [ + ShelvesShowTags::class, + ShelvesShowDetails::class, + ShelvesShowActivity::class, + ], + 'right' => [ + ShelvesShowActions::class, + ], + ], + 'books-index' => [ + 'left' => [ + BooksIndexRecents::class, + BooksIndexPopular::class, + BooksIndexNew::class, + ], + 'right' => [ + BooksIndexActions::class, + ], + ], + 'books-show' => [ + 'left' => [ + BooksShowSearchForm::class, + BooksShowTags::class, + BooksShowShelves::class, + BooksShowActivity::class, + ], + 'right' => [ + BooksShowDetails::class, + BooksShowActions::class, + ], + ], + 'chapters-show' => [ + 'left' => [ + ChaptersShowSearchForm::class, + ChaptersShowTags::class, + ChaptersShowBookTree::class, + ], + 'right' => [ + ChaptersShowDetails::class, + ChaptersShowActions::class, + ], + ], + 'pages-show' => [ + 'left' => [ + PagesShowTags::class, + PagesShowAttachments::class, + PagesShowPageNav::class, + PagesShowBookTree::class, + ], + 'right' => [ + PagesShowDetails::class, + PagesShowActions::class, + ], + ], + ]; + + /** + * Register a block type to be displayed at the given location and position. + * @param class-string $blockClass + */ + public function register(string $location, string $position, string $blockClass): void + { + if (!isset($this->blocksByLocationAndPosition[$location])) { + $this->blocksByLocationAndPosition[$location] = []; + } + + if (!isset($this->blocksByLocationAndPosition[$location][$position])) { + $this->blocksByLocationAndPosition[$location][$position] = []; + } + + $this->blocksByLocationAndPosition[$location][$position][] = $blockClass; + } + + /** + * Get all blocks registered for a given location and position. + * + * @return ViewBlockInterface[] + * @throws BindingResolutionException + */ + public function getForLocation(string $location, string $position): array + { + $sections = array_unique($this->blocksByLocationAndPosition[$location][$position] ?? []); + return array_map(function (string $className) { + return app()->make($className); + }, $sections); + } +} diff --git a/app/View/SidebarSections/BooksIndexActions.php b/app/View/ViewBlocks/BooksIndexActions.php similarity index 68% rename from app/View/SidebarSections/BooksIndexActions.php rename to app/View/ViewBlocks/BooksIndexActions.php index 62f856a65..fd290c601 100644 --- a/app/View/SidebarSections/BooksIndexActions.php +++ b/app/View/ViewBlocks/BooksIndexActions.php @@ -1,11 +1,11 @@ getSectionsForLocation($location, $position) as $section) +@foreach($viewBlocks->getForLocation($location, $position) as $section) @include($section->getView(), $section->withData($sectionVars, $sectionRequest)) @endforeach \ No newline at end of file From e3ebcb22508112f6e599151f3d134e93c5851209 Mon Sep 17 00:00:00 2001 From: Dan Brown Date: Sun, 9 Aug 2026 11:54:42 +0100 Subject: [PATCH 05/21] Views: Extracted homepage sections into their own views --- resources/views/home/default.blade.php | 57 ++----------------- .../default-card-recent-activity.blade.php | 6 ++ .../default-card-recent-drafts.blade.php | 11 ++++ ...ault-card-recently-updates-pages.blade.php | 16 ++++++ ...-recently-viewed-or-recent-books.blade.php | 14 +++++ .../default-card-top-favourites.blade.php | 15 +++++ .../sidebar-section-recent-activity.blade.php | 7 +++ .../sidebar-section-recent-drafts.blade.php | 9 +++ ...r-section-recently-updated-pages.blade.php | 14 +++++ ...-recently-viewed-or-recent-books.blade.php | 11 ++++ .../sidebar-section-top-favourites.blade.php | 13 +++++ resources/views/home/parts/sidebar.blade.php | 48 ++-------------- 12 files changed, 126 insertions(+), 95 deletions(-) create mode 100644 resources/views/home/parts/default-card-recent-activity.blade.php create mode 100644 resources/views/home/parts/default-card-recent-drafts.blade.php create mode 100644 resources/views/home/parts/default-card-recently-updates-pages.blade.php create mode 100644 resources/views/home/parts/default-card-recently-viewed-or-recent-books.blade.php create mode 100644 resources/views/home/parts/default-card-top-favourites.blade.php create mode 100644 resources/views/home/parts/sidebar-section-recent-activity.blade.php create mode 100644 resources/views/home/parts/sidebar-section-recent-drafts.blade.php create mode 100644 resources/views/home/parts/sidebar-section-recently-updated-pages.blade.php create mode 100644 resources/views/home/parts/sidebar-section-recently-viewed-or-recent-books.blade.php create mode 100644 resources/views/home/parts/sidebar-section-top-favourites.blade.php diff --git a/resources/views/home/default.blade.php b/resources/views/home/default.blade.php index a546ddeca..7d229d6ac 100644 --- a/resources/views/home/default.blade.php +++ b/resources/views/home/default.blade.php @@ -16,65 +16,18 @@
    - @if(count($draftPages) > 0) -
    -

    {{ trans('entities.my_recent_drafts') }}

    -
    - @include('entities.list', ['entities' => $draftPages, 'style' => 'compact']) -
    -
    - @endif - -
    -

    {{ trans('entities.' . (auth()->check() ? 'my_recently_viewed' : 'books_recent')) }}

    -
    - @include('entities.list', [ - 'entities' => $recents, - 'style' => 'compact', - 'emptyText' => auth()->check() ? trans('entities.no_pages_viewed') : trans('entities.books_empty') - ]) -
    -
    + @include('home.parts.default-card-recent-drafts', ['draftPages' => $draftPages]) + @include('home.parts.default-card-recently-viewed-or-recent-books', ['recents' => $recents])
    - @if(count($favourites) > 0) -
    -

    {{ trans('entities.my_most_viewed_favourites') }}

    -
    - @include('entities.list', [ - 'entities' => $favourites, - 'style' => 'compact', - ]) -
    - {{ trans('common.view_all') }} -
    - @endif - -
    -

    {{ trans('entities.recently_updated_pages') }}

    -
    - @include('entities.list', [ - 'entities' => $recentlyUpdatedPages, - 'style' => 'compact', - 'emptyText' => trans('entities.no_pages_recently_updated'), - ]) -
    - @if(count($recentlyUpdatedPages) > 0) - {{ trans('common.view_all') }} - @endif -
    + @include('home.parts.default-card-top-favourites', ['favourites' => $favourites]) + @include('home.parts.default-card-recently-updates-pages', ['recentlyUpdatedPages' => $recentlyUpdatedPages])
    -
    -

    {{ trans('entities.recent_activity') }}

    -
    - @include('common.activity-list', ['activity' => $activity]) -
    -
    + @include('home.parts.default-card-recent-activity', ['activity' => $activity])
    -
    diff --git a/resources/views/home/parts/default-card-recent-activity.blade.php b/resources/views/home/parts/default-card-recent-activity.blade.php new file mode 100644 index 000000000..035cd9987 --- /dev/null +++ b/resources/views/home/parts/default-card-recent-activity.blade.php @@ -0,0 +1,6 @@ +
    +

    {{ trans('entities.recent_activity') }}

    +
    + @include('common.activity-list', ['activity' => $activity]) +
    +
    j \ No newline at end of file diff --git a/resources/views/home/parts/default-card-recent-drafts.blade.php b/resources/views/home/parts/default-card-recent-drafts.blade.php new file mode 100644 index 000000000..d9d7a5130 --- /dev/null +++ b/resources/views/home/parts/default-card-recent-drafts.blade.php @@ -0,0 +1,11 @@ +{{-- +$draftPages - array +--}} +@if(count($draftPages) > 0) +
    +

    {{ trans('entities.my_recent_drafts') }}

    +
    + @include('entities.list', ['entities' => $draftPages, 'style' => 'compact']) +
    +
    +@endif diff --git a/resources/views/home/parts/default-card-recently-updates-pages.blade.php b/resources/views/home/parts/default-card-recently-updates-pages.blade.php new file mode 100644 index 000000000..6e637d710 --- /dev/null +++ b/resources/views/home/parts/default-card-recently-updates-pages.blade.php @@ -0,0 +1,16 @@ +{{-- +$recentlyUpdatedPages - array +--}} +
    +

    {{ trans('entities.recently_updated_pages') }}

    +
    + @include('entities.list', [ + 'entities' => $recentlyUpdatedPages, + 'style' => 'compact', + 'emptyText' => trans('entities.no_pages_recently_updated'), + ]) +
    + @if(count($recentlyUpdatedPages) > 0) + {{ trans('common.view_all') }} + @endif +
    \ No newline at end of file diff --git a/resources/views/home/parts/default-card-recently-viewed-or-recent-books.blade.php b/resources/views/home/parts/default-card-recently-viewed-or-recent-books.blade.php new file mode 100644 index 000000000..31bafaa2b --- /dev/null +++ b/resources/views/home/parts/default-card-recently-viewed-or-recent-books.blade.php @@ -0,0 +1,14 @@ +{{-- +$recents - array +--}} + +
    +

    {{ trans('entities.' . (auth()->check() ? 'my_recently_viewed' : 'books_recent')) }}

    +
    + @include('entities.list', [ + 'entities' => $recents, + 'style' => 'compact', + 'emptyText' => auth()->check() ? trans('entities.no_pages_viewed') : trans('entities.books_empty') + ]) +
    +
    \ No newline at end of file diff --git a/resources/views/home/parts/default-card-top-favourites.blade.php b/resources/views/home/parts/default-card-top-favourites.blade.php new file mode 100644 index 000000000..81f3895c6 --- /dev/null +++ b/resources/views/home/parts/default-card-top-favourites.blade.php @@ -0,0 +1,15 @@ +{{-- +$favourites - array +--}} +@if(count($favourites) > 0) +
    +

    {{ trans('entities.my_most_viewed_favourites') }}

    +
    + @include('entities.list', [ + 'entities' => $favourites, + 'style' => 'compact', + ]) +
    + {{ trans('common.view_all') }} +
    +@endif \ No newline at end of file diff --git a/resources/views/home/parts/sidebar-section-recent-activity.blade.php b/resources/views/home/parts/sidebar-section-recent-activity.blade.php new file mode 100644 index 000000000..8a82f9006 --- /dev/null +++ b/resources/views/home/parts/sidebar-section-recent-activity.blade.php @@ -0,0 +1,7 @@ +{{-- +$activity - array +--}} +
    +
    {{ trans('entities.recent_activity') }}
    + @include('common.activity-list', ['activity' => $activity]) +
    \ No newline at end of file diff --git a/resources/views/home/parts/sidebar-section-recent-drafts.blade.php b/resources/views/home/parts/sidebar-section-recent-drafts.blade.php new file mode 100644 index 000000000..648f54155 --- /dev/null +++ b/resources/views/home/parts/sidebar-section-recent-drafts.blade.php @@ -0,0 +1,9 @@ +{{-- +$draftPages - array +--}} +@if(count($draftPages) > 0) +
    +
    {{ trans('entities.my_recent_drafts') }}
    + @include('entities.list', ['entities' => $draftPages, 'style' => 'compact']) +
    +@endif \ No newline at end of file diff --git a/resources/views/home/parts/sidebar-section-recently-updated-pages.blade.php b/resources/views/home/parts/sidebar-section-recently-updated-pages.blade.php new file mode 100644 index 000000000..290c385a1 --- /dev/null +++ b/resources/views/home/parts/sidebar-section-recently-updated-pages.blade.php @@ -0,0 +1,14 @@ +{{-- +$recentlyUpdatedPages - array +--}} +
    +
    {{ trans('entities.recently_updated_pages') }}
    +
    + @include('entities.list', [ + 'entities' => $recentlyUpdatedPages, + 'style' => 'compact', + 'emptyText' => trans('entities.no_pages_recently_updated') + ]) +
    + {{ trans('common.view_all') }} +
    \ No newline at end of file diff --git a/resources/views/home/parts/sidebar-section-recently-viewed-or-recent-books.blade.php b/resources/views/home/parts/sidebar-section-recently-viewed-or-recent-books.blade.php new file mode 100644 index 000000000..d9c6edea8 --- /dev/null +++ b/resources/views/home/parts/sidebar-section-recently-viewed-or-recent-books.blade.php @@ -0,0 +1,11 @@ +{{-- +$recents - array +--}} +
    +
    {{ trans('entities.' . (auth()->check() ? 'my_recently_viewed' : 'books_recent')) }}
    + @include('entities.list', [ + 'entities' => $recents, + 'style' => 'compact', + 'emptyText' => auth()->check() ? trans('entities.no_pages_viewed') : trans('entities.books_empty') + ]) +
    \ No newline at end of file diff --git a/resources/views/home/parts/sidebar-section-top-favourites.blade.php b/resources/views/home/parts/sidebar-section-top-favourites.blade.php new file mode 100644 index 000000000..09b33b176 --- /dev/null +++ b/resources/views/home/parts/sidebar-section-top-favourites.blade.php @@ -0,0 +1,13 @@ +{{-- +$favourites - array +--}} +@if(count($favourites) > 0) +
    +
    {{ trans('entities.my_most_viewed_favourites') }}
    + @include('entities.list', [ + 'entities' => $favourites, + 'style' => 'compact', + ]) + {{ trans('common.view_all') }} +
    +@endif \ No newline at end of file diff --git a/resources/views/home/parts/sidebar.blade.php b/resources/views/home/parts/sidebar.blade.php index 78f7e7a80..07c199f9e 100644 --- a/resources/views/home/parts/sidebar.blade.php +++ b/resources/views/home/parts/sidebar.blade.php @@ -1,43 +1,5 @@ -@if(count($draftPages) > 0) -
    -
    {{ trans('entities.my_recent_drafts') }}
    - @include('entities.list', ['entities' => $draftPages, 'style' => 'compact']) -
    -@endif - -@if(count($favourites) > 0) -
    -
    {{ trans('entities.my_most_viewed_favourites') }}
    - @include('entities.list', [ - 'entities' => $favourites, - 'style' => 'compact', - ]) - {{ trans('common.view_all') }} -
    -@endif - -
    -
    {{ trans('entities.' . (auth()->check() ? 'my_recently_viewed' : 'books_recent')) }}
    - @include('entities.list', [ - 'entities' => $recents, - 'style' => 'compact', - 'emptyText' => auth()->check() ? trans('entities.no_pages_viewed') : trans('entities.books_empty') - ]) -
    - -
    -
    {{ trans('entities.recently_updated_pages') }}
    -
    - @include('entities.list', [ - 'entities' => $recentlyUpdatedPages, - 'style' => 'compact', - 'emptyText' => trans('entities.no_pages_recently_updated') - ]) -
    - {{ trans('common.view_all') }} -
    - -
    -
    {{ trans('entities.recent_activity') }}
    - @include('common.activity-list', ['activity' => $activity]) -
    \ No newline at end of file +@include('home.parts.sidebar-section-recent-drafts', ['draftPages' => $draftPages]) +@include('home.parts.sidebar-section-top-favourites', ['favourites' => $favourites]) +@include('home.parts.sidebar-section-recently-viewed-or-recent-books', ['recents' => $recents]) +@include('home.parts.sidebar-section-recently-updated-pages', ['recentlyUpdatedPages' => $recentlyUpdatedPages]) +@include('home.parts.sidebar-section-recent-activity', ['activity' => $activity]) \ No newline at end of file From 600497b1109d65c3c3c7853dc3a924b75e6e124b Mon Sep 17 00:00:00 2001 From: Dan Brown Date: Sun, 9 Aug 2026 15:12:07 +0100 Subject: [PATCH 06/21] Views: Extracted home view sections into blocks Also removed unused request from block interface. --- app/App/HomeController.php | 51 ++++--------------- app/View/ViewBlock.php | 2 +- app/View/ViewBlockInterface.php | 7 ++- app/View/ViewBlockManager.php | 31 +++++++++++ app/View/ViewBlocks/BooksIndexActions.php | 2 +- app/View/ViewBlocks/BooksIndexNew.php | 2 +- app/View/ViewBlocks/BooksIndexPopular.php | 2 +- app/View/ViewBlocks/BooksIndexRecents.php | 2 +- app/View/ViewBlocks/BooksShowActions.php | 2 +- app/View/ViewBlocks/BooksShowActivity.php | 2 +- app/View/ViewBlocks/BooksShowDetails.php | 2 +- app/View/ViewBlocks/BooksShowSearchForm.php | 2 +- app/View/ViewBlocks/BooksShowShelves.php | 2 +- app/View/ViewBlocks/BooksShowTags.php | 2 +- app/View/ViewBlocks/ChaptersShowActions.php | 2 +- app/View/ViewBlocks/ChaptersShowBookTree.php | 2 +- app/View/ViewBlocks/ChaptersShowDetails.php | 2 +- .../ViewBlocks/ChaptersShowSearchForm.php | 2 +- app/View/ViewBlocks/ChaptersShowTags.php | 2 +- app/View/ViewBlocks/HomeActions.php | 19 +++++++ app/View/ViewBlocks/HomeRecentActivity.php | 32 ++++++++++++ app/View/ViewBlocks/HomeRecentDrafts.php | 40 +++++++++++++++ .../ViewBlocks/HomeRecentlyUpdatedPages.php | 37 ++++++++++++++ .../HomeRecentlyViewedOrRecentBooks.php | 43 ++++++++++++++++ app/View/ViewBlocks/HomeTopFavourites.php | 32 ++++++++++++ app/View/ViewBlocks/PagesShowActions.php | 2 +- app/View/ViewBlocks/PagesShowAttachments.php | 2 +- app/View/ViewBlocks/PagesShowBookTree.php | 2 +- app/View/ViewBlocks/PagesShowDetails.php | 2 +- app/View/ViewBlocks/PagesShowPageNav.php | 2 +- app/View/ViewBlocks/PagesShowTags.php | 2 +- app/View/ViewBlocks/ShelvesIndexActions.php | 2 +- app/View/ViewBlocks/ShelvesIndexNew.php | 2 +- app/View/ViewBlocks/ShelvesIndexPopular.php | 2 +- app/View/ViewBlocks/ShelvesIndexRecents.php | 2 +- app/View/ViewBlocks/ShelvesShowActions.php | 2 +- app/View/ViewBlocks/ShelvesShowActivity.php | 2 +- app/View/ViewBlocks/ShelvesShowDetails.php | 2 +- app/View/ViewBlocks/ShelvesShowTags.php | 2 +- resources/views/books/index.blade.php | 4 +- resources/views/books/show.blade.php | 4 +- resources/views/chapters/show.blade.php | 4 +- ...ctions.blade.php => view-blocks.blade.php} | 3 +- resources/views/home/books.blade.php | 21 +------- resources/views/home/default.blade.php | 8 ++- .../configured-section-actions.blade.php | 36 +++++++++++++ ...figured-section-recent-activity.blade.php} | 0 ...onfigured-section-recent-drafts.blade.php} | 0 ...-section-recently-updated-pages.blade.php} | 0 ...recently-viewed-or-recent-books.blade.php} | 0 ...nfigured-section-top-favourites.blade.php} | 0 .../default-card-recent-activity.blade.php | 5 +- ...ult-card-recently-updated-pages.blade.php} | 0 resources/views/home/parts/sidebar.blade.php | 5 -- resources/views/home/shelves.blade.php | 21 +------- resources/views/home/specific-page.blade.php | 10 +--- resources/views/pages/show.blade.php | 4 +- resources/views/shelves/index.blade.php | 4 +- resources/views/shelves/show.blade.php | 4 +- 59 files changed, 338 insertions(+), 147 deletions(-) create mode 100644 app/View/ViewBlocks/HomeActions.php create mode 100644 app/View/ViewBlocks/HomeRecentActivity.php create mode 100644 app/View/ViewBlocks/HomeRecentDrafts.php create mode 100644 app/View/ViewBlocks/HomeRecentlyUpdatedPages.php create mode 100644 app/View/ViewBlocks/HomeRecentlyViewedOrRecentBooks.php create mode 100644 app/View/ViewBlocks/HomeTopFavourites.php rename resources/views/common/{sidebar-sections.blade.php => view-blocks.blade.php} (60%) create mode 100644 resources/views/home/parts/configured-section-actions.blade.php rename resources/views/home/parts/{sidebar-section-recent-activity.blade.php => configured-section-recent-activity.blade.php} (100%) rename resources/views/home/parts/{sidebar-section-recent-drafts.blade.php => configured-section-recent-drafts.blade.php} (100%) rename resources/views/home/parts/{sidebar-section-recently-updated-pages.blade.php => configured-section-recently-updated-pages.blade.php} (100%) rename resources/views/home/parts/{sidebar-section-recently-viewed-or-recent-books.blade.php => configured-section-recently-viewed-or-recent-books.blade.php} (100%) rename resources/views/home/parts/{sidebar-section-top-favourites.blade.php => configured-section-top-favourites.blade.php} (100%) rename resources/views/home/parts/{default-card-recently-updates-pages.blade.php => default-card-recently-updated-pages.blade.php} (100%) delete mode 100644 resources/views/home/parts/sidebar.blade.php diff --git a/app/App/HomeController.php b/app/App/HomeController.php index 00e2db3df..bbff07902 100644 --- a/app/App/HomeController.php +++ b/app/App/HomeController.php @@ -2,11 +2,8 @@ namespace BookStack\App; -use BookStack\Activity\ActivityQueries; use BookStack\Entities\Models\Page; use BookStack\Entities\Queries\EntityQueries; -use BookStack\Entities\Queries\QueryRecentlyViewed; -use BookStack\Entities\Queries\QueryTopFavourites; use BookStack\Entities\Tools\PageContent; use BookStack\Http\Controller; use BookStack\Util\SimpleListOptions; @@ -24,49 +21,19 @@ class HomeController extends Controller */ public function index( Request $request, - ActivityQueries $activities, - QueryRecentlyViewed $recentlyViewed, - QueryTopFavourites $topFavourites, ) { - $activity = $activities->latest(10); - $draftPages = []; - - if ($this->isSignedIn()) { - $draftPages = $this->queries->pages->currentUserDraftsForList() - ->orderBy('updated_at', 'desc') - ->with('book') - ->take(6) - ->get(); - } - - $recentFactor = count($draftPages) > 0 ? 0.5 : 1; - $recents = $this->isSignedIn() ? - $recentlyViewed->run(12 * $recentFactor, 1) - : $this->queries->books->visibleForList()->orderBy('created_at', 'desc')->take(12 * $recentFactor)->get(); - $favourites = $topFavourites->run(6); - $recentlyUpdatedPages = $this->queries->pages->visibleForList() - ->where('draft', false) - ->orderBy('updated_at', 'desc') - ->take($favourites->count() > 0 ? 5 : 10) - ->get(); - - $homepageOptions = ['default', 'books', 'bookshelves', 'page']; - $homepageOption = setting('app-homepage-type', 'default'); - if (!in_array($homepageOption, $homepageOptions)) { - $homepageOption = 'default'; + $homepageType = setting('app-homepage-type', 'default'); + if (!in_array($homepageType, ['default', 'books', 'bookshelves', 'page'])) { + $homepageType = 'default'; } $commonData = [ - 'activity' => $activity, - 'recents' => $recents, - 'recentlyUpdatedPages' => $recentlyUpdatedPages, - 'draftPages' => $draftPages, - 'favourites' => $favourites, + 'homeView' => $homepageType, ]; // Add required list ordering & sorting for books & shelves views. - if ($homepageOption === 'bookshelves' || $homepageOption === 'books') { - $key = $homepageOption; + if ($homepageType === 'bookshelves' || $homepageType === 'books') { + $key = $homepageType; $view = setting()->getForCurrentUser($key . '_view_type'); $listOptions = SimpleListOptions::fromRequest($request, $key)->withSortOptions([ 'name' => trans('common.sort_name'), @@ -80,7 +47,7 @@ class HomeController extends Controller ]); } - if ($homepageOption === 'bookshelves') { + if ($homepageType === 'bookshelves') { $shelves = $this->queries->shelves->visibleForListWithCover() ->orderBy($commonData['listOptions']->getSort(), $commonData['listOptions']->getOrder()) ->paginate(setting()->getInteger('lists-page-count-shelves', 18, 1, 1000)); @@ -89,7 +56,7 @@ class HomeController extends Controller return view('home.shelves', $data); } - if ($homepageOption === 'books') { + if ($homepageType === 'books') { $books = $this->queries->books->visibleForListWithCover() ->orderBy($commonData['listOptions']->getSort(), $commonData['listOptions']->getOrder()) ->paginate(setting()->getInteger('lists-page-count-books', 18, 1, 1000)); @@ -98,7 +65,7 @@ class HomeController extends Controller return view('home.books', $data); } - if ($homepageOption === 'page') { + if ($homepageType === 'page') { $homepageSetting = setting('app-homepage', '0:'); $id = intval(explode(':', $homepageSetting)[0]); /** @var Page $customHomepage */ diff --git a/app/View/ViewBlock.php b/app/View/ViewBlock.php index a835e7075..636746044 100644 --- a/app/View/ViewBlock.php +++ b/app/View/ViewBlock.php @@ -9,7 +9,7 @@ abstract class ViewBlock implements ViewBlockInterface /** * @inheritDoc */ - public function getView(): string + public function getView(array $viewData): string { return $this->view; } diff --git a/app/View/ViewBlockInterface.php b/app/View/ViewBlockInterface.php index b9c25e9f6..756cf4298 100644 --- a/app/View/ViewBlockInterface.php +++ b/app/View/ViewBlockInterface.php @@ -2,19 +2,18 @@ namespace BookStack\View; -use Illuminate\Http\Request; - interface ViewBlockInterface { /** * Get the view to render for this block. + * Is provided with the existing parent view data. */ - public function getView(): string; + public function getView(array $viewData): string; /** * Specify the data to pass to the view on render. * Is provided with the existing parent view data and the original request. * @return array */ - public function withData(array $viewData, Request $request): array; + public function withData(array $viewData): array; } diff --git a/app/View/ViewBlockManager.php b/app/View/ViewBlockManager.php index 88d866135..7287c0293 100644 --- a/app/View/ViewBlockManager.php +++ b/app/View/ViewBlockManager.php @@ -17,6 +17,12 @@ use BookStack\View\ViewBlocks\ChaptersShowBookTree; use BookStack\View\ViewBlocks\ChaptersShowDetails; use BookStack\View\ViewBlocks\ChaptersShowSearchForm; use BookStack\View\ViewBlocks\ChaptersShowTags; +use BookStack\View\ViewBlocks\HomeActions; +use BookStack\View\ViewBlocks\HomeRecentActivity; +use BookStack\View\ViewBlocks\HomeRecentDrafts; +use BookStack\View\ViewBlocks\HomeRecentlyUpdatedPages; +use BookStack\View\ViewBlocks\HomeRecentlyViewedOrRecentBooks; +use BookStack\View\ViewBlocks\HomeTopFavourites; use BookStack\View\ViewBlocks\PagesShowActions; use BookStack\View\ViewBlocks\PagesShowAttachments; use BookStack\View\ViewBlocks\PagesShowBookTree; @@ -39,6 +45,31 @@ class ViewBlockManager * @var array[]>> */ protected array $blocksByLocationAndPosition = [ + 'home-default' => [ + 'left' => [ + HomeRecentDrafts::class, + HomeRecentlyViewedOrRecentBooks::class, + ], + 'center' => [ + HomeTopFavourites::class, + HomeRecentlyUpdatedPages::class, + ], + 'right' => [ + HomeRecentActivity::class, + ], + ], + 'home-non-default' => [ + 'left' => [ + HomeRecentDrafts::class, + HomeTopFavourites::class, + HomeRecentlyViewedOrRecentBooks::class, + HomeRecentlyUpdatedPages::class, + HomeRecentActivity::class, + ], + 'right' => [ + HomeActions::class, + ], + ], 'shelves-index' => [ 'left' => [ ShelvesIndexRecents::class, diff --git a/app/View/ViewBlocks/BooksIndexActions.php b/app/View/ViewBlocks/BooksIndexActions.php index fd290c601..8e6721ddf 100644 --- a/app/View/ViewBlocks/BooksIndexActions.php +++ b/app/View/ViewBlocks/BooksIndexActions.php @@ -9,7 +9,7 @@ class BooksIndexActions extends ViewBlock { protected string $view = 'books.parts.index-sidebar-section-actions'; - public function withData(array $viewData, Request $request): array + public function withData(array $viewData): array { return [ 'view' => $viewData['view'], diff --git a/app/View/ViewBlocks/BooksIndexNew.php b/app/View/ViewBlocks/BooksIndexNew.php index dee863b36..76d2750ba 100644 --- a/app/View/ViewBlocks/BooksIndexNew.php +++ b/app/View/ViewBlocks/BooksIndexNew.php @@ -15,7 +15,7 @@ class BooksIndexNew extends ViewBlock ) { } - public function withData(array $viewData, Request $request): array + public function withData(array $viewData): array { $new = $this->queries->visibleForList() ->orderBy('created_at', 'desc') diff --git a/app/View/ViewBlocks/BooksIndexPopular.php b/app/View/ViewBlocks/BooksIndexPopular.php index cb478170d..ee3138f79 100644 --- a/app/View/ViewBlocks/BooksIndexPopular.php +++ b/app/View/ViewBlocks/BooksIndexPopular.php @@ -15,7 +15,7 @@ class BooksIndexPopular extends ViewBlock ) { } - public function withData(array $viewData, Request $request): array + public function withData(array $viewData): array { return [ 'popular' => $this->queries->popularForList()->take(4)->get(), diff --git a/app/View/ViewBlocks/BooksIndexRecents.php b/app/View/ViewBlocks/BooksIndexRecents.php index e3a632e54..1c7b865d8 100644 --- a/app/View/ViewBlocks/BooksIndexRecents.php +++ b/app/View/ViewBlocks/BooksIndexRecents.php @@ -15,7 +15,7 @@ class BooksIndexRecents extends ViewBlock ) { } - public function withData(array $viewData, Request $request): array + public function withData(array $viewData): array { $recents = null; if (!user()->isGuest()) { diff --git a/app/View/ViewBlocks/BooksShowActions.php b/app/View/ViewBlocks/BooksShowActions.php index 43f63c24e..155c0c7ba 100644 --- a/app/View/ViewBlocks/BooksShowActions.php +++ b/app/View/ViewBlocks/BooksShowActions.php @@ -11,7 +11,7 @@ class BooksShowActions extends ViewBlock { protected string $view = 'books.parts.show-sidebar-section-actions'; - public function withData(array $viewData, Request $request): array + public function withData(array $viewData): array { /** @var Book $book */ $book = $viewData['book']; diff --git a/app/View/ViewBlocks/BooksShowActivity.php b/app/View/ViewBlocks/BooksShowActivity.php index 4ba8697c5..ecf626df5 100644 --- a/app/View/ViewBlocks/BooksShowActivity.php +++ b/app/View/ViewBlocks/BooksShowActivity.php @@ -16,7 +16,7 @@ class BooksShowActivity extends ViewBlock ) { } - public function withData(array $viewData, Request $request): array + public function withData(array $viewData): array { /** @var Book $book */ $book = $viewData['book']; diff --git a/app/View/ViewBlocks/BooksShowDetails.php b/app/View/ViewBlocks/BooksShowDetails.php index 12699d5a0..9a0ef8b47 100644 --- a/app/View/ViewBlocks/BooksShowDetails.php +++ b/app/View/ViewBlocks/BooksShowDetails.php @@ -17,7 +17,7 @@ class BooksShowDetails extends ViewBlock ) { } - public function withData(array $viewData, Request $request): array + public function withData(array $viewData): array { /** @var Book $book */ $book = $viewData['book']; diff --git a/app/View/ViewBlocks/BooksShowSearchForm.php b/app/View/ViewBlocks/BooksShowSearchForm.php index d75db3d93..061875d3d 100644 --- a/app/View/ViewBlocks/BooksShowSearchForm.php +++ b/app/View/ViewBlocks/BooksShowSearchForm.php @@ -9,7 +9,7 @@ class BooksShowSearchForm extends ViewBlock { protected string $view = 'entities.search-form'; - public function withData(array $viewData, Request $request): array + public function withData(array $viewData): array { return [ 'label' => trans('entities.books_search_this'), diff --git a/app/View/ViewBlocks/BooksShowShelves.php b/app/View/ViewBlocks/BooksShowShelves.php index 48a3208fc..42d832589 100644 --- a/app/View/ViewBlocks/BooksShowShelves.php +++ b/app/View/ViewBlocks/BooksShowShelves.php @@ -10,7 +10,7 @@ class BooksShowShelves extends ViewBlock { protected string $view = 'books.parts.show-sidebar-section-shelves'; - public function withData(array $viewData, Request $request): array + public function withData(array $viewData): array { /** @var Book $book */ $book = $viewData['book']; diff --git a/app/View/ViewBlocks/BooksShowTags.php b/app/View/ViewBlocks/BooksShowTags.php index 60603b91a..5c5aa8fd9 100644 --- a/app/View/ViewBlocks/BooksShowTags.php +++ b/app/View/ViewBlocks/BooksShowTags.php @@ -9,7 +9,7 @@ class BooksShowTags extends ViewBlock { protected string $view = 'books.parts.show-sidebar-section-tags'; - public function withData(array $viewData, Request $request): array + public function withData(array $viewData): array { return [ 'book' => $viewData['book'], diff --git a/app/View/ViewBlocks/ChaptersShowActions.php b/app/View/ViewBlocks/ChaptersShowActions.php index e75b86a1e..9b6137a20 100644 --- a/app/View/ViewBlocks/ChaptersShowActions.php +++ b/app/View/ViewBlocks/ChaptersShowActions.php @@ -11,7 +11,7 @@ class ChaptersShowActions extends ViewBlock { protected string $view = 'chapters.parts.show-sidebar-section-actions'; - public function withData(array $viewData, Request $request): array + public function withData(array $viewData): array { /** @var Chapter $chapter */ $chapter = $viewData['chapter']; diff --git a/app/View/ViewBlocks/ChaptersShowBookTree.php b/app/View/ViewBlocks/ChaptersShowBookTree.php index 79e65908b..827e8d917 100644 --- a/app/View/ViewBlocks/ChaptersShowBookTree.php +++ b/app/View/ViewBlocks/ChaptersShowBookTree.php @@ -10,7 +10,7 @@ class ChaptersShowBookTree extends ViewBlock { protected string $view = 'entities.book-tree'; - public function withData(array $viewData, Request $request): array + public function withData(array $viewData): array { /** @var Book $book */ $book = $viewData['book']; diff --git a/app/View/ViewBlocks/ChaptersShowDetails.php b/app/View/ViewBlocks/ChaptersShowDetails.php index ed7efaa89..fd235fbe1 100644 --- a/app/View/ViewBlocks/ChaptersShowDetails.php +++ b/app/View/ViewBlocks/ChaptersShowDetails.php @@ -18,7 +18,7 @@ class ChaptersShowDetails extends ViewBlock ) { } - public function withData(array $viewData, Request $request): array + public function withData(array $viewData): array { /** @var Book $book */ $book = $viewData['book']; diff --git a/app/View/ViewBlocks/ChaptersShowSearchForm.php b/app/View/ViewBlocks/ChaptersShowSearchForm.php index ad671a2d3..92664e5f5 100644 --- a/app/View/ViewBlocks/ChaptersShowSearchForm.php +++ b/app/View/ViewBlocks/ChaptersShowSearchForm.php @@ -9,7 +9,7 @@ class ChaptersShowSearchForm extends ViewBlock { protected string $view = 'entities.search-form'; - public function withData(array $viewData, Request $request): array + public function withData(array $viewData): array { return [ 'label' => trans('entities.chapters_search_this'), diff --git a/app/View/ViewBlocks/ChaptersShowTags.php b/app/View/ViewBlocks/ChaptersShowTags.php index 369946e2b..198604eee 100644 --- a/app/View/ViewBlocks/ChaptersShowTags.php +++ b/app/View/ViewBlocks/ChaptersShowTags.php @@ -10,7 +10,7 @@ class ChaptersShowTags extends ViewBlock { protected string $view = 'chapters.parts.show-sidebar-section-tags'; - public function withData(array $viewData, Request $request): array + public function withData(array $viewData): array { /** @var Chapter $chapter */ $chapter = $viewData['chapter']; diff --git a/app/View/ViewBlocks/HomeActions.php b/app/View/ViewBlocks/HomeActions.php new file mode 100644 index 000000000..f4e4295b8 --- /dev/null +++ b/app/View/ViewBlocks/HomeActions.php @@ -0,0 +1,19 @@ + $viewData['view'] ?? '', + 'homeView' => $viewData['homeView'] ?? 'default', + ]; + } +} diff --git a/app/View/ViewBlocks/HomeRecentActivity.php b/app/View/ViewBlocks/HomeRecentActivity.php new file mode 100644 index 000000000..1f915061c --- /dev/null +++ b/app/View/ViewBlocks/HomeRecentActivity.php @@ -0,0 +1,32 @@ +activityQueries->latest(10); + return [ + 'activity' => $activity, + ]; + } +} diff --git a/app/View/ViewBlocks/HomeRecentDrafts.php b/app/View/ViewBlocks/HomeRecentDrafts.php new file mode 100644 index 000000000..71586d83a --- /dev/null +++ b/app/View/ViewBlocks/HomeRecentDrafts.php @@ -0,0 +1,40 @@ +isGuest()) { + $draftPages = $this->pageQueries->currentUserDraftsForList() + ->orderBy('updated_at', 'desc') + ->with('book') + ->take(6) + ->get(); + } + + return [ + 'draftPages' => $draftPages, + ]; + } +} diff --git a/app/View/ViewBlocks/HomeRecentlyUpdatedPages.php b/app/View/ViewBlocks/HomeRecentlyUpdatedPages.php new file mode 100644 index 000000000..3f669ee6e --- /dev/null +++ b/app/View/ViewBlocks/HomeRecentlyUpdatedPages.php @@ -0,0 +1,37 @@ +queries->visibleForList() + ->where('draft', false) + ->orderBy('updated_at', 'desc') + ->take(8) + ->get(); + + return [ + 'recentlyUpdatedPages' => $recentlyUpdatedPages, + ]; + } +} diff --git a/app/View/ViewBlocks/HomeRecentlyViewedOrRecentBooks.php b/app/View/ViewBlocks/HomeRecentlyViewedOrRecentBooks.php new file mode 100644 index 000000000..7db3d7935 --- /dev/null +++ b/app/View/ViewBlocks/HomeRecentlyViewedOrRecentBooks.php @@ -0,0 +1,43 @@ +isGuest()) { + $recents = $this->queries->books->visibleForList() + ->orderBy('created_at', 'desc') + ->take(10) + ->get(); + } else { + $recents = $this->recentlyViewed->run(10, 1); + } + + return [ + 'recents' => $recents + ]; + } +} diff --git a/app/View/ViewBlocks/HomeTopFavourites.php b/app/View/ViewBlocks/HomeTopFavourites.php new file mode 100644 index 000000000..9ce84aa3d --- /dev/null +++ b/app/View/ViewBlocks/HomeTopFavourites.php @@ -0,0 +1,32 @@ +topFavourites->run(6); + return [ + 'favourites' => $favourites + ]; + } +} diff --git a/app/View/ViewBlocks/PagesShowActions.php b/app/View/ViewBlocks/PagesShowActions.php index b09a64081..5486b59a4 100644 --- a/app/View/ViewBlocks/PagesShowActions.php +++ b/app/View/ViewBlocks/PagesShowActions.php @@ -11,7 +11,7 @@ class PagesShowActions extends ViewBlock { protected string $view = 'pages.parts.show-sidebar-section-actions'; - public function withData(array $viewData, Request $request): array + public function withData(array $viewData): array { /** @var Page $page */ $page = $viewData['page']; diff --git a/app/View/ViewBlocks/PagesShowAttachments.php b/app/View/ViewBlocks/PagesShowAttachments.php index 43e076671..5d0cd23ba 100644 --- a/app/View/ViewBlocks/PagesShowAttachments.php +++ b/app/View/ViewBlocks/PagesShowAttachments.php @@ -10,7 +10,7 @@ class PagesShowAttachments extends ViewBlock { protected string $view = 'pages.parts.show-sidebar-section-attachments'; - public function withData(array $viewData, Request $request): array + public function withData(array $viewData): array { /** @var Page $page */ $page = $viewData['page']; diff --git a/app/View/ViewBlocks/PagesShowBookTree.php b/app/View/ViewBlocks/PagesShowBookTree.php index 4631e4ef4..b0947a208 100644 --- a/app/View/ViewBlocks/PagesShowBookTree.php +++ b/app/View/ViewBlocks/PagesShowBookTree.php @@ -10,7 +10,7 @@ class PagesShowBookTree extends ViewBlock { protected string $view = 'entities.book-tree'; - public function withData(array $viewData, Request $request): array + public function withData(array $viewData): array { /** @var Book $book */ $book = $viewData['book']; diff --git a/app/View/ViewBlocks/PagesShowDetails.php b/app/View/ViewBlocks/PagesShowDetails.php index 9fcaadac9..2dfd61417 100644 --- a/app/View/ViewBlocks/PagesShowDetails.php +++ b/app/View/ViewBlocks/PagesShowDetails.php @@ -18,7 +18,7 @@ class PagesShowDetails extends ViewBlock ) { } - public function withData(array $viewData, Request $request): array + public function withData(array $viewData): array { /** @var Book $book */ $book = $viewData['book']; diff --git a/app/View/ViewBlocks/PagesShowPageNav.php b/app/View/ViewBlocks/PagesShowPageNav.php index c43b4d7f8..dbee578ab 100644 --- a/app/View/ViewBlocks/PagesShowPageNav.php +++ b/app/View/ViewBlocks/PagesShowPageNav.php @@ -11,7 +11,7 @@ class PagesShowPageNav extends ViewBlock { protected string $view = 'pages.parts.show-sidebar-section-page-nav'; - public function withData(array $viewData, Request $request): array + public function withData(array $viewData): array { /** @var Page $page */ $page = $viewData['page']; diff --git a/app/View/ViewBlocks/PagesShowTags.php b/app/View/ViewBlocks/PagesShowTags.php index bee6846f1..a2197a161 100644 --- a/app/View/ViewBlocks/PagesShowTags.php +++ b/app/View/ViewBlocks/PagesShowTags.php @@ -10,7 +10,7 @@ class PagesShowTags extends ViewBlock { protected string $view = 'pages.parts.show-sidebar-section-tags'; - public function withData(array $viewData, Request $request): array + public function withData(array $viewData): array { /** @var Page $page */ $page = $viewData['page']; diff --git a/app/View/ViewBlocks/ShelvesIndexActions.php b/app/View/ViewBlocks/ShelvesIndexActions.php index 51d4b3a89..2cc436116 100644 --- a/app/View/ViewBlocks/ShelvesIndexActions.php +++ b/app/View/ViewBlocks/ShelvesIndexActions.php @@ -9,7 +9,7 @@ class ShelvesIndexActions extends ViewBlock { protected string $view = 'shelves.parts.index-sidebar-section-actions'; - public function withData(array $viewData, Request $request): array + public function withData(array $viewData): array { return [ 'view' => $viewData['view'], diff --git a/app/View/ViewBlocks/ShelvesIndexNew.php b/app/View/ViewBlocks/ShelvesIndexNew.php index f3ad4a650..1d6af72c0 100644 --- a/app/View/ViewBlocks/ShelvesIndexNew.php +++ b/app/View/ViewBlocks/ShelvesIndexNew.php @@ -15,7 +15,7 @@ class ShelvesIndexNew extends ViewBlock ) { } - public function withData(array $viewData, Request $request): array + public function withData(array $viewData): array { $new = $this->queries->visibleForList() ->orderBy('created_at', 'desc') diff --git a/app/View/ViewBlocks/ShelvesIndexPopular.php b/app/View/ViewBlocks/ShelvesIndexPopular.php index 5e880e18e..fc2950b92 100644 --- a/app/View/ViewBlocks/ShelvesIndexPopular.php +++ b/app/View/ViewBlocks/ShelvesIndexPopular.php @@ -15,7 +15,7 @@ class ShelvesIndexPopular extends ViewBlock ) { } - public function withData(array $viewData, Request $request): array + public function withData(array $viewData): array { return [ 'popular' => $this->queries->popularForList()->take(4)->get(), diff --git a/app/View/ViewBlocks/ShelvesIndexRecents.php b/app/View/ViewBlocks/ShelvesIndexRecents.php index 77e3a2af4..6570841a9 100644 --- a/app/View/ViewBlocks/ShelvesIndexRecents.php +++ b/app/View/ViewBlocks/ShelvesIndexRecents.php @@ -15,7 +15,7 @@ class ShelvesIndexRecents extends ViewBlock ) { } - public function withData(array $viewData, Request $request): array + public function withData(array $viewData): array { $recents = null; if (!user()->isGuest()) { diff --git a/app/View/ViewBlocks/ShelvesShowActions.php b/app/View/ViewBlocks/ShelvesShowActions.php index 9c0ab7321..d1bb965c8 100644 --- a/app/View/ViewBlocks/ShelvesShowActions.php +++ b/app/View/ViewBlocks/ShelvesShowActions.php @@ -10,7 +10,7 @@ class ShelvesShowActions extends ViewBlock { protected string $view = 'shelves.parts.show-sidebar-section-actions'; - public function withData(array $viewData, Request $request): array + public function withData(array $viewData): array { /** @var Bookshelf $shelf */ $shelf = $viewData['shelf']; diff --git a/app/View/ViewBlocks/ShelvesShowActivity.php b/app/View/ViewBlocks/ShelvesShowActivity.php index a4ae874be..40e715bcd 100644 --- a/app/View/ViewBlocks/ShelvesShowActivity.php +++ b/app/View/ViewBlocks/ShelvesShowActivity.php @@ -16,7 +16,7 @@ class ShelvesShowActivity extends ViewBlock ) { } - public function withData(array $viewData, Request $request): array + public function withData(array $viewData): array { /** @var Bookshelf $shelf */ $shelf = $viewData['shelf']; diff --git a/app/View/ViewBlocks/ShelvesShowDetails.php b/app/View/ViewBlocks/ShelvesShowDetails.php index 2a892cf88..413b4796c 100644 --- a/app/View/ViewBlocks/ShelvesShowDetails.php +++ b/app/View/ViewBlocks/ShelvesShowDetails.php @@ -16,7 +16,7 @@ class ShelvesShowDetails extends ViewBlock ) { } - public function withData(array $viewData, Request $request): array + public function withData(array $viewData): array { /** @var Bookshelf $shelf */ $shelf = $viewData['shelf']; diff --git a/app/View/ViewBlocks/ShelvesShowTags.php b/app/View/ViewBlocks/ShelvesShowTags.php index 35dd56ede..9a61cc94d 100644 --- a/app/View/ViewBlocks/ShelvesShowTags.php +++ b/app/View/ViewBlocks/ShelvesShowTags.php @@ -9,7 +9,7 @@ class ShelvesShowTags extends ViewBlock { protected string $view = 'shelves.parts.show-sidebar-section-tags'; - public function withData(array $viewData, Request $request): array + public function withData(array $viewData): array { return [ 'book' => $viewData['book'], diff --git a/resources/views/books/index.blade.php b/resources/views/books/index.blade.php index 020cb0966..6e311a88b 100644 --- a/resources/views/books/index.blade.php +++ b/resources/views/books/index.blade.php @@ -5,9 +5,9 @@ @stop @section('left') - @include('common.sidebar-sections', ['location' => 'books-index', 'position' => 'left']) + @include('common.view-blocks', ['location' => 'books-index', 'position' => 'left']) @stop @section('right') - @include('common.sidebar-sections', ['location' => 'books-index', 'position' => 'right']) + @include('common.view-blocks', ['location' => 'books-index', 'position' => 'right']) @stop diff --git a/resources/views/books/show.blade.php b/resources/views/books/show.blade.php index 48a951132..b2fa0b393 100644 --- a/resources/views/books/show.blade.php +++ b/resources/views/books/show.blade.php @@ -67,9 +67,9 @@ @stop @section('left') - @include('common.sidebar-sections', ['location' => 'books-show', 'position' => 'left']) + @include('common.view-blocks', ['location' => 'books-show', 'position' => 'left']) @stop @section('right') - @include('common.sidebar-sections', ['location' => 'books-show', 'position' => 'right']) + @include('common.view-blocks', ['location' => 'books-show', 'position' => 'right']) @stop diff --git a/resources/views/chapters/show.blade.php b/resources/views/chapters/show.blade.php index 569d901f6..2c481ee14 100644 --- a/resources/views/chapters/show.blade.php +++ b/resources/views/chapters/show.blade.php @@ -63,9 +63,9 @@ @stop @section('left') - @include('common.sidebar-sections', ['location' => 'chapters-show', 'position' => 'left']) + @include('common.view-blocks', ['location' => 'chapters-show', 'position' => 'left']) @stop @section('right') - @include('common.sidebar-sections', ['location' => 'chapters-show', 'position' => 'right']) + @include('common.view-blocks', ['location' => 'chapters-show', 'position' => 'right']) @stop diff --git a/resources/views/common/sidebar-sections.blade.php b/resources/views/common/view-blocks.blade.php similarity index 60% rename from resources/views/common/sidebar-sections.blade.php rename to resources/views/common/view-blocks.blade.php index f18f9e4c2..fb5e83e19 100644 --- a/resources/views/common/sidebar-sections.blade.php +++ b/resources/views/common/view-blocks.blade.php @@ -4,8 +4,7 @@ $position - string --}} @php $sectionVars = get_defined_vars(); - $sectionRequest = request(); @endphp @foreach($viewBlocks->getForLocation($location, $position) as $section) - @include($section->getView(), $section->withData($sectionVars, $sectionRequest)) + @include($section->getView($sectionVars), $section->withData($sectionVars)) @endforeach \ No newline at end of file diff --git a/resources/views/home/books.blade.php b/resources/views/home/books.blade.php index afd3f30fc..3c4225b56 100644 --- a/resources/views/home/books.blade.php +++ b/resources/views/home/books.blade.php @@ -5,26 +5,9 @@ @stop @section('left') - @include('home.parts.sidebar') + @include('common.view-blocks', ['location' => 'home-non-default', 'position' => 'left']) @stop @section('right') -
    -
    {{ trans('common.actions') }}
    -
    - @if(userCan(\BookStack\Permissions\Permission::BookCreateAll)) - - @icon('add') - {{ trans('entities.books_create') }} - - @endif - @include('entities.view-toggle', ['view' => $view, 'type' => 'books']) - - @icon('tag') - {{ trans('entities.tags_view_tags') }} - - @include('home.parts.expand-toggle', ['classes' => 'text-link', 'target' => '.entity-list.compact .entity-item-snippet', 'key' => 'home-details']) - @include('common.dark-mode-toggle', ['classes' => 'icon-list-item text-link']) -
    -
    + @include('common.view-blocks', ['location' => 'home-non-default', 'position' => 'right']) @stop diff --git a/resources/views/home/default.blade.php b/resources/views/home/default.blade.php index 7d229d6ac..508631c8c 100644 --- a/resources/views/home/default.blade.php +++ b/resources/views/home/default.blade.php @@ -16,17 +16,15 @@
    - @include('home.parts.default-card-recent-drafts', ['draftPages' => $draftPages]) - @include('home.parts.default-card-recently-viewed-or-recent-books', ['recents' => $recents]) + @include('common.view-blocks', ['location' => 'home-default', 'position' => 'left'])
    - @include('home.parts.default-card-top-favourites', ['favourites' => $favourites]) - @include('home.parts.default-card-recently-updates-pages', ['recentlyUpdatedPages' => $recentlyUpdatedPages]) + @include('common.view-blocks', ['location' => 'home-default', 'position' => 'center'])
    - @include('home.parts.default-card-recent-activity', ['activity' => $activity]) + @include('common.view-blocks', ['location' => 'home-default', 'position' => 'right'])
    diff --git a/resources/views/home/parts/configured-section-actions.blade.php b/resources/views/home/parts/configured-section-actions.blade.php new file mode 100644 index 000000000..6386670d5 --- /dev/null +++ b/resources/views/home/parts/configured-section-actions.blade.php @@ -0,0 +1,36 @@ +{{-- +$homeView - string - The type of home view being used +$view - string - Configured view type from parent (for books/shelves list/grid view) +--}} +
    +
    {{ trans('common.actions') }}
    +
    + @if($homeView === 'books') + @if(userCan(\BookStack\Permissions\Permission::BookCreateAll)) + + @icon('add') + {{ trans('entities.books_create') }} + + @endif + @include('entities.view-toggle', ['view' => $view, 'type' => 'books']) + + @icon('tag') + {{ trans('entities.tags_view_tags') }} + + @elseif ($homeView === 'bookshelves') + @if(userCan(\BookStack\Permissions\Permission::BookshelfCreateAll)) + + @icon('add') + {{ trans('entities.shelves_new_action') }} + + @endif + @include('entities.view-toggle', ['view' => $view, 'type' => 'bookshelves']) + + @icon('tag') + {{ trans('entities.tags_view_tags') }} + + @endif + @include('home.parts.expand-toggle', ['classes' => 'text-link', 'target' => '.entity-list.compact .entity-item-snippet', 'key' => 'home-details']) + @include('common.dark-mode-toggle', ['classes' => 'icon-list-item text-link']) +
    +
    \ No newline at end of file diff --git a/resources/views/home/parts/sidebar-section-recent-activity.blade.php b/resources/views/home/parts/configured-section-recent-activity.blade.php similarity index 100% rename from resources/views/home/parts/sidebar-section-recent-activity.blade.php rename to resources/views/home/parts/configured-section-recent-activity.blade.php diff --git a/resources/views/home/parts/sidebar-section-recent-drafts.blade.php b/resources/views/home/parts/configured-section-recent-drafts.blade.php similarity index 100% rename from resources/views/home/parts/sidebar-section-recent-drafts.blade.php rename to resources/views/home/parts/configured-section-recent-drafts.blade.php diff --git a/resources/views/home/parts/sidebar-section-recently-updated-pages.blade.php b/resources/views/home/parts/configured-section-recently-updated-pages.blade.php similarity index 100% rename from resources/views/home/parts/sidebar-section-recently-updated-pages.blade.php rename to resources/views/home/parts/configured-section-recently-updated-pages.blade.php diff --git a/resources/views/home/parts/sidebar-section-recently-viewed-or-recent-books.blade.php b/resources/views/home/parts/configured-section-recently-viewed-or-recent-books.blade.php similarity index 100% rename from resources/views/home/parts/sidebar-section-recently-viewed-or-recent-books.blade.php rename to resources/views/home/parts/configured-section-recently-viewed-or-recent-books.blade.php diff --git a/resources/views/home/parts/sidebar-section-top-favourites.blade.php b/resources/views/home/parts/configured-section-top-favourites.blade.php similarity index 100% rename from resources/views/home/parts/sidebar-section-top-favourites.blade.php rename to resources/views/home/parts/configured-section-top-favourites.blade.php diff --git a/resources/views/home/parts/default-card-recent-activity.blade.php b/resources/views/home/parts/default-card-recent-activity.blade.php index 035cd9987..c848ff1bc 100644 --- a/resources/views/home/parts/default-card-recent-activity.blade.php +++ b/resources/views/home/parts/default-card-recent-activity.blade.php @@ -1,6 +1,9 @@ +{{-- +$activity - array +--}}

    {{ trans('entities.recent_activity') }}

    @include('common.activity-list', ['activity' => $activity])
    -
    j \ No newline at end of file +
    • \ No newline at end of file diff --git a/resources/views/home/parts/default-card-recently-updates-pages.blade.php b/resources/views/home/parts/default-card-recently-updated-pages.blade.php similarity index 100% rename from resources/views/home/parts/default-card-recently-updates-pages.blade.php rename to resources/views/home/parts/default-card-recently-updated-pages.blade.php diff --git a/resources/views/home/parts/sidebar.blade.php b/resources/views/home/parts/sidebar.blade.php deleted file mode 100644 index 07c199f9e..000000000 --- a/resources/views/home/parts/sidebar.blade.php +++ /dev/null @@ -1,5 +0,0 @@ -@include('home.parts.sidebar-section-recent-drafts', ['draftPages' => $draftPages]) -@include('home.parts.sidebar-section-top-favourites', ['favourites' => $favourites]) -@include('home.parts.sidebar-section-recently-viewed-or-recent-books', ['recents' => $recents]) -@include('home.parts.sidebar-section-recently-updated-pages', ['recentlyUpdatedPages' => $recentlyUpdatedPages]) -@include('home.parts.sidebar-section-recent-activity', ['activity' => $activity]) \ No newline at end of file diff --git a/resources/views/home/shelves.blade.php b/resources/views/home/shelves.blade.php index 7a4d002a0..647951543 100644 --- a/resources/views/home/shelves.blade.php +++ b/resources/views/home/shelves.blade.php @@ -5,26 +5,9 @@ @stop @section('left') - @include('home.parts.sidebar') + @include('common.view-blocks', ['location' => 'home-non-default', 'position' => 'left']) @stop @section('right') -
    -
    {{ trans('common.actions') }}
    -
    - @if(userCan(\BookStack\Permissions\Permission::BookshelfCreateAll)) - - @icon('add') - {{ trans('entities.shelves_new_action') }} - - @endif - @include('entities.view-toggle', ['view' => $view, 'type' => 'bookshelves']) - - @icon('tag') - {{ trans('entities.tags_view_tags') }} - - @include('home.parts.expand-toggle', ['classes' => 'text-link', 'target' => '.entity-list.compact .entity-item-snippet', 'key' => 'home-details']) - @include('common.dark-mode-toggle', ['classes' => 'icon-list-item text-link']) -
    -
    + @include('common.view-blocks', ['location' => 'home-non-default', 'position' => 'right']) @stop diff --git a/resources/views/home/specific-page.blade.php b/resources/views/home/specific-page.blade.php index 10ffe9629..329c4a6c2 100644 --- a/resources/views/home/specific-page.blade.php +++ b/resources/views/home/specific-page.blade.php @@ -13,15 +13,9 @@ @stop @section('left') - @include('home.parts.sidebar') + @include('common.view-blocks', ['location' => 'home-non-default', 'position' => 'left']) @stop @section('right') -
    -
    {{ trans('common.actions') }}
    -
    - @include('home.parts.expand-toggle', ['classes' => 'text-link', 'target' => '.entity-list.compact .entity-item-snippet', 'key' => 'home-details']) - @include('common.dark-mode-toggle', ['classes' => 'icon-list-item text-link']) -
    -
    + @include('common.view-blocks', ['location' => 'home-non-default', 'position' => 'right']) @stop \ No newline at end of file diff --git a/resources/views/pages/show.blade.php b/resources/views/pages/show.blade.php index 08d01973d..eef559320 100644 --- a/resources/views/pages/show.blade.php +++ b/resources/views/pages/show.blade.php @@ -36,9 +36,9 @@ @stop @section('left') - @include('common.sidebar-sections', ['location' => 'pages-show', 'position' => 'left']) + @include('common.view-blocks', ['location' => 'pages-show', 'position' => 'left']) @stop @section('right') - @include('common.sidebar-sections', ['location' => 'pages-show', 'position' => 'right']) + @include('common.view-blocks', ['location' => 'pages-show', 'position' => 'right']) @stop diff --git a/resources/views/shelves/index.blade.php b/resources/views/shelves/index.blade.php index 2fc78fa50..87c639298 100644 --- a/resources/views/shelves/index.blade.php +++ b/resources/views/shelves/index.blade.php @@ -5,9 +5,9 @@ @stop @section('left') - @include('common.sidebar-sections', ['location' => 'shelves-index', 'position' => 'left']) + @include('common.view-blocks', ['location' => 'shelves-index', 'position' => 'left']) @stop @section('right') - @include('common.sidebar-sections', ['location' => 'shelves-index', 'position' => 'right']) + @include('common.view-blocks', ['location' => 'shelves-index', 'position' => 'right']) @stop diff --git a/resources/views/shelves/show.blade.php b/resources/views/shelves/show.blade.php index 320e1066c..b942f35ff 100644 --- a/resources/views/shelves/show.blade.php +++ b/resources/views/shelves/show.blade.php @@ -69,9 +69,9 @@ @stop @section('left') - @include('common.sidebar-sections', ['location' => 'shelves-show', 'position' => 'left']) + @include('common.view-blocks', ['location' => 'shelves-show', 'position' => 'left']) @stop @section('right') - @include('common.sidebar-sections', ['location' => 'shelves-show', 'position' => 'right']) + @include('common.view-blocks', ['location' => 'shelves-show', 'position' => 'right']) @stop From 45ff36714ff78afa27078b4948edeae1e854b105 Mon Sep 17 00:00:00 2001 From: Dan Brown Date: Sun, 9 Aug 2026 18:35:10 +0100 Subject: [PATCH 07/21] View Blocks: Extracted default set, added labels to each --- app/View/ViewBlock.php | 9 ++ app/View/ViewBlockDefaults.php | 110 +++++++++++++++ app/View/ViewBlockInterface.php | 5 + app/View/ViewBlockManager.php | 133 +----------------- app/View/ViewBlocks/BooksIndexActions.php | 1 + app/View/ViewBlocks/BooksIndexNew.php | 1 + app/View/ViewBlocks/BooksIndexPopular.php | 1 + app/View/ViewBlocks/BooksIndexRecents.php | 1 + app/View/ViewBlocks/BooksShowActions.php | 1 + app/View/ViewBlocks/BooksShowActivity.php | 1 + app/View/ViewBlocks/BooksShowDetails.php | 1 + app/View/ViewBlocks/BooksShowSearchForm.php | 1 + app/View/ViewBlocks/BooksShowShelves.php | 1 + app/View/ViewBlocks/BooksShowTags.php | 1 + app/View/ViewBlocks/ChaptersShowActions.php | 1 + app/View/ViewBlocks/ChaptersShowBookTree.php | 1 + app/View/ViewBlocks/ChaptersShowDetails.php | 1 + .../ViewBlocks/ChaptersShowSearchForm.php | 1 + app/View/ViewBlocks/ChaptersShowTags.php | 1 + app/View/ViewBlocks/HomeActions.php | 2 +- app/View/ViewBlocks/HomeRecentActivity.php | 5 + app/View/ViewBlocks/HomeRecentDrafts.php | 5 + .../ViewBlocks/HomeRecentlyUpdatedPages.php | 5 + .../HomeRecentlyViewedOrRecentBooks.php | 6 + app/View/ViewBlocks/HomeTopFavourites.php | 5 + app/View/ViewBlocks/PagesShowActions.php | 1 + app/View/ViewBlocks/PagesShowAttachments.php | 1 + app/View/ViewBlocks/PagesShowBookTree.php | 1 + app/View/ViewBlocks/PagesShowDetails.php | 1 + app/View/ViewBlocks/PagesShowPageNav.php | 1 + app/View/ViewBlocks/PagesShowTags.php | 1 + app/View/ViewBlocks/ShelvesIndexActions.php | 1 + app/View/ViewBlocks/ShelvesIndexNew.php | 1 + app/View/ViewBlocks/ShelvesIndexPopular.php | 1 + app/View/ViewBlocks/ShelvesIndexRecents.php | 1 + app/View/ViewBlocks/ShelvesShowActions.php | 1 + app/View/ViewBlocks/ShelvesShowActivity.php | 1 + app/View/ViewBlocks/ShelvesShowDetails.php | 1 + app/View/ViewBlocks/ShelvesShowTags.php | 1 + 39 files changed, 185 insertions(+), 129 deletions(-) create mode 100644 app/View/ViewBlockDefaults.php diff --git a/app/View/ViewBlock.php b/app/View/ViewBlock.php index 636746044..921ecd342 100644 --- a/app/View/ViewBlock.php +++ b/app/View/ViewBlock.php @@ -5,6 +5,7 @@ namespace BookStack\View; abstract class ViewBlock implements ViewBlockInterface { protected string $view; + protected string $labelTranslationKey; /** * @inheritDoc @@ -13,4 +14,12 @@ abstract class ViewBlock implements ViewBlockInterface { return $this->view; } + + /** + * @inheritDoc + */ + public function getLabel(): string + { + return trans($this->labelTranslationKey); + } } diff --git a/app/View/ViewBlockDefaults.php b/app/View/ViewBlockDefaults.php new file mode 100644 index 000000000..33a08adc1 --- /dev/null +++ b/app/View/ViewBlockDefaults.php @@ -0,0 +1,110 @@ +[]>> + */ + protected static array $defaults = [ + 'home-default' => [ + 'left' => [ + ViewBlocks\HomeRecentDrafts::class, + ViewBlocks\HomeRecentlyViewedOrRecentBooks::class, + ], + 'center' => [ + ViewBlocks\HomeTopFavourites::class, + ViewBlocks\HomeRecentlyUpdatedPages::class, + ], + 'right' => [ + ViewBlocks\HomeRecentActivity::class, + ], + ], + 'home-non-default' => [ + 'left' => [ + ViewBlocks\HomeRecentDrafts::class, + ViewBlocks\HomeTopFavourites::class, + ViewBlocks\HomeRecentlyViewedOrRecentBooks::class, + ViewBlocks\HomeRecentlyUpdatedPages::class, + ViewBlocks\HomeRecentActivity::class, + ], + 'right' => [ + ViewBlocks\HomeActions::class, + ], + ], + 'shelves-index' => [ + 'left' => [ + ViewBlocks\ShelvesIndexRecents::class, + ViewBlocks\ShelvesIndexPopular::class, + ViewBlocks\ShelvesIndexNew::class, + ], + 'right' => [ + ViewBlocks\ShelvesIndexActions::class, + ], + ], + 'shelves-show' => [ + 'left' => [ + ViewBlocks\ShelvesShowTags::class, + ViewBlocks\ShelvesShowDetails::class, + ViewBlocks\ShelvesShowActivity::class, + ], + 'right' => [ + ViewBlocks\ShelvesShowActions::class, + ], + ], + 'books-index' => [ + 'left' => [ + ViewBlocks\BooksIndexRecents::class, + ViewBlocks\BooksIndexPopular::class, + ViewBlocks\BooksIndexNew::class, + ], + 'right' => [ + ViewBlocks\BooksIndexActions::class, + ], + ], + 'books-show' => [ + 'left' => [ + ViewBlocks\BooksShowSearchForm::class, + ViewBlocks\BooksShowTags::class, + ViewBlocks\BooksShowShelves::class, + ViewBlocks\BooksShowActivity::class, + ], + 'right' => [ + ViewBlocks\BooksShowDetails::class, + ViewBlocks\BooksShowActions::class, + ], + ], + 'chapters-show' => [ + 'left' => [ + ViewBlocks\ChaptersShowSearchForm::class, + ViewBlocks\ChaptersShowTags::class, + ViewBlocks\ChaptersShowBookTree::class, + ], + 'right' => [ + ViewBlocks\ChaptersShowDetails::class, + ViewBlocks\ChaptersShowActions::class, + ], + ], + 'pages-show' => [ + 'left' => [ + ViewBlocks\PagesShowTags::class, + ViewBlocks\PagesShowAttachments::class, + ViewBlocks\PagesShowPageNav::class, + ViewBlocks\PagesShowBookTree::class, + ], + 'right' => [ + ViewBlocks\PagesShowDetails::class, + ViewBlocks\PagesShowActions::class, + ], + ], + ]; + + /** + * Get the default view blocks for the given location. + */ + public static function getForLocation(string $location): array + { + return self::$defaults[$location] ?? []; + } +} diff --git a/app/View/ViewBlockInterface.php b/app/View/ViewBlockInterface.php index 756cf4298..d72456870 100644 --- a/app/View/ViewBlockInterface.php +++ b/app/View/ViewBlockInterface.php @@ -4,6 +4,11 @@ namespace BookStack\View; interface ViewBlockInterface { + /** + * Get the label for this block. + */ + public function getLabel(): string; + /** * Get the view to render for this block. * Is provided with the existing parent view data. diff --git a/app/View/ViewBlockManager.php b/app/View/ViewBlockManager.php index 7287c0293..00535b2c1 100644 --- a/app/View/ViewBlockManager.php +++ b/app/View/ViewBlockManager.php @@ -2,41 +2,6 @@ namespace BookStack\View; -use BookStack\View\ViewBlocks\BooksIndexActions; -use BookStack\View\ViewBlocks\BooksIndexNew; -use BookStack\View\ViewBlocks\BooksIndexPopular; -use BookStack\View\ViewBlocks\BooksIndexRecents; -use BookStack\View\ViewBlocks\BooksShowActions; -use BookStack\View\ViewBlocks\BooksShowActivity; -use BookStack\View\ViewBlocks\BooksShowDetails; -use BookStack\View\ViewBlocks\BooksShowSearchForm; -use BookStack\View\ViewBlocks\BooksShowShelves; -use BookStack\View\ViewBlocks\BooksShowTags; -use BookStack\View\ViewBlocks\ChaptersShowActions; -use BookStack\View\ViewBlocks\ChaptersShowBookTree; -use BookStack\View\ViewBlocks\ChaptersShowDetails; -use BookStack\View\ViewBlocks\ChaptersShowSearchForm; -use BookStack\View\ViewBlocks\ChaptersShowTags; -use BookStack\View\ViewBlocks\HomeActions; -use BookStack\View\ViewBlocks\HomeRecentActivity; -use BookStack\View\ViewBlocks\HomeRecentDrafts; -use BookStack\View\ViewBlocks\HomeRecentlyUpdatedPages; -use BookStack\View\ViewBlocks\HomeRecentlyViewedOrRecentBooks; -use BookStack\View\ViewBlocks\HomeTopFavourites; -use BookStack\View\ViewBlocks\PagesShowActions; -use BookStack\View\ViewBlocks\PagesShowAttachments; -use BookStack\View\ViewBlocks\PagesShowBookTree; -use BookStack\View\ViewBlocks\PagesShowDetails; -use BookStack\View\ViewBlocks\PagesShowPageNav; -use BookStack\View\ViewBlocks\PagesShowTags; -use BookStack\View\ViewBlocks\ShelvesIndexActions; -use BookStack\View\ViewBlocks\ShelvesIndexNew; -use BookStack\View\ViewBlocks\ShelvesIndexPopular; -use BookStack\View\ViewBlocks\ShelvesIndexRecents; -use BookStack\View\ViewBlocks\ShelvesShowActions; -use BookStack\View\ViewBlocks\ShelvesShowActivity; -use BookStack\View\ViewBlocks\ShelvesShowDetails; -use BookStack\View\ViewBlocks\ShelvesShowTags; use Illuminate\Contracts\Container\BindingResolutionException; class ViewBlockManager @@ -44,98 +9,7 @@ class ViewBlockManager /** * @var array[]>> */ - protected array $blocksByLocationAndPosition = [ - 'home-default' => [ - 'left' => [ - HomeRecentDrafts::class, - HomeRecentlyViewedOrRecentBooks::class, - ], - 'center' => [ - HomeTopFavourites::class, - HomeRecentlyUpdatedPages::class, - ], - 'right' => [ - HomeRecentActivity::class, - ], - ], - 'home-non-default' => [ - 'left' => [ - HomeRecentDrafts::class, - HomeTopFavourites::class, - HomeRecentlyViewedOrRecentBooks::class, - HomeRecentlyUpdatedPages::class, - HomeRecentActivity::class, - ], - 'right' => [ - HomeActions::class, - ], - ], - 'shelves-index' => [ - 'left' => [ - ShelvesIndexRecents::class, - ShelvesIndexPopular::class, - ShelvesIndexNew::class, - ], - 'right' => [ - ShelvesIndexActions::class, - ], - ], - 'shelves-show' => [ - 'left' => [ - ShelvesShowTags::class, - ShelvesShowDetails::class, - ShelvesShowActivity::class, - ], - 'right' => [ - ShelvesShowActions::class, - ], - ], - 'books-index' => [ - 'left' => [ - BooksIndexRecents::class, - BooksIndexPopular::class, - BooksIndexNew::class, - ], - 'right' => [ - BooksIndexActions::class, - ], - ], - 'books-show' => [ - 'left' => [ - BooksShowSearchForm::class, - BooksShowTags::class, - BooksShowShelves::class, - BooksShowActivity::class, - ], - 'right' => [ - BooksShowDetails::class, - BooksShowActions::class, - ], - ], - 'chapters-show' => [ - 'left' => [ - ChaptersShowSearchForm::class, - ChaptersShowTags::class, - ChaptersShowBookTree::class, - ], - 'right' => [ - ChaptersShowDetails::class, - ChaptersShowActions::class, - ], - ], - 'pages-show' => [ - 'left' => [ - PagesShowTags::class, - PagesShowAttachments::class, - PagesShowPageNav::class, - PagesShowBookTree::class, - ], - 'right' => [ - PagesShowDetails::class, - PagesShowActions::class, - ], - ], - ]; + protected array $blocksByLocationAndPosition = []; /** * Register a block type to be displayed at the given location and position. @@ -162,7 +36,10 @@ class ViewBlockManager */ public function getForLocation(string $location, string $position): array { - $sections = array_unique($this->blocksByLocationAndPosition[$location][$position] ?? []); + $defaults = ViewBlockDefaults::getForLocation($location)[$position] ?? []; + $registered = $this->blocksByLocationAndPosition[$location][$position] ?? []; + $sections = array_unique(array_merge($defaults, $registered)); + return array_map(function (string $className) { return app()->make($className); }, $sections); diff --git a/app/View/ViewBlocks/BooksIndexActions.php b/app/View/ViewBlocks/BooksIndexActions.php index 8e6721ddf..2802809f5 100644 --- a/app/View/ViewBlocks/BooksIndexActions.php +++ b/app/View/ViewBlocks/BooksIndexActions.php @@ -8,6 +8,7 @@ use Illuminate\Http\Request; class BooksIndexActions extends ViewBlock { protected string $view = 'books.parts.index-sidebar-section-actions'; + protected string $labelTranslationKey = 'common.actions'; public function withData(array $viewData): array { diff --git a/app/View/ViewBlocks/BooksIndexNew.php b/app/View/ViewBlocks/BooksIndexNew.php index 76d2750ba..ef88976e3 100644 --- a/app/View/ViewBlocks/BooksIndexNew.php +++ b/app/View/ViewBlocks/BooksIndexNew.php @@ -9,6 +9,7 @@ use Illuminate\Http\Request; class BooksIndexNew extends ViewBlock { protected string $view = 'books.parts.index-sidebar-section-new'; + protected string $labelTranslationKey = 'entities.books_new'; public function __construct( protected BookQueries $queries, diff --git a/app/View/ViewBlocks/BooksIndexPopular.php b/app/View/ViewBlocks/BooksIndexPopular.php index ee3138f79..6428a6098 100644 --- a/app/View/ViewBlocks/BooksIndexPopular.php +++ b/app/View/ViewBlocks/BooksIndexPopular.php @@ -9,6 +9,7 @@ use Illuminate\Http\Request; class BooksIndexPopular extends ViewBlock { protected string $view = 'books.parts.index-sidebar-section-popular'; + protected string $labelTranslationKey = 'entities.books_popular'; public function __construct( protected BookQueries $queries, diff --git a/app/View/ViewBlocks/BooksIndexRecents.php b/app/View/ViewBlocks/BooksIndexRecents.php index 1c7b865d8..a6bb90a84 100644 --- a/app/View/ViewBlocks/BooksIndexRecents.php +++ b/app/View/ViewBlocks/BooksIndexRecents.php @@ -9,6 +9,7 @@ use Illuminate\Http\Request; class BooksIndexRecents extends ViewBlock { protected string $view = 'books.parts.index-sidebar-section-recents'; + protected string $labelTranslationKey = 'entities.recently_viewed'; public function __construct( protected BookQueries $queries, diff --git a/app/View/ViewBlocks/BooksShowActions.php b/app/View/ViewBlocks/BooksShowActions.php index 155c0c7ba..4f868e4a3 100644 --- a/app/View/ViewBlocks/BooksShowActions.php +++ b/app/View/ViewBlocks/BooksShowActions.php @@ -10,6 +10,7 @@ use Illuminate\Http\Request; class BooksShowActions extends ViewBlock { protected string $view = 'books.parts.show-sidebar-section-actions'; + protected string $labelTranslationKey = 'common.actions'; public function withData(array $viewData): array { diff --git a/app/View/ViewBlocks/BooksShowActivity.php b/app/View/ViewBlocks/BooksShowActivity.php index ecf626df5..e5c0acdd2 100644 --- a/app/View/ViewBlocks/BooksShowActivity.php +++ b/app/View/ViewBlocks/BooksShowActivity.php @@ -10,6 +10,7 @@ use Illuminate\Http\Request; class BooksShowActivity extends ViewBlock { protected string $view = 'books.parts.show-sidebar-section-activity'; + protected string $labelTranslationKey = 'entities.recent_activity'; public function __construct( protected ActivityQueries $activityQueries, diff --git a/app/View/ViewBlocks/BooksShowDetails.php b/app/View/ViewBlocks/BooksShowDetails.php index 9a0ef8b47..ccd7a7704 100644 --- a/app/View/ViewBlocks/BooksShowDetails.php +++ b/app/View/ViewBlocks/BooksShowDetails.php @@ -11,6 +11,7 @@ use Illuminate\Http\Request; class BooksShowDetails extends ViewBlock { protected string $view = 'books.parts.show-sidebar-section-details'; + protected string $labelTranslationKey = 'common.details'; public function __construct( protected ReferenceFetcher $referenceFetcher, diff --git a/app/View/ViewBlocks/BooksShowSearchForm.php b/app/View/ViewBlocks/BooksShowSearchForm.php index 061875d3d..492579505 100644 --- a/app/View/ViewBlocks/BooksShowSearchForm.php +++ b/app/View/ViewBlocks/BooksShowSearchForm.php @@ -8,6 +8,7 @@ use Illuminate\Http\Request; class BooksShowSearchForm extends ViewBlock { protected string $view = 'entities.search-form'; + protected string $labelTranslationKey = 'common.search'; public function withData(array $viewData): array { diff --git a/app/View/ViewBlocks/BooksShowShelves.php b/app/View/ViewBlocks/BooksShowShelves.php index 42d832589..310bdffef 100644 --- a/app/View/ViewBlocks/BooksShowShelves.php +++ b/app/View/ViewBlocks/BooksShowShelves.php @@ -9,6 +9,7 @@ use Illuminate\Http\Request; class BooksShowShelves extends ViewBlock { protected string $view = 'books.parts.show-sidebar-section-shelves'; + protected string $labelTranslationKey = 'entities.shelves'; public function withData(array $viewData): array { diff --git a/app/View/ViewBlocks/BooksShowTags.php b/app/View/ViewBlocks/BooksShowTags.php index 5c5aa8fd9..ac9f4c1ef 100644 --- a/app/View/ViewBlocks/BooksShowTags.php +++ b/app/View/ViewBlocks/BooksShowTags.php @@ -8,6 +8,7 @@ use Illuminate\Http\Request; class BooksShowTags extends ViewBlock { protected string $view = 'books.parts.show-sidebar-section-tags'; + protected string $labelTranslationKey = 'entities.tags'; public function withData(array $viewData): array { diff --git a/app/View/ViewBlocks/ChaptersShowActions.php b/app/View/ViewBlocks/ChaptersShowActions.php index 9b6137a20..6fe1b6955 100644 --- a/app/View/ViewBlocks/ChaptersShowActions.php +++ b/app/View/ViewBlocks/ChaptersShowActions.php @@ -10,6 +10,7 @@ use Illuminate\Http\Request; class ChaptersShowActions extends ViewBlock { protected string $view = 'chapters.parts.show-sidebar-section-actions'; + protected string $labelTranslationKey = 'common.actions'; public function withData(array $viewData): array { diff --git a/app/View/ViewBlocks/ChaptersShowBookTree.php b/app/View/ViewBlocks/ChaptersShowBookTree.php index 827e8d917..3f6d93418 100644 --- a/app/View/ViewBlocks/ChaptersShowBookTree.php +++ b/app/View/ViewBlocks/ChaptersShowBookTree.php @@ -9,6 +9,7 @@ use Illuminate\Http\Request; class ChaptersShowBookTree extends ViewBlock { protected string $view = 'entities.book-tree'; + protected string $labelTranslationKey = 'entities.books_navigation'; public function withData(array $viewData): array { diff --git a/app/View/ViewBlocks/ChaptersShowDetails.php b/app/View/ViewBlocks/ChaptersShowDetails.php index fd235fbe1..7a90c9058 100644 --- a/app/View/ViewBlocks/ChaptersShowDetails.php +++ b/app/View/ViewBlocks/ChaptersShowDetails.php @@ -12,6 +12,7 @@ use Illuminate\Http\Request; class ChaptersShowDetails extends ViewBlock { protected string $view = 'chapters.parts.show-sidebar-section-details'; + protected string $labelTranslationKey = 'common.details'; public function __construct( protected ReferenceFetcher $referenceFetcher, diff --git a/app/View/ViewBlocks/ChaptersShowSearchForm.php b/app/View/ViewBlocks/ChaptersShowSearchForm.php index 92664e5f5..6c2b8aa5a 100644 --- a/app/View/ViewBlocks/ChaptersShowSearchForm.php +++ b/app/View/ViewBlocks/ChaptersShowSearchForm.php @@ -8,6 +8,7 @@ use Illuminate\Http\Request; class ChaptersShowSearchForm extends ViewBlock { protected string $view = 'entities.search-form'; + protected string $labelTranslationKey = 'common.search'; public function withData(array $viewData): array { diff --git a/app/View/ViewBlocks/ChaptersShowTags.php b/app/View/ViewBlocks/ChaptersShowTags.php index 198604eee..fa7fa200d 100644 --- a/app/View/ViewBlocks/ChaptersShowTags.php +++ b/app/View/ViewBlocks/ChaptersShowTags.php @@ -9,6 +9,7 @@ use Illuminate\Http\Request; class ChaptersShowTags extends ViewBlock { protected string $view = 'chapters.parts.show-sidebar-section-tags'; + protected string $labelTranslationKey = 'entities.tags'; public function withData(array $viewData): array { diff --git a/app/View/ViewBlocks/HomeActions.php b/app/View/ViewBlocks/HomeActions.php index f4e4295b8..2e64c97fb 100644 --- a/app/View/ViewBlocks/HomeActions.php +++ b/app/View/ViewBlocks/HomeActions.php @@ -3,11 +3,11 @@ namespace BookStack\View\ViewBlocks; use BookStack\View\ViewBlock; -use Illuminate\Http\Request; class HomeActions extends ViewBlock { protected string $view = 'home.parts.configured-section-actions'; + protected string $labelTranslationKey = 'common.actions'; public function withData(array $viewData): array { diff --git a/app/View/ViewBlocks/HomeRecentActivity.php b/app/View/ViewBlocks/HomeRecentActivity.php index 1f915061c..7b5a74ecb 100644 --- a/app/View/ViewBlocks/HomeRecentActivity.php +++ b/app/View/ViewBlocks/HomeRecentActivity.php @@ -13,6 +13,11 @@ class HomeRecentActivity implements ViewBlockInterface ) { } + public function getLabel(): string + { + return trans('entities.recent_activity'); + } + public function getView(array $viewData): string { if ($viewData['homeView'] === 'default') { diff --git a/app/View/ViewBlocks/HomeRecentDrafts.php b/app/View/ViewBlocks/HomeRecentDrafts.php index 71586d83a..e65078cb4 100644 --- a/app/View/ViewBlocks/HomeRecentDrafts.php +++ b/app/View/ViewBlocks/HomeRecentDrafts.php @@ -12,6 +12,11 @@ class HomeRecentDrafts implements ViewBlockInterface ) { } + public function getLabel(): string + { + return trans('entities.my_recent_drafts'); + } + public function getView(array $viewData): string { if ($viewData['homeView'] === 'default') { diff --git a/app/View/ViewBlocks/HomeRecentlyUpdatedPages.php b/app/View/ViewBlocks/HomeRecentlyUpdatedPages.php index 3f669ee6e..b56a91acd 100644 --- a/app/View/ViewBlocks/HomeRecentlyUpdatedPages.php +++ b/app/View/ViewBlocks/HomeRecentlyUpdatedPages.php @@ -13,6 +13,11 @@ class HomeRecentlyUpdatedPages implements ViewBlockInterface ) { } + public function getLabel(): string + { + return trans('entities.recently_updated_pages'); + } + public function getView(array $viewData): string { if ($viewData['homeView'] === 'default') { diff --git a/app/View/ViewBlocks/HomeRecentlyViewedOrRecentBooks.php b/app/View/ViewBlocks/HomeRecentlyViewedOrRecentBooks.php index 7db3d7935..99666e2c8 100644 --- a/app/View/ViewBlocks/HomeRecentlyViewedOrRecentBooks.php +++ b/app/View/ViewBlocks/HomeRecentlyViewedOrRecentBooks.php @@ -16,6 +16,12 @@ class HomeRecentlyViewedOrRecentBooks implements ViewBlockInterface ) { } + public function getLabel(): string + { + $key = user()->isGuest() ? 'books_recent' : 'my_recently_viewed'; + return trans("entities.{$key}"); + } + public function getView(array $viewData): string { if ($viewData['homeView'] === 'default') { diff --git a/app/View/ViewBlocks/HomeTopFavourites.php b/app/View/ViewBlocks/HomeTopFavourites.php index 9ce84aa3d..be753a719 100644 --- a/app/View/ViewBlocks/HomeTopFavourites.php +++ b/app/View/ViewBlocks/HomeTopFavourites.php @@ -13,6 +13,11 @@ class HomeTopFavourites implements ViewBlockInterface ) { } + public function getLabel(): string + { + return trans('entities.my_most_viewed_favourites'); + } + public function getView(array $viewData): string { if ($viewData['homeView'] === 'default') { diff --git a/app/View/ViewBlocks/PagesShowActions.php b/app/View/ViewBlocks/PagesShowActions.php index 5486b59a4..8abfc8a6e 100644 --- a/app/View/ViewBlocks/PagesShowActions.php +++ b/app/View/ViewBlocks/PagesShowActions.php @@ -10,6 +10,7 @@ use Illuminate\Http\Request; class PagesShowActions extends ViewBlock { protected string $view = 'pages.parts.show-sidebar-section-actions'; + protected string $labelTranslationKey = 'common.actions'; public function withData(array $viewData): array { diff --git a/app/View/ViewBlocks/PagesShowAttachments.php b/app/View/ViewBlocks/PagesShowAttachments.php index 5d0cd23ba..baf8713f1 100644 --- a/app/View/ViewBlocks/PagesShowAttachments.php +++ b/app/View/ViewBlocks/PagesShowAttachments.php @@ -9,6 +9,7 @@ use Illuminate\Http\Request; class PagesShowAttachments extends ViewBlock { protected string $view = 'pages.parts.show-sidebar-section-attachments'; + protected string $labelTranslationKey = 'entities.pages_attachments'; public function withData(array $viewData): array { diff --git a/app/View/ViewBlocks/PagesShowBookTree.php b/app/View/ViewBlocks/PagesShowBookTree.php index b0947a208..05c4c9a1f 100644 --- a/app/View/ViewBlocks/PagesShowBookTree.php +++ b/app/View/ViewBlocks/PagesShowBookTree.php @@ -9,6 +9,7 @@ use Illuminate\Http\Request; class PagesShowBookTree extends ViewBlock { protected string $view = 'entities.book-tree'; + protected string $labelTranslationKey = 'entities.books_navigation'; public function withData(array $viewData): array { diff --git a/app/View/ViewBlocks/PagesShowDetails.php b/app/View/ViewBlocks/PagesShowDetails.php index 2dfd61417..eb50e859c 100644 --- a/app/View/ViewBlocks/PagesShowDetails.php +++ b/app/View/ViewBlocks/PagesShowDetails.php @@ -12,6 +12,7 @@ use Illuminate\Http\Request; class PagesShowDetails extends ViewBlock { protected string $view = 'pages.parts.show-sidebar-section-details'; + protected string $labelTranslationKey = 'common.details'; public function __construct( protected ReferenceFetcher $referenceFetcher, diff --git a/app/View/ViewBlocks/PagesShowPageNav.php b/app/View/ViewBlocks/PagesShowPageNav.php index dbee578ab..cbde72f21 100644 --- a/app/View/ViewBlocks/PagesShowPageNav.php +++ b/app/View/ViewBlocks/PagesShowPageNav.php @@ -10,6 +10,7 @@ use Illuminate\Http\Request; class PagesShowPageNav extends ViewBlock { protected string $view = 'pages.parts.show-sidebar-section-page-nav'; + protected string $labelTranslationKey = 'entities.pages_navigation'; public function withData(array $viewData): array { diff --git a/app/View/ViewBlocks/PagesShowTags.php b/app/View/ViewBlocks/PagesShowTags.php index a2197a161..c200f197a 100644 --- a/app/View/ViewBlocks/PagesShowTags.php +++ b/app/View/ViewBlocks/PagesShowTags.php @@ -9,6 +9,7 @@ use Illuminate\Http\Request; class PagesShowTags extends ViewBlock { protected string $view = 'pages.parts.show-sidebar-section-tags'; + protected string $labelTranslationKey = 'entities.tags'; public function withData(array $viewData): array { diff --git a/app/View/ViewBlocks/ShelvesIndexActions.php b/app/View/ViewBlocks/ShelvesIndexActions.php index 2cc436116..81a35ded6 100644 --- a/app/View/ViewBlocks/ShelvesIndexActions.php +++ b/app/View/ViewBlocks/ShelvesIndexActions.php @@ -8,6 +8,7 @@ use Illuminate\Http\Request; class ShelvesIndexActions extends ViewBlock { protected string $view = 'shelves.parts.index-sidebar-section-actions'; + protected string $labelTranslationKey = 'common.actions'; public function withData(array $viewData): array { diff --git a/app/View/ViewBlocks/ShelvesIndexNew.php b/app/View/ViewBlocks/ShelvesIndexNew.php index 1d6af72c0..5a09a46af 100644 --- a/app/View/ViewBlocks/ShelvesIndexNew.php +++ b/app/View/ViewBlocks/ShelvesIndexNew.php @@ -9,6 +9,7 @@ use Illuminate\Http\Request; class ShelvesIndexNew extends ViewBlock { protected string $view = 'shelves.parts.index-sidebar-section-new'; + protected string $labelTranslationKey = 'entities.shelves_new'; public function __construct( protected BookshelfQueries $queries, diff --git a/app/View/ViewBlocks/ShelvesIndexPopular.php b/app/View/ViewBlocks/ShelvesIndexPopular.php index fc2950b92..5bf811ac5 100644 --- a/app/View/ViewBlocks/ShelvesIndexPopular.php +++ b/app/View/ViewBlocks/ShelvesIndexPopular.php @@ -9,6 +9,7 @@ use Illuminate\Http\Request; class ShelvesIndexPopular extends ViewBlock { protected string $view = 'shelves.parts.index-sidebar-section-popular'; + protected string $labelTranslationKey = 'entities.shelves_popular'; public function __construct( protected BookshelfQueries $queries, diff --git a/app/View/ViewBlocks/ShelvesIndexRecents.php b/app/View/ViewBlocks/ShelvesIndexRecents.php index 6570841a9..219cae559 100644 --- a/app/View/ViewBlocks/ShelvesIndexRecents.php +++ b/app/View/ViewBlocks/ShelvesIndexRecents.php @@ -9,6 +9,7 @@ use Illuminate\Http\Request; class ShelvesIndexRecents extends ViewBlock { protected string $view = 'shelves.parts.index-sidebar-section-recents'; + protected string $labelTranslationKey = 'entities.recently_viewed'; public function __construct( protected BookshelfQueries $queries, diff --git a/app/View/ViewBlocks/ShelvesShowActions.php b/app/View/ViewBlocks/ShelvesShowActions.php index d1bb965c8..d9683714d 100644 --- a/app/View/ViewBlocks/ShelvesShowActions.php +++ b/app/View/ViewBlocks/ShelvesShowActions.php @@ -9,6 +9,7 @@ use Illuminate\Http\Request; class ShelvesShowActions extends ViewBlock { protected string $view = 'shelves.parts.show-sidebar-section-actions'; + protected string $labelTranslationKey = 'common.actions'; public function withData(array $viewData): array { diff --git a/app/View/ViewBlocks/ShelvesShowActivity.php b/app/View/ViewBlocks/ShelvesShowActivity.php index 40e715bcd..e6bb612cc 100644 --- a/app/View/ViewBlocks/ShelvesShowActivity.php +++ b/app/View/ViewBlocks/ShelvesShowActivity.php @@ -10,6 +10,7 @@ use Illuminate\Http\Request; class ShelvesShowActivity extends ViewBlock { protected string $view = 'shelves.parts.show-sidebar-section-activity'; + protected string $labelTranslationKey = 'entities.recent_activity'; public function __construct( protected ActivityQueries $activityQueries, diff --git a/app/View/ViewBlocks/ShelvesShowDetails.php b/app/View/ViewBlocks/ShelvesShowDetails.php index 413b4796c..1d5d69cec 100644 --- a/app/View/ViewBlocks/ShelvesShowDetails.php +++ b/app/View/ViewBlocks/ShelvesShowDetails.php @@ -10,6 +10,7 @@ use Illuminate\Http\Request; class ShelvesShowDetails extends ViewBlock { protected string $view = 'shelves.parts.show-sidebar-section-details'; + protected string $labelTranslationKey = 'common.details'; public function __construct( protected ReferenceFetcher $referenceFetcher, diff --git a/app/View/ViewBlocks/ShelvesShowTags.php b/app/View/ViewBlocks/ShelvesShowTags.php index 9a61cc94d..c8efa2431 100644 --- a/app/View/ViewBlocks/ShelvesShowTags.php +++ b/app/View/ViewBlocks/ShelvesShowTags.php @@ -8,6 +8,7 @@ use Illuminate\Http\Request; class ShelvesShowTags extends ViewBlock { protected string $view = 'shelves.parts.show-sidebar-section-tags'; + protected string $labelTranslationKey = 'entities.tags'; public function withData(array $viewData): array { From 3205e73bb933cfcd26c0a74fedde99bac1afcb23 Mon Sep 17 00:00:00 2001 From: Dan Brown Date: Mon, 10 Aug 2026 19:18:05 +0100 Subject: [PATCH 08/21] User Prefs: Added new interface preferences section --- .../Controllers/UserAccountController.php | 14 +++++ app/View/ViewBlockDefaults.php | 22 ++++++++ app/View/ViewBlockManager.php | 21 ++++++++ lang/en/preferences.php | 6 +++ resources/icons/interface.svg | 1 + .../views/users/account/interface.blade.php | 53 +++++++++++++++++++ .../views/users/account/layout.blade.php | 1 + routes/web.php | 1 + 8 files changed, 119 insertions(+) create mode 100644 resources/icons/interface.svg create mode 100644 resources/views/users/account/interface.blade.php diff --git a/app/Users/Controllers/UserAccountController.php b/app/Users/Controllers/UserAccountController.php index 21816d5b8..a72025422 100644 --- a/app/Users/Controllers/UserAccountController.php +++ b/app/Users/Controllers/UserAccountController.php @@ -10,6 +10,7 @@ use BookStack\Settings\UserNotificationPreferences; use BookStack\Settings\UserShortcutMap; use BookStack\Uploads\ImageRepo; use BookStack\Users\UserRepo; +use BookStack\View\ViewBlockManager; use Closure; use Illuminate\Http\Request; use Illuminate\Validation\Rules\Password; @@ -159,6 +160,19 @@ class UserAccountController extends Controller return redirect('/my-account/notifications'); } + /** + * Show the view for the "Interface Preferences" user account area. + */ + public function showInterface(ViewBlockManager $viewBlockManager) + { + // TODO - Handle language save + + return view('users.account.interface', [ + 'category' => 'interface', + 'namedLocations' => $viewBlockManager->getNamedLocations(), + ]); + } + /** * Show the view for the "Access & Security" account options. */ diff --git a/app/View/ViewBlockDefaults.php b/app/View/ViewBlockDefaults.php index 33a08adc1..1e235b751 100644 --- a/app/View/ViewBlockDefaults.php +++ b/app/View/ViewBlockDefaults.php @@ -107,4 +107,26 @@ class ViewBlockDefaults { return self::$defaults[$location] ?? []; } + + /** + * Get the locations for all default blocks. + */ + public static function getLocations(): array + { + return array_keys(self::$defaults); + } + + public static function getLocationLabels(): array + { + return [ + 'home-default' => trans('common.homepage'), + 'home-non-default' => trans('preferences.layouts_home_non_default'), + 'shelves-index' => trans('entities.shelves'), + 'shelves-show' => trans('entities.shelf'), + 'books-index' => trans('entities.books'), + 'books-show' => trans('entities.book'), + 'chapters-show' => trans('entities.chapter'), + 'pages-show' => trans('entities.page'), + ]; + } } diff --git a/app/View/ViewBlockManager.php b/app/View/ViewBlockManager.php index 00535b2c1..939e4107d 100644 --- a/app/View/ViewBlockManager.php +++ b/app/View/ViewBlockManager.php @@ -44,4 +44,25 @@ class ViewBlockManager return app()->make($className); }, $sections); } + + /** + * Get the names of all locations where blocks are registered. + * Returns an array where the keys are location strings, and the + * values are translated labels for that location. + * @return array + */ + public function getNamedLocations(): array + { + $labels = ViewBlockDefaults::getLocationLabels(); + $defaults = ViewBlockDefaults::getLocations(); + $registered = array_keys($this->blocksByLocationAndPosition); + $merged = array_unique(array_merge($defaults, $registered)); + + $results = []; + foreach ($merged as $location) { + $results[$location] = $labels[$location] ?? $location; + } + + return $results; + } } diff --git a/lang/en/preferences.php b/lang/en/preferences.php index f4459d738..e197efd97 100644 --- a/lang/en/preferences.php +++ b/lang/en/preferences.php @@ -19,6 +19,12 @@ return [ 'shortcuts_update_success' => 'Shortcut preferences have been updated!', 'shortcuts_overview_desc' => 'Manage keyboard shortcuts you can use to navigate the system user interface.', + 'interface' => 'Interface Preferences', + 'interface_desc' => 'Customize various user interface elements you see across the application.', + 'layouts' => 'Customize UI Layout', + 'layouts_desc' => 'Customize the layout of sections shown in the user interface across various views.', + 'layouts_home_non_default' => 'Homepage (non-default)', + 'notifications' => 'Notification Preferences', 'notifications_desc' => 'Control the email notifications you receive when certain activity is performed within the system.', 'notifications_opt_own_page_changes' => 'Notify upon changes to pages I own', diff --git a/resources/icons/interface.svg b/resources/icons/interface.svg new file mode 100644 index 000000000..af1c5fc49 --- /dev/null +++ b/resources/icons/interface.svg @@ -0,0 +1 @@ + \ No newline at end of file diff --git a/resources/views/users/account/interface.blade.php b/resources/views/users/account/interface.blade.php new file mode 100644 index 000000000..8831b055a --- /dev/null +++ b/resources/views/users/account/interface.blade.php @@ -0,0 +1,53 @@ +@extends('users.account.layout') + +@section('main') +
    +
    + {{ method_field('put') }} + {{ csrf_field() }} + +

    {{ trans('preferences.interface') }}

    +

    {{ trans('preferences.interface_desc') }}

    + +
    + @include('users.parts.language-option-row', ['value' => old('language') ?? user()->getLocale()->appLocale()]) +
    + +
    + +
    + +
    +
    + +
    +

    {{ trans('preferences.layouts') }}

    +

    {{ trans('preferences.layouts_desc') }}

    + +
    + @foreach($namedLocations as $locationKey => $locationName) +
    +
    + {{ $locationName }} +
    +
    + {{ trans('common.configure') }} +
    +
    + @endforeach +
    + +
    + +
    +
    +
    +

    {{ trans('preferences.shortcuts_interface') }}

    +

    {{ trans('preferences.shortcuts_overview_desc') }}

    +
    +
    + {{ trans('common.open') }} +
    +
    +
    +@stop diff --git a/resources/views/users/account/layout.blade.php b/resources/views/users/account/layout.blade.php index df8ebc2d9..94f1f328d 100644 --- a/resources/views/users/account/layout.blade.php +++ b/resources/views/users/account/layout.blade.php @@ -11,6 +11,7 @@