refactor(notifications): move the notification pages to Twig
Two contracts change with them, and both serve every list page still to come:
- Pagination::render() returns Twig\Markup, so a template prints {{ pagination }}
with no raw filter. Plates keeps echoing it, since Markup is a string when used
as one.
- Notification::getMessageAttribute() returns Markup too: the event templates in
the configuration carry links, and under autoescape they would print as tags.
Whether the navigation is shown is decided by Pagination::hasPages() instead of
comparing the total against the per-page setting of the visitor, which is what
that comparison meant.
- Pagination::render() returns Twig\Markup, so a template prints {{ pagination }}
with no raw filter. Plates keeps echoing it, since Markup is a string when used
as one.
- Notification::getMessageAttribute() returns Markup too: the event templates in
the configuration carry links, and under autoescape they would print as tags.
Whether the navigation is shown is decided by Pagination::hasPages() instead of
comparing the total against the per-page setting of the visitor, which is what
that comparison meant.