diff --git a/app/Http/Controllers/PageRevisionController.php b/app/Http/Controllers/PageRevisionController.php
index 85ee6c2bc..3da5e7c2d 100644
--- a/app/Http/Controllers/PageRevisionController.php
+++ b/app/Http/Controllers/PageRevisionController.php
@@ -8,6 +8,8 @@ use BookStack\Entities\Repos\PageRepo;
use BookStack\Entities\Tools\PageContent;
use BookStack\Exceptions\NotFoundException;
use BookStack\Facades\Activity;
+use BookStack\Util\SimpleListOptions;
+use Illuminate\Http\Request;
use Ssddanbrown\HtmlDiff\Diff;
class PageRevisionController extends Controller
@@ -24,22 +26,29 @@ class PageRevisionController extends Controller
*
* @throws NotFoundException
*/
- public function index(string $bookSlug, string $pageSlug)
+ public function index(Request $request, string $bookSlug, string $pageSlug)
{
$page = $this->pageRepo->getBySlug($bookSlug, $pageSlug);
+ $listOptions = SimpleListOptions::fromRequest($request, 'page_revisions', true)->withSortOptions([
+ 'id' => trans('entities.pages_revisions_sort_number')
+ ]);
+
$revisions = $page->revisions()->select([
- 'id', 'page_id', 'name', 'created_at', 'created_by', 'updated_at',
- 'type', 'revision_number', 'summary',
- ])
+ 'id', 'page_id', 'name', 'created_at', 'created_by', 'updated_at',
+ 'type', 'revision_number', 'summary',
+ ])
->selectRaw("IF(markdown = '', false, true) as is_markdown")
->with(['page.book', 'createdBy'])
- ->get();
+ ->reorder('id', $listOptions->getOrder())
+ ->reorder('created_at', $listOptions->getOrder())
+ ->paginate(50);
$this->setPageTitle(trans('entities.pages_revisions_named', ['pageName' => $page->getShortName()]));
return view('pages.revisions', [
- 'revisions' => $revisions,
- 'page' => $page,
+ 'revisions' => $revisions,
+ 'page' => $page,
+ 'listOptions' => $listOptions,
]);
}
diff --git a/app/Http/Controllers/UserPreferencesController.php b/app/Http/Controllers/UserPreferencesController.php
index ca77dcd0b..972742e03 100644
--- a/app/Http/Controllers/UserPreferencesController.php
+++ b/app/Http/Controllers/UserPreferencesController.php
@@ -62,7 +62,7 @@ class UserPreferencesController extends Controller
*/
public function changeSort(Request $request, string $id, string $type)
{
- $validSortTypes = ['books', 'bookshelves', 'shelf_books', 'users', 'roles', 'webhooks', 'tags'];
+ $validSortTypes = ['books', 'bookshelves', 'shelf_books', 'users', 'roles', 'webhooks', 'tags', 'page_revisions'];
if (!in_array($type, $validSortTypes)) {
return redirect()->back(500);
}
diff --git a/app/Util/SimpleListOptions.php b/app/Util/SimpleListOptions.php
index f6daa6954..cb7e75a2d 100644
--- a/app/Util/SimpleListOptions.php
+++ b/app/Util/SimpleListOptions.php
@@ -28,11 +28,11 @@ class SimpleListOptions
* Create a new instance from the given request.
* Takes the item type (plural) that's used as a key for storing sort preferences.
*/
- public static function fromRequest(Request $request, string $typeKey): self
+ public static function fromRequest(Request $request, string $typeKey, bool $sortDescDefault = false): self
{
$search = $request->get('search', '');
$sort = setting()->getForCurrentUser($typeKey . '_sort', '');
- $order = setting()->getForCurrentUser($typeKey . '_sort_order', 'asc');
+ $order = setting()->getForCurrentUser($typeKey . '_sort_order', $sortDescDefault ? 'desc' : 'asc');
return new static($typeKey, $sort, $order, $search);
}
diff --git a/resources/lang/en/entities.php b/resources/lang/en/entities.php
index b3dfb0bf7..e7fbe37d9 100644
--- a/resources/lang/en/entities.php
+++ b/resources/lang/en/entities.php
@@ -233,12 +233,14 @@ return [
'pages_permissions_success' => 'Page permissions updated',
'pages_revision' => 'Revision',
'pages_revisions' => 'Page Revisions',
+ 'pages_revisions_desc' => 'Listed below are all the past revisions of this page. You can look back upon, compare, and restore old page versions if permissions allow. The full history of the page may not be fully reflected here since, depending on system configuration, old revisions could be auto-deleted.',
'pages_revisions_named' => 'Page Revisions for :pageName',
'pages_revision_named' => 'Page Revision for :pageName',
'pages_revision_restored_from' => 'Restored from #:id; :summary',
'pages_revisions_created_by' => 'Created By',
'pages_revisions_date' => 'Revision Date',
'pages_revisions_number' => '#',
+ 'pages_revisions_sort_number' => 'Revision Number',
'pages_revisions_numbered' => 'Revision #:id',
'pages_revisions_numbered_changes' => 'Revision #:id Changes',
'pages_revisions_editor' => 'Editor Type',
diff --git a/resources/sass/_layout.scss b/resources/sass/_layout.scss
index 105b6a16f..a5f895f80 100644
--- a/resources/sass/_layout.scss
+++ b/resources/sass/_layout.scss
@@ -202,6 +202,15 @@ body.flexbox {
/**
* Min width utilities
*/
+.min-width-xxxxs {
+ min-width: 60px;
+}
+.min-width-xxxs {
+ min-width: 80px;
+}
+.min-width-xxs {
+ min-width: 100px;
+}
.min-width-xs {
min-width: 120px;
}
diff --git a/resources/views/pages/parts/revision-table-row.blade.php b/resources/views/pages/parts/revisions-index-row.blade.php
similarity index 62%
rename from resources/views/pages/parts/revision-table-row.blade.php
rename to resources/views/pages/parts/revisions-index-row.blade.php
index 24301adc3..597b53234 100644
--- a/resources/views/pages/parts/revision-table-row.blade.php
+++ b/resources/views/pages/parts/revisions-index-row.blade.php
@@ -1,38 +1,43 @@
-<tr>
- <td>{{ $revision->revision_number == 0 ? '' : $revision->revision_number }}</td>
- <td>
+<div class="item-list-row flex-container-row items-center wrap">
+ <div class="flex fit-content min-width-xxxxs px-m py-xs">
+ <span class="hide-over-l">{{ trans('entities.pages_revisions_number') }}</span>
+ {{ $revision->revision_number == 0 ? '' : $revision->revision_number }}
+ </div>
+ <div class="flex-2 px-m py-xs min-width-s">
{{ $revision->name }}
<br>
- <small class="text-muted">({{ $revision->is_markdown ? 'Markdown' : 'WYSIWYG' }})</small>
- </td>
- <td style="line-height: 0;" width="30">
- @if($revision->createdBy)
- <img class="avatar" src="{{ $revision->createdBy->getAvatar(30) }}" alt="{{ $revision->createdBy->name }}">
- @endif
- </td>
- <td width="260">
- @if($revision->createdBy) {{ $revision->createdBy->name }} @else {{ trans('common.deleted_user') }} @endif
- <br>
- <div class="text-muted">
- <small>{{ $revision->created_at->formatLocalized('%e %B %Y %H:%M:%S') }}</small>
- <small>({{ $revision->created_at->diffForHumans() }})</small>
+ <small class="text-muted">(<strong class="hide-over-l">{{ trans('entities.pages_revisions_editor') }}: </strong>{{ $revision->is_markdown ? 'Markdown' : 'WYSIWYG' }})</small>
+ </div>
+ <div class="flex-3 px-m py-xs min-width-l">
+ <div class="flex-container-row items-center gap-s">
+ @if($revision->createdBy)
+ <img class="avatar flex-none" height="30" width="30" src="{{ $revision->createdBy->getAvatar(30) }}" alt="{{ $revision->createdBy->name }}">
+ @endif
+ <div>
+ @if($revision->createdBy) {{ $revision->createdBy->name }} @else {{ trans('common.deleted_user') }} @endif
+ <br>
+ <div class="text-muted">
+ <small>{{ $revision->created_at->formatLocalized('%e %B %Y %H:%M:%S') }}</small>
+ <small>({{ $revision->created_at->diffForHumans() }})</small>
+ </div>
+ </div>
</div>
- </td>
- <td>
+ </div>
+ <div class="flex-2 px-m py-xs min-width-m text-small">
{{ $revision->summary }}
- </td>
- <td class="actions text-small text-right">
+ </div>
+ <div class="flex-2 px-m py-xs actions text-small text-l-right min-width-l">
<a href="{{ $revision->getUrl('changes') }}" target="_blank" rel="noopener">{{ trans('entities.pages_revisions_changes') }}</a>
- <span class="text-muted"> | </span>
+ <span class="text-muted opacity-70"> | </span>
- @if ($index === 0)
+ @if ($current)
<a target="_blank" rel="noopener" href="{{ $revision->page->getUrl() }}"><i>{{ trans('entities.pages_revisions_current') }}</i></a>
@else
<a href="{{ $revision->getUrl() }}" target="_blank" rel="noopener">{{ trans('entities.pages_revisions_preview') }}</a>
@if(userCan('page-update', $revision->page))
- <span class="text-muted"> | </span>
+ <span class="text-muted opacity-70"> | </span>
<div component="dropdown" class="dropdown-container">
<a refs="dropdown@toggle" href="#" aria-haspopup="true" aria-expanded="false">{{ trans('entities.pages_revisions_restore') }}</a>
<ul refs="dropdown@menu" class="dropdown-menu" role="menu">
@@ -52,7 +57,7 @@
@endif
@if(userCan('page-delete', $revision->page))
- <span class="text-muted"> | </span>
+ <span class="text-muted opacity-70"> | </span>
<div component="dropdown" class="dropdown-container">
<a refs="dropdown@toggle" href="#" aria-haspopup="true" aria-expanded="false">{{ trans('common.delete') }}</a>
<ul refs="dropdown@menu" class="dropdown-menu" role="menu">
@@ -71,5 +76,5 @@
</div>
@endif
@endif
- </td>
-</tr>
\ No newline at end of file
+ </div>
+</div>
\ No newline at end of file
diff --git a/resources/views/pages/revisions.blade.php b/resources/views/pages/revisions.blade.php
index 3e7edad99..9f462e930 100644
--- a/resources/views/pages/revisions.blade.php
+++ b/resources/views/pages/revisions.blade.php
@@ -17,26 +17,36 @@
<main class="card content-wrap">
<h1 class="list-heading">{{ trans('entities.pages_revisions') }}</h1>
+
+ <p class="text-muted">{{ trans('entities.pages_revisions_desc') }}</p>
+
+ <div class="flex-container-row my-m items-center justify-space-between wrap gap-x-m gap-y-s">
+ {{ $revisions->links() }}
+ <div>
+ @include('common.sort', $listOptions->getSortControlData())
+ </div>
+ </div>
+
@if(count($revisions) > 0)
-
- <table class="table">
- <tr>
- <th width="56">{{ trans('entities.pages_revisions_number') }}</th>
- <th>
- {{ trans('entities.pages_name') }} / {{ trans('entities.pages_revisions_editor') }}
- </th>
- <th colspan="2">{{ trans('entities.pages_revisions_created_by') }} / {{ trans('entities.pages_revisions_date') }}</th>
- <th>{{ trans('entities.pages_revisions_changelog') }}</th>
- <th class="text-right">{{ trans('common.actions') }}</th>
- </tr>
+ <div class="item-list">
+ <div class="item-list-row flex-container-row items-center strong hide-under-l">
+ <div class="flex fit-content min-width-xxxxs px-m py-xs">{{ trans('entities.pages_revisions_number') }}</div>
+ <div class="flex-2 px-m py-xs">{{ trans('entities.pages_name') }} / {{ trans('entities.pages_revisions_editor') }}</div>
+ <div class="flex-3 px-m py-xs">{{ trans('entities.pages_revisions_created_by') }} / {{ trans('entities.pages_revisions_date') }}</div>
+ <div class="flex-2 px-m py-xs">{{ trans('entities.pages_revisions_changelog') }}</div>
+ <div class="flex-2 px-m py-xs text-right">{{ trans('common.actions') }}</div>
+ </div>
@foreach($revisions as $index => $revision)
- @include('pages.parts.revision-table-row', ['revision' => $revision])
+ @include('pages.parts.revisions-index-row', ['revision' => $revision, 'current' => $page->revision_count === $revision->revision_number])
@endforeach
- </table>
-
+ </div>
@else
<p>{{ trans('entities.pages_revisions_none') }}</p>
@endif
+
+ <div class="my-m">
+ {{ $revisions->links() }}
+ </div>
</main>
</div>