From 90da48c1234aaf43bed3c2212bba0d69e21ecb8d Mon Sep 17 00:00:00 2001
From: "David M. Johnson" <snoopdave@apache.org>
Date: Sat, 24 Oct 2026 16:43:04 -0400
Subject: [PATCH 1/4] Admin UI polish: pagination, accordions, spacing, date
 fields

- One pager for Entries, Comments and media search: First / Newer /
  Page N of M / Older / Last. Backed by new count queries that share
  the list queries' WHERE clauses. Media search results are reachable
  past page 1 for the first time.
- Entry editor panels are Bootstrap 5 accordions with a chevron.
- Spacing for editor buttons, accordions and sidebar filter forms.
- Comment and entry tables fit narrow screens; long unbroken spam
  URLs wrap.
- Date fields: attached calendar button that works, clear button on
  filters, no longer greyed out; one shared script replaces five
  per-page snippets.
- CommentTest had no @Test annotations and never ran; it does now.
---
 .../weblogger/business/MediaFileManager.java  |   7 +
 .../business/WeblogEntryManager.java          |  18 +
 .../business/jpa/JPAMediaFileManagerImpl.java |  89 +-
 .../jpa/JPAWeblogEntryManagerImpl.java        | 134 ++-
 .../admin/GlobalCommentManagement.java        |  38 +-
 .../weblogger/ui/struts2/editor/Comments.java |  34 +-
 .../weblogger/ui/struts2/editor/Entries.java  |  18 +-
 .../struts2/editor/MediaFileSearchBean.java   | 420 ++++----
 .../ui/struts2/editor/MediaFileView.java      | 978 +++++++++---------
 .../ui/struts2/pagers/CommentsPager.java      |  70 +-
 .../ui/struts2/pagers/EntriesPager.java       |  70 +-
 .../ui/struts2/pagers/LinkPager.java          | 110 ++
 .../ui/struts2/pagers/MediaFilePager.java     | 157 ++-
 .../resources/ApplicationResources.properties |   8 +
 .../webapp/WEB-INF/jsps/editor/Comments.jsp   | 427 ++++----
 .../WEB-INF/jsps/editor/CommentsSidebar.jsp   |  50 +-
 .../webapp/WEB-INF/jsps/editor/Entries.jsp    | 208 ++--
 .../WEB-INF/jsps/editor/EntriesSidebar.jsp    |  38 +-
 .../webapp/WEB-INF/jsps/editor/EntryEdit.jsp  | 114 +-
 .../WEB-INF/jsps/editor/EntryEditor.jsp       |  23 +-
 .../WEB-INF/jsps/editor/MediaFilePager.jsp    |  83 ++
 .../WEB-INF/jsps/editor/MediaFileSidebar.jsp  |   8 +
 .../WEB-INF/jsps/editor/MediaFileView.jsp     |   8 +-
 .../main/webapp/WEB-INF/jsps/editor/Pager.jsp |  78 ++
 .../WEB-INF/jsps/editor/TemplateEdit.jsp      |  24 +-
 .../main/webapp/WEB-INF/jsps/tiles/head.jsp   |   1 +
 .../webapp/roller-ui/scripts/date-fields.js   |  35 +
 .../main/webapp/roller-ui/styles/roller.css   |  44 +-
 .../weblogger/business/CommentTest.java       |   8 +
 .../weblogger/business/MediaFileTest.java     |   6 +
 .../weblogger/business/WeblogEntryTest.java   |  39 +
 .../weblogger/ui/EntryEditorAssetsTest.java   |  23 +-
 .../ui/struts2/pagers/LinkPagerTest.java      |  98 ++
 33 files changed, 1915 insertions(+), 1551 deletions(-)
 create mode 100644 app/src/main/java/org/apache/roller/weblogger/ui/struts2/pagers/LinkPager.java
 create mode 100644 app/src/main/webapp/WEB-INF/jsps/editor/MediaFilePager.jsp
 create mode 100644 app/src/main/webapp/WEB-INF/jsps/editor/Pager.jsp
 create mode 100644 app/src/main/webapp/roller-ui/scripts/date-fields.js
 create mode 100644 app/src/test/java/org/apache/roller/weblogger/ui/struts2/pagers/LinkPagerTest.java

diff --git a/app/src/main/java/org/apache/roller/weblogger/business/MediaFileManager.java b/app/src/main/java/org/apache/roller/weblogger/business/MediaFileManager.java
index f8a3611328..326b46feb4 100644
--- a/app/src/main/java/org/apache/roller/weblogger/business/MediaFileManager.java
+++ b/app/src/main/java/org/apache/roller/weblogger/business/MediaFileManager.java
@@ -103,6 +103,13 @@ void removeMediaFile(Weblog weblog, MediaFile mediaFile)
     List<MediaFile> searchMediaFiles(Weblog weblog,
             MediaFileFilter filter) throws WebloggerException;
 
+    /**
+     * Count the media files searchMediaFiles() would return for the same filter,
+     * ignoring start index, length and order. Used to size pagers.
+     */
+    long getMediaFileCount(Weblog weblog,
+            MediaFileFilter filter) throws WebloggerException;
+
     /**
      * Create default folder for media files in a weblog.
      */
diff --git a/app/src/main/java/org/apache/roller/weblogger/business/WeblogEntryManager.java b/app/src/main/java/org/apache/roller/weblogger/business/WeblogEntryManager.java
index 3ed56d16b6..f841193e72 100644
--- a/app/src/main/java/org/apache/roller/weblogger/business/WeblogEntryManager.java
+++ b/app/src/main/java/org/apache/roller/weblogger/business/WeblogEntryManager.java
@@ -79,6 +79,16 @@ WeblogEntry getWeblogEntryByAnchor(Weblog website, String anchor)
     List<WeblogEntry> getWeblogEntries(WeblogEntrySearchCriteria wesc)
             throws WebloggerException;
 
+    /**
+     * Count the rows getWeblogEntries() would return for the same criteria,
+     * ignoring offset, max results and sort order. Used to size pagers.
+     * @param wesc WeblogEntrySearchCriteria object listing desired search parameters
+     * @return number of matching rows
+     * @throws WebloggerException
+     */
+    long getWeblogEntryCount(WeblogEntrySearchCriteria wesc)
+            throws WebloggerException;
+
     /**
      * Get Weblog Entries grouped by day.
      * @param wesc WeblogEntrySearchCriteria object listing desired search parameters
@@ -227,6 +237,14 @@ List<WeblogCategory> getWeblogCategories(Weblog website)
      */
     List<WeblogEntryComment> getComments(CommentSearchCriteria csc) throws WebloggerException;
 
+    /**
+     * Count the comments getComments() would return for the same criteria,
+     * ignoring offset, max results and sort order. Used to size pagers.
+     * @param csc CommentSearchCriteria object with fields indicating search criteria
+     * @return number of matching comments
+     */
+    long getCommentCount(CommentSearchCriteria csc) throws WebloggerException;
+
     /**
      * Deletes comments that match paramters.
      * @param website    Website or null for all comments on site
diff --git a/app/src/main/java/org/apache/roller/weblogger/business/jpa/JPAMediaFileManagerImpl.java b/app/src/main/java/org/apache/roller/weblogger/business/jpa/JPAMediaFileManagerImpl.java
index e3c0ddcaa4..a3eec1b38e 100644
--- a/app/src/main/java/org/apache/roller/weblogger/business/jpa/JPAMediaFileManagerImpl.java
+++ b/app/src/main/java/org/apache/roller/weblogger/business/jpa/JPAMediaFileManagerImpl.java
@@ -583,11 +583,65 @@ public List<MediaFile> searchMediaFiles(Weblog weblog,
             MediaFileFilter filter) throws WebloggerException {
 
         List<Object> params = new ArrayList<>();
-        int size = 0;
-        String queryString = "SELECT m FROM MediaFile m WHERE ";
-        StringBuilder whereClause = new StringBuilder();
+        String queryString = "SELECT m " + mediaFilesFromWhere(weblog, filter, params);
         StringBuilder orderBy = new StringBuilder();
 
+        if (filter.getOrder() != null) {
+            switch (filter.getOrder()) {
+            case NAME:
+                orderBy.append(" order by m.name");
+                break;
+            case DATE_UPLOADED:
+                orderBy.append(" order by m.dateUploaded");
+                break;
+            case TYPE:
+                orderBy.append(" order by m.contentType");
+                break;
+            default:
+            }
+        } else {
+            orderBy.append(" order by m.name");
+        }
+
+        TypedQuery<MediaFile> query = strategy.getDynamicQuery(queryString
+                + orderBy.toString(), MediaFile.class);
+        for (int i = 0; i < params.size(); i++) {
+            query.setParameter(i + 1, params.get(i));
+        }
+
+        if (filter.getStartIndex() >= 0) {
+            query.setFirstResult(filter.getStartIndex());
+            query.setMaxResults(filter.getLength());
+        }
+        return query.getResultList();
+    }
+
+    /**
+     * {@inheritDoc}
+     */
+    @Override
+    public long getMediaFileCount(Weblog weblog,
+            MediaFileFilter filter) throws WebloggerException {
+
+        List<Object> params = new ArrayList<>();
+        TypedQuery<Long> query = strategy.getDynamicQuery(
+                "SELECT COUNT(m) " + mediaFilesFromWhere(weblog, filter, params), Long.class);
+        for (int i = 0; i < params.size(); i++) {
+            query.setParameter(i + 1, params.get(i));
+        }
+        return query.getSingleResult();
+    }
+
+    /**
+     * Build the FROM and WHERE clauses shared by searchMediaFiles() and
+     * getMediaFileCount(), adding positional parameters to params.
+     */
+    private String mediaFilesFromWhere(Weblog weblog, MediaFileFilter filter,
+            List<Object> params) {
+
+        int size = params.size();
+        StringBuilder whereClause = new StringBuilder();
+
         params.add(size++, weblog);
         whereClause.append("m.directory.weblog = ?").append(size);
 
@@ -659,34 +713,7 @@ public List<MediaFile> searchMediaFiles(Weblog weblog,
             }
         }
 
-        if (filter.getOrder() != null) {
-            switch (filter.getOrder()) {
-            case NAME:
-                orderBy.append(" order by m.name");
-                break;
-            case DATE_UPLOADED:
-                orderBy.append(" order by m.dateUploaded");
-                break;
-            case TYPE:
-                orderBy.append(" order by m.contentType");
-                break;
-            default:
-            }
-        } else {
-            orderBy.append(" order by m.name");
-        }
-
-        TypedQuery<MediaFile> query = strategy.getDynamicQuery(queryString
-                + whereClause.toString() + orderBy.toString(), MediaFile.class);
-        for (int i = 0; i < params.size(); i++) {
-            query.setParameter(i + 1, params.get(i));
-        }
-
-        if (filter.getStartIndex() >= 0) {
-            query.setFirstResult(filter.getStartIndex());
-            query.setMaxResults(filter.getLength());
-        }
-        return query.getResultList();
+        return "FROM MediaFile m WHERE " + whereClause;
     }
 
     /**
diff --git a/app/src/main/java/org/apache/roller/weblogger/business/jpa/JPAWeblogEntryManagerImpl.java b/app/src/main/java/org/apache/roller/weblogger/business/jpa/JPAWeblogEntryManagerImpl.java
index 477179a3f7..93a478a9ed 100644
--- a/app/src/main/java/org/apache/roller/weblogger/business/jpa/JPAWeblogEntryManagerImpl.java
+++ b/app/src/main/java/org/apache/roller/weblogger/business/jpa/JPAWeblogEntryManagerImpl.java
@@ -381,19 +381,67 @@ public List<WeblogCategory> getWeblogCategories(Weblog website)
     @Override
     public List<WeblogEntry> getWeblogEntries(WeblogEntrySearchCriteria wesc) throws WebloggerException {
 
+        List<Object> params = new ArrayList<>();
+        StringBuilder queryString = new StringBuilder("SELECT e ");
+        queryString.append(weblogEntriesFromWhere(wesc, params));
+
+        if (wesc.getSortBy() != null && wesc.getSortBy().equals(WeblogEntrySearchCriteria.SortBy.UPDATE_TIME)) {
+            queryString.append(" ORDER BY e.updateTime ");
+        } else {
+            queryString.append(" ORDER BY e.pubTime ");
+        }
+        
+        if (wesc.getSortOrder() != null && wesc.getSortOrder().equals(WeblogEntrySearchCriteria.SortOrder.ASCENDING)) {
+            queryString.append("ASC ");
+        } else {
+            queryString.append("DESC ");
+        }
+        
+        
+        TypedQuery<WeblogEntry> query = strategy.getDynamicQuery(queryString.toString(), WeblogEntry.class);
+        for (int i=0; i<params.size(); i++) {
+            query.setParameter(i+1, params.get(i));
+        }
+        
+        setFirstMax( query, wesc.getOffset(), wesc.getMaxResults() );
+        return query.getResultList();
+    }
+
+    /**
+     * @inheritDoc
+     */
+    @Override
+    public long getWeblogEntryCount(WeblogEntrySearchCriteria wesc) throws WebloggerException {
+        List<Object> params = new ArrayList<>();
+        // COUNT(e), not COUNT(DISTINCT e): the tag join can repeat an entry, and the
+        // count must match the rows getWeblogEntries() pages through.
+        TypedQuery<Long> query = strategy.getDynamicQuery(
+                "SELECT COUNT(e) " + weblogEntriesFromWhere(wesc, params), Long.class);
+        for (int i=0; i<params.size(); i++) {
+            query.setParameter(i+1, params.get(i));
+        }
+        return query.getSingleResult();
+    }
+
+    /**
+     * Build the FROM and WHERE clauses shared by getWeblogEntries() and
+     * getWeblogEntryCount(), adding positional parameters to params.
+     */
+    private String weblogEntriesFromWhere(WeblogEntrySearchCriteria wesc, List<Object> params)
+            throws WebloggerException {
+
         WeblogCategory cat = null;
         if (StringUtils.isNotEmpty(wesc.getCatName()) && wesc.getWeblog() != null) {
             cat = getWeblogCategoryByName(wesc.getWeblog(), wesc.getCatName());
         }
 
-        List<Object> params = new ArrayList<>();
-        int size = 0;
+        int size = params.size();
         StringBuilder queryString = new StringBuilder();
         
         if (wesc.getTags() == null || wesc.getTags().isEmpty()) {
-            queryString.append("SELECT e FROM WeblogEntry e WHERE ");
+            queryString.append("FROM WeblogEntry e WHERE ");
         } else {
-            queryString.append("SELECT e FROM WeblogEntry e JOIN e.tags t WHERE ");
+            queryString.append("FROM WeblogEntry e JOIN e.tags t WHERE ");
             queryString.append("(");
             for (int i = 0; i < wesc.getTags().size(); i++) {
                 if (i != 0) {
@@ -453,26 +501,7 @@ public List<WeblogEntry> getWeblogEntries(WeblogEntrySearchCriteria wesc) throws
             queryString.append(") ");
         }
 
-        if (wesc.getSortBy() != null && wesc.getSortBy().equals(WeblogEntrySearchCriteria.SortBy.UPDATE_TIME)) {
-            queryString.append(" ORDER BY e.updateTime ");
-        } else {
-            queryString.append(" ORDER BY e.pubTime ");
-        }
-        
-        if (wesc.getSortOrder() != null && wesc.getSortOrder().equals(WeblogEntrySearchCriteria.SortOrder.ASCENDING)) {
-            queryString.append("ASC ");
-        } else {
-            queryString.append("DESC ");
-        }
-        
-        
-        TypedQuery<WeblogEntry> query = strategy.getDynamicQuery(queryString.toString(), WeblogEntry.class);
-        for (int i=0; i<params.size(); i++) {
-            query.setParameter(i+1, params.get(i));
-        }
-        
-        setFirstMax( query, wesc.getOffset(), wesc.getMaxResults() );
-        return query.getResultList();
+        return queryString.toString();
     }
     
     /**
@@ -631,10 +660,46 @@ public boolean isWeblogCategoryInUse(WeblogCategory cat)
     public List<WeblogEntryComment> getComments(CommentSearchCriteria csc) throws WebloggerException {
         
         List<Object> params = new ArrayList<>();
-        int size = 0;
-        StringBuilder queryString = new StringBuilder();
-        queryString.append("SELECT c FROM WeblogEntryComment c ");
+        StringBuilder queryString = new StringBuilder("SELECT c ");
+        queryString.append(commentsFromWhere(csc, params));
+        if (csc.isReverseChrono()) {
+            queryString.append(" ORDER BY c.postTime DESC");
+        } else {
+            queryString.append(" ORDER BY c.postTime ASC");
+        }
         
+        TypedQuery<WeblogEntryComment> query = strategy.getDynamicQuery(queryString.toString(), WeblogEntryComment.class);
+        setFirstMax( query, csc.getOffset(), csc.getMaxResults());
+        for (int i=0; i<params.size(); i++) {
+            query.setParameter(i+1, params.get(i));
+        }
+        return query.getResultList();
+        
+    }
+
+    /**
+     * @inheritDoc
+     */
+    @Override
+    public long getCommentCount(CommentSearchCriteria csc) throws WebloggerException {
+        List<Object> params = new ArrayList<>();
+        TypedQuery<Long> query = strategy.getDynamicQuery(
+                "SELECT COUNT(c) " + commentsFromWhere(csc, params), Long.class);
+        for (int i=0; i<params.size(); i++) {
+            query.setParameter(i+1, params.get(i));
+        }
+        return query.getSingleResult();
+    }
+
+    /**
+     * Build the FROM and WHERE clauses shared by getComments() and
+     * getCommentCount(), adding positional parameters to params.
+     */
+    private String commentsFromWhere(CommentSearchCriteria csc, List<Object> params) {
+
+        int size = params.size();
+        StringBuilder queryString = new StringBuilder("FROM WeblogEntryComment c ");
+
         StringBuilder whereClause = new StringBuilder();
         if (csc.getEntry() != null) {
             params.add(size++, csc.getEntry());
@@ -669,19 +734,8 @@ public List<WeblogEntryComment> getComments(CommentSearchCriteria csc) throws We
         if(whereClause.length() != 0) {
             queryString.append(" WHERE ").append(whereClause);
         }
-        if (csc.isReverseChrono()) {
-            queryString.append(" ORDER BY c.postTime DESC");
-        } else {
-            queryString.append(" ORDER BY c.postTime ASC");
-        }
-        
-        TypedQuery<WeblogEntryComment> query = strategy.getDynamicQuery(queryString.toString(), WeblogEntryComment.class);
-        setFirstMax( query, csc.getOffset(), csc.getMaxResults());
-        for (int i=0; i<params.size(); i++) {
-            query.setParameter(i+1, params.get(i));
-        }
-        return query.getResultList();
-        
+
+        return queryString.toString();
     }
     
     
diff --git a/app/src/main/java/org/apache/roller/weblogger/ui/struts2/admin/GlobalCommentManagement.java b/app/src/main/java/org/apache/roller/weblogger/ui/struts2/admin/GlobalCommentManagement.java
index caac1056d9..6f2c230785 100644
--- a/app/src/main/java/org/apache/roller/weblogger/ui/struts2/admin/GlobalCommentManagement.java
+++ b/app/src/main/java/org/apache/roller/weblogger/ui/struts2/admin/GlobalCommentManagement.java
@@ -99,7 +99,7 @@ public boolean isWeblogRequired() {
     public void loadComments() {
         
         List<WeblogEntryComment> comments = Collections.emptyList();
-        boolean hasMore = false;
+        long totalCount = 0;
         try {
             WeblogEntryManager wmgr = WebloggerFactory.getWeblogger().getWeblogEntryManager();
 
@@ -109,19 +109,13 @@ public void loadComments() {
             csc.setEndDate(getBean().getEndDate());
             csc.setStatus(getBean().getStatus());
             csc.setReverseChrono(true);
+            totalCount = wmgr.getCommentCount(csc);
             csc.setOffset(getBean().getPage() * COUNT);
-            csc.setMaxResults(COUNT+1);
+            csc.setMaxResults(COUNT);
 
-            List<WeblogEntryComment> rawComments = wmgr.getComments(csc);
-            comments = new ArrayList<>();
-            comments.addAll(rawComments);   
+            comments = new ArrayList<>(wmgr.getComments(csc));
             
             if(!comments.isEmpty()) {
-                if(comments.size() > COUNT) {
-                    comments.remove(comments.size()-1);
-                    hasMore = true;
-                }
-                
                 setFirstComment(comments.get(0));
                 setLastComment(comments.get(comments.size()-1));
             }
@@ -132,7 +126,7 @@ public void loadComments() {
         
         // build comments pager
         String baseUrl = buildBaseUrl();
-        setPager(new CommentsPager(baseUrl, getBean().getPage(), comments, hasMore));
+        setPager(new CommentsPager(baseUrl, getBean().getPage(), comments, totalCount, COUNT));
     }
     
     
@@ -184,25 +178,9 @@ public String query() {
         // load bean data using comments list
         getBean().loadCheckboxes(getPager().getItems());
         
-        try {
-            WeblogEntryManager wmgr = WebloggerFactory.getWeblogger().getWeblogEntryManager();
-
-            CommentSearchCriteria csc = new CommentSearchCriteria();
-            csc.setSearchText(getBean().getSearchString());
-            csc.setStartDate(getBean().getStartDate());
-            csc.setEndDate(getBean().getEndDate());
-            csc.setStatus(getBean().getStatus());
-            csc.setReverseChrono(true);
-
-            List<WeblogEntryComment> allMatchingComments = wmgr.getComments(csc);
-
-            if(allMatchingComments.size() > COUNT) {
-                setBulkDeleteCount(allMatchingComments.size());
-            }
-            
-        } catch (WebloggerException ex) {
-            log.error("Error looking up comments", ex);
-            addError("commentManagement.lookupError");
+        // offer bulk delete when the matches span more than one page
+        if (getPager().getTotalCount() > COUNT) {
+            setBulkDeleteCount((int) getPager().getTotalCount());
         }
         
         return LIST;
diff --git a/app/src/main/java/org/apache/roller/weblogger/ui/struts2/editor/Comments.java b/app/src/main/java/org/apache/roller/weblogger/ui/struts2/editor/Comments.java
index 889fc9cae4..a19f7f9800 100644
--- a/app/src/main/java/org/apache/roller/weblogger/ui/struts2/editor/Comments.java
+++ b/app/src/main/java/org/apache/roller/weblogger/ui/struts2/editor/Comments.java
@@ -99,30 +99,24 @@ public List<String> requiredWeblogPermissionActions() {
     public void loadComments() {
 
         List<WeblogEntryComment> comments = Collections.emptyList();
-        boolean hasMore = false;
+        long totalCount = 0;
         try {
             WeblogEntryManager wmgr = WebloggerFactory.getWeblogger()
                     .getWeblogEntryManager();
 
             if (!resolveQueryEntry(wmgr)) {
                 setPager(new CommentsPager(buildBaseUrl(), getBean().getPage(),
-                        comments, false));
+                        comments, 0, COUNT));
                 return;
             }
 
             CommentSearchCriteria csc = getCommentSearchCriteria();
+            totalCount = wmgr.getCommentCount(csc);
             csc.setOffset(getBean().getPage() * COUNT);
-            csc.setMaxResults(COUNT + 1);
+            csc.setMaxResults(COUNT);
 
-            List<WeblogEntryComment> rawComments = wmgr.getComments(csc);
-            comments = new ArrayList<>();
-            comments.addAll(rawComments);
+            comments = new ArrayList<>(wmgr.getComments(csc));
             if (!comments.isEmpty()) {
-                if (comments.size() > COUNT) {
-                    comments.remove(comments.size() - 1);
-                    hasMore = true;
-                }
-
                 setFirstComment(comments.get(0));
                 setLastComment(comments.get(comments
                         .size() - 1));
@@ -135,7 +129,7 @@ public void loadComments() {
         // build comments pager
         String baseUrl = buildBaseUrl();
         setPager(new CommentsPager(baseUrl, getBean().getPage(), comments,
-                hasMore));
+                totalCount, COUNT));
     }
 
     // use the action data to build a url representing this action, including
@@ -191,19 +185,9 @@ public String query() {
             return LIST;
         }
 
-        try {
-            WeblogEntryManager wmgr = WebloggerFactory.getWeblogger().getWeblogEntryManager();
-
-            CommentSearchCriteria csc = getCommentSearchCriteria();
-
-            List<WeblogEntryComment> allMatchingComments = wmgr.getComments(csc);
-            if (allMatchingComments.size() > COUNT) {
-                setBulkDeleteCount(allMatchingComments.size());
-            }
-
-        } catch (WebloggerException ex) {
-            log.error("Error looking up comments", ex);
-            addError("Error looking up comments");
+        // offer bulk delete when the matches span more than one page
+        if (getPager().getTotalCount() > COUNT) {
+            setBulkDeleteCount((int) getPager().getTotalCount());
         }
 
         return LIST;
diff --git a/app/src/main/java/org/apache/roller/weblogger/ui/struts2/editor/Entries.java b/app/src/main/java/org/apache/roller/weblogger/ui/struts2/editor/Entries.java
index e34aa47784..942a12648f 100644
--- a/app/src/main/java/org/apache/roller/weblogger/ui/struts2/editor/Entries.java
+++ b/app/src/main/java/org/apache/roller/weblogger/ui/struts2/editor/Entries.java
@@ -85,7 +85,7 @@ public String execute() {
         }
         
         List<WeblogEntry> entries = null;
-        boolean hasMore = false;
+        long totalCount = 0;
         try {
             String status = getBean().getStatus();
             
@@ -99,18 +99,12 @@ public String execute() {
             wesc.setStatus("ALL".equals(status) ? null : WeblogEntry.PubStatus.valueOf(status));
             wesc.setText(getBean().getText());
             wesc.setSortBy(getBean().getSortBy());
+            totalCount = wmgr.getWeblogEntryCount(wesc);
             wesc.setOffset(getBean().getPage() * COUNT);
-            wesc.setMaxResults(COUNT + 1);
-            List<WeblogEntry> rawEntries = wmgr.getWeblogEntries(wesc);
-            entries = new ArrayList<>();
-            entries.addAll(rawEntries);
+            wesc.setMaxResults(COUNT);
+            entries = new ArrayList<>(wmgr.getWeblogEntries(wesc));
             if (!entries.isEmpty()) {
-                log.debug("query found "+rawEntries.size()+" results");
-                
-                if(rawEntries.size() > COUNT) {
-                    entries.remove(entries.size()-1);
-                    hasMore = true;
-                }
+                log.debug("query found "+entries.size()+" of "+totalCount+" results");
                 
                 setFirstEntry(entries.get(0));
                 setLastEntry(entries.get(entries.size()-1));
@@ -122,7 +116,7 @@ public String execute() {
         
         // build entries pager
         String baseUrl = buildBaseUrl();
-        setPager(new EntriesPager(baseUrl, getBean().getPage(), entries, hasMore));
+        setPager(new EntriesPager(baseUrl, getBean().getPage(), entries, totalCount, COUNT));
                 
         return LIST;
     }
diff --git a/app/src/main/java/org/apache/roller/weblogger/ui/struts2/editor/MediaFileSearchBean.java b/app/src/main/java/org/apache/roller/weblogger/ui/struts2/editor/MediaFileSearchBean.java
index 3b96c875c9..d015ae1ad9 100644
--- a/app/src/main/java/org/apache/roller/weblogger/ui/struts2/editor/MediaFileSearchBean.java
+++ b/app/src/main/java/org/apache/roller/weblogger/ui/struts2/editor/MediaFileSearchBean.java
@@ -1,211 +1,209 @@
-/*
- * Licensed to the Apache Software Foundation (ASF) under one or more
- *  contributor license agreements.  The ASF licenses this file to You
- * under the Apache License, Version 2.0 (the "License"); you may not
- * use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *     http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.  For additional information regarding
- * copyright in this work, please see the NOTICE file in the top level
- * directory of this distribution.
- */
-package org.apache.roller.weblogger.ui.struts2.editor;
-
-import java.util.Arrays;
-
-import java.util.ResourceBundle;
-import org.apache.commons.lang3.StringUtils;
-import org.apache.roller.util.RollerConstants;
-import org.apache.roller.weblogger.pojos.MediaFileFilter;
-import org.apache.roller.weblogger.pojos.MediaFileType;
-import org.apache.roller.weblogger.pojos.MediaFileFilter.MediaFileOrder;
-import org.apache.roller.weblogger.pojos.MediaFileFilter.SizeFilterType;
-
-/**
- * Bean for holding media file search criteria.
- */
-public class MediaFileSearchBean {
-    private transient ResourceBundle bundle =
-            ResourceBundle.getBundle("ApplicationResources");
-
-    public static final int PAGE_SIZE = 10;
-
-    // Media file name as search criteria
-    private String name;
-
-    // Media file type as search criteria
-    private String type;
-
-    // Type of size filter as search criteria
-    private String sizeFilterType;
-
-    // Size of file as search criteria
-    private long size;
-
-    // Size unit
-    private String sizeUnit;
-
-    // Tags as search criteria
-    private String tags;
-
-    // Page number of results
-    private int pageNum = 0;
-    
-    // Sort option for search results
-    private int sortOption;
-
-    public String getName() {
-        return name;
-    }
-
-    public void setName(String name) {
-        this.name = name;
-    }
-
-    public String getType() {
-        return type;
-    }
-
-    public String getTypeLabel() {
-        return this.bundle.getString(type);
-    }
-
-    public void setType(String type) {
-        this.type = type;
-    }
-
-    public String getSizeFilterType() {
-        return sizeFilterType;
-    }
-
-    public void setSizeFilterType(String sizeFilterType) {
-        this.sizeFilterType = sizeFilterType;
-    }
-
-    public String getSizeFilterTypeLabel() {
-        return this.bundle.getString(sizeFilterType);
-    }
-
-    public long getSize() {
-        return size;
-    }
-
-    public void setSize(long size) {
-        this.size = size;
-    }
-
-    public String getSizeUnit() {
-        return sizeUnit;
-    }
-
-    public String getSizeUnitLabel() {
-        return this.bundle.getString(sizeUnit);
-    }
-
-    public void setSizeUnit(String sizeUnit) {
-        this.sizeUnit = sizeUnit;
-    }
-
-    public String getTags() {
-        return tags;
-    }
-
-    public void setTags(String tags) {
-        this.tags = tags;
-    }
-
-    public int getPageNum() {
-        return pageNum;
-    }
-
-    public void setPageNum(int pageNum) {
-        this.pageNum = pageNum;
-    }
-
-    public int getSortOption() {
-        return sortOption;
-    }
-
-    public void setSortOption(int sortOption) {
-        this.sortOption = sortOption;
-    }
-
-    /**
-     * Copies data from this bean to media file filter object.
-     *
-     */
-    public void copyTo(MediaFileFilter dataHolder) {
-        dataHolder.setName(this.name);
-
-        if (!StringUtils.isEmpty(this.type)) {
-            MediaFileType filterType = null;
-            if ("mediaFileView.audio".equals(this.type)) {
-                filterType = MediaFileType.AUDIO;
-            } else if ("mediaFileView.video".equals(this.type)) {
-                filterType = MediaFileType.VIDEO;
-            } else if ("mediaFileView.image".equals(this.type)) {
-                filterType = MediaFileType.IMAGE;
-            } else if ("mediaFileView.others".equals(this.type)) {
-                filterType = MediaFileType.OTHERS;
-            } 
-
-            dataHolder.setType(filterType);
-        }
-
-        if (this.size > 0) {
-            SizeFilterType sftype = SizeFilterType.EQ;
-            if ("mediaFileView.gt".equals(this.sizeFilterType)) {
-                sftype = SizeFilterType.GT;
-            } else if ("mediaFileView.ge".equals(this.sizeFilterType)) {
-                sftype = SizeFilterType.GTE;
-            } else if ("mediaFileView.eq".equals(this.sizeFilterType)) {
-                sftype = SizeFilterType.EQ;
-            } else if ("mediaFileView.le".equals(this.sizeFilterType)) {
-                sftype = SizeFilterType.LTE;
-            } else if ("mediaFileView.lt".equals(this.sizeFilterType)) {
-                sftype = SizeFilterType.LT;
-            }
-            dataHolder.setSizeFilterType(sftype);
-
-            long filterSize = this.size ;
-            if ("mediaFileView.kb".equals(this.sizeUnit)) {
-                filterSize = this.size * RollerConstants.ONE_KB_IN_BYTES;
-            } else if ("mediaFileView.mb".equals(this.sizeUnit)) {
-                    filterSize = this.size * RollerConstants.ONE_MB_IN_BYTES;
-            }
-            dataHolder.setSize(filterSize);
-        }
-
-        if (!StringUtils.isEmpty(this.tags)) {
-            dataHolder.setTags(Arrays.asList(this.tags.split(" ")));
-        }
-
-        dataHolder.setStartIndex(pageNum * PAGE_SIZE);
-
-        // set length to fetch to one more than what is required.
-        // this would help us determine whether there are more pages
-        dataHolder.setLength(PAGE_SIZE + 1);
-
-        MediaFileOrder order;
-        switch (this.sortOption) {
-            case 0:
-                order = MediaFileOrder.NAME;
-                break;
-            case 1:
-                order = MediaFileOrder.DATE_UPLOADED;
-                break;
-            case 2:
-                order = MediaFileOrder.TYPE;
-                break;
-            default:
-                order = null;
-        }
-        dataHolder.setOrder(order);
-    }
-}
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ *  contributor license agreements.  The ASF licenses this file to You
+ * under the Apache License, Version 2.0 (the "License"); you may not
+ * use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.  For additional information regarding
+ * copyright in this work, please see the NOTICE file in the top level
+ * directory of this distribution.
+ */
+package org.apache.roller.weblogger.ui.struts2.editor;
+
+import java.util.Arrays;
+
+import java.util.ResourceBundle;
+import org.apache.commons.lang3.StringUtils;
+import org.apache.roller.util.RollerConstants;
+import org.apache.roller.weblogger.pojos.MediaFileFilter;
+import org.apache.roller.weblogger.pojos.MediaFileType;
+import org.apache.roller.weblogger.pojos.MediaFileFilter.MediaFileOrder;
+import org.apache.roller.weblogger.pojos.MediaFileFilter.SizeFilterType;
+
+/**
+ * Bean for holding media file search criteria.
+ */
+public class MediaFileSearchBean {
+    private transient ResourceBundle bundle =
+            ResourceBundle.getBundle("ApplicationResources");
+
+    public static final int PAGE_SIZE = 10;
+
+    // Media file name as search criteria
+    private String name;
+
+    // Media file type as search criteria
+    private String type;
+
+    // Type of size filter as search criteria
+    private String sizeFilterType;
+
+    // Size of file as search criteria
+    private long size;
+
+    // Size unit
+    private String sizeUnit;
+
+    // Tags as search criteria
+    private String tags;
+
+    // Page number of results
+    private int pageNum = 0;
+    
+    // Sort option for search results
+    private int sortOption;
+
+    public String getName() {
+        return name;
+    }
+
+    public void setName(String name) {
+        this.name = name;
+    }
+
+    public String getType() {
+        return type;
+    }
+
+    public String getTypeLabel() {
+        return this.bundle.getString(type);
+    }
+
+    public void setType(String type) {
+        this.type = type;
+    }
+
+    public String getSizeFilterType() {
+        return sizeFilterType;
+    }
+
+    public void setSizeFilterType(String sizeFilterType) {
+        this.sizeFilterType = sizeFilterType;
+    }
+
+    public String getSizeFilterTypeLabel() {
+        return this.bundle.getString(sizeFilterType);
+    }
+
+    public long getSize() {
+        return size;
+    }
+
+    public void setSize(long size) {
+        this.size = size;
+    }
+
+    public String getSizeUnit() {
+        return sizeUnit;
+    }
+
+    public String getSizeUnitLabel() {
+        return this.bundle.getString(sizeUnit);
+    }
+
+    public void setSizeUnit(String sizeUnit) {
+        this.sizeUnit = sizeUnit;
+    }
+
+    public String getTags() {
+        return tags;
+    }
+
+    public void setTags(String tags) {
+        this.tags = tags;
+    }
+
+    public int getPageNum() {
+        return pageNum;
+    }
+
+    public void setPageNum(int pageNum) {
+        this.pageNum = pageNum;
+    }
+
+    public int getSortOption() {
+        return sortOption;
+    }
+
+    public void setSortOption(int sortOption) {
+        this.sortOption = sortOption;
+    }
+
+    /**
+     * Copies data from this bean to media file filter object.
+     *
+     */
+    public void copyTo(MediaFileFilter dataHolder) {
+        dataHolder.setName(this.name);
+
+        if (!StringUtils.isEmpty(this.type)) {
+            MediaFileType filterType = null;
+            if ("mediaFileView.audio".equals(this.type)) {
+                filterType = MediaFileType.AUDIO;
+            } else if ("mediaFileView.video".equals(this.type)) {
+                filterType = MediaFileType.VIDEO;
+            } else if ("mediaFileView.image".equals(this.type)) {
+                filterType = MediaFileType.IMAGE;
+            } else if ("mediaFileView.others".equals(this.type)) {
+                filterType = MediaFileType.OTHERS;
+            } 
+
+            dataHolder.setType(filterType);
+        }
+
+        if (this.size > 0) {
+            SizeFilterType sftype = SizeFilterType.EQ;
+            if ("mediaFileView.gt".equals(this.sizeFilterType)) {
+                sftype = SizeFilterType.GT;
+            } else if ("mediaFileView.ge".equals(this.sizeFilterType)) {
+                sftype = SizeFilterType.GTE;
+            } else if ("mediaFileView.eq".equals(this.sizeFilterType)) {
+                sftype = SizeFilterType.EQ;
+            } else if ("mediaFileView.le".equals(this.sizeFilterType)) {
+                sftype = SizeFilterType.LTE;
+            } else if ("mediaFileView.lt".equals(this.sizeFilterType)) {
+                sftype = SizeFilterType.LT;
+            }
+            dataHolder.setSizeFilterType(sftype);
+
+            long filterSize = this.size ;
+            if ("mediaFileView.kb".equals(this.sizeUnit)) {
+                filterSize = this.size * RollerConstants.ONE_KB_IN_BYTES;
+            } else if ("mediaFileView.mb".equals(this.sizeUnit)) {
+                    filterSize = this.size * RollerConstants.ONE_MB_IN_BYTES;
+            }
+            dataHolder.setSize(filterSize);
+        }
+
+        if (!StringUtils.isEmpty(this.tags)) {
+            dataHolder.setTags(Arrays.asList(this.tags.split(" ")));
+        }
+
+        dataHolder.setStartIndex(pageNum * PAGE_SIZE);
+
+        dataHolder.setLength(PAGE_SIZE);
+
+        MediaFileOrder order;
+        switch (this.sortOption) {
+            case 0:
+                order = MediaFileOrder.NAME;
+                break;
+            case 1:
+                order = MediaFileOrder.DATE_UPLOADED;
+                break;
+            case 2:
+                order = MediaFileOrder.TYPE;
+                break;
+            default:
+                order = null;
+        }
+        dataHolder.setOrder(order);
+    }
+}
diff --git a/app/src/main/java/org/apache/roller/weblogger/ui/struts2/editor/MediaFileView.java b/app/src/main/java/org/apache/roller/weblogger/ui/struts2/editor/MediaFileView.java
index e52a7b2431..fdf8ba498d 100644
--- a/app/src/main/java/org/apache/roller/weblogger/ui/struts2/editor/MediaFileView.java
+++ b/app/src/main/java/org/apache/roller/weblogger/ui/struts2/editor/MediaFileView.java
@@ -1,491 +1,487 @@
-/*
- * Licensed to the Apache Software Foundation (ASF) under one or more
- *  contributor license agreements.  The ASF licenses this file to You
- * under the Apache License, Version 2.0 (the "License"); you may not
- * use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *     http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.  For additional information regarding
- * copyright in this work, please see the NOTICE file in the top level
- * directory of this distribution.
- */
-package org.apache.roller.weblogger.ui.struts2.editor;
-
-import java.util.ArrayList;
-import java.util.Arrays;
-import java.util.List;
-
-import org.apache.commons.lang3.StringUtils;
-import org.apache.commons.logging.Log;
-import org.apache.commons.logging.LogFactory;
-import org.apache.roller.weblogger.WebloggerException;
-import org.apache.roller.weblogger.business.FileIOException;
-import org.apache.roller.weblogger.business.MediaFileManager;
-import org.apache.roller.weblogger.business.WebloggerFactory;
-import org.apache.roller.weblogger.pojos.MediaFile;
-import org.apache.roller.weblogger.pojos.MediaFileComparator;
-import org.apache.roller.weblogger.pojos.MediaFileComparator.MediaFileComparatorType;
-import org.apache.roller.weblogger.pojos.MediaFileDirectory;
-import org.apache.roller.weblogger.pojos.MediaFileFilter;
-import org.apache.roller.weblogger.ui.struts2.pagers.MediaFilePager;
-import org.apache.roller.weblogger.ui.struts2.util.KeyValueObject;
-import org.apache.roller.weblogger.util.cache.CacheManager;
-import org.apache.struts2.convention.annotation.AllowedMethods;
-import org.apache.struts2.interceptor.parameter.StrutsParameter;
-import org.apache.struts2.interceptor.validation.SkipValidation;
-
-/**
- * View media files.
- */
-@SuppressWarnings("serial")
-// TODO: make this work @AllowedMethods({"execute","view","search","delete","deleteSelected","deleteFolder","includeInGallery","moveSelected"})
-public class MediaFileView extends MediaFileBase {
-
-    private static Log log = LogFactory.getLog(MediaFileView.class);
-
-    private String directoryId;
-    private String directoryName;
-    private String sortBy;
-    private String newDirectoryName;
-
-    private List<MediaFile> childFiles;
-    private MediaFileDirectory currentDirectory;
-
-    // Search criteria - drop-down for file type
-    private static List<KeyValueObject> FILE_TYPES = null;
-
-    // Search criteria - drop-down for size filter
-    private static List<KeyValueObject> SIZE_FILTER_TYPES = null;
-
-    // Search criteria - drop-down for size unit
-    private static List<KeyValueObject> SIZE_UNITS = null;
-
-    // Sort options for search results.
-    private static List<KeyValueObject> SORT_OPTIONS = null;
-
-    // Pager for displaying search results.
-    private MediaFilePager pager;
-
-    // Path of new directory to be created.
-    private String newDirectoryPath;
-
-    // a new directory the user wishes to view
-    private String viewDirectoryId = null;
-
-    private MediaFileSearchBean bean = new MediaFileSearchBean();
-
-    public MediaFileView() {
-        this.actionName = "mediaFileView";
-        this.desiredMenu = "editor";
-        this.pageTitle = "mediaFileView.title";
-    }
-
-    /**
-     * Prepares view action
-     */
-    @Override
-    public void myPrepare() {
-
-        if (SIZE_FILTER_TYPES == null) {
-
-            SIZE_FILTER_TYPES = Arrays.asList(new KeyValueObject(
-                    "mediaFileView.gt", getText("mediaFileView.gt")),
-                    new KeyValueObject("mediaFileView.ge",
-                            getText("mediaFileView.ge")), new KeyValueObject(
-                            "mediaFileView.eq", getText("mediaFileView.eq")),
-                    new KeyValueObject("mediaFileView.le",
-                            getText("mediaFileView.le")), new KeyValueObject(
-                            "mediaFileView.lt", getText("mediaFileView.lt")));
-
-            FILE_TYPES = Arrays.asList(new KeyValueObject("mediaFileView.any",
-                    getText("mediaFileView.any")), new KeyValueObject(
-                    "mediaFileView.others", getText("mediaFileView.others")),
-                    new KeyValueObject("mediaFileView.image",
-                            getText("mediaFileView.image")),
-                    new KeyValueObject("mediaFileView.video",
-                            getText("mediaFileView.video")),
-                    new KeyValueObject("mediaFileView.audio",
-                            getText("mediaFileView.audio")));
-
-            SIZE_UNITS = Arrays.asList(new KeyValueObject(
-                    "mediaFileView.bytes", getText("mediaFileView.bytes")),
-                    new KeyValueObject("mediaFileView.kb",
-                            getText("mediaFileView.kb")), new KeyValueObject(
-                            "mediaFileView.mb", getText("mediaFileView.mb")));
-
-            SORT_OPTIONS = Arrays.asList(new KeyValueObject("name",
-                    getText("generic.name")), new KeyValueObject(
-                    "date_uploaded", getText("mediaFileView.date")),
-                    new KeyValueObject("type", getText("mediaFileView.type")));
-        }
-
-        refreshAllDirectories();
-    }
-
-    /**
-     * Create a new directory by name. All folders placed at the root.
-     */
-    public String createNewDirectory() {
-        boolean dirCreated = false;
-        if (StringUtils.isEmpty(this.newDirectoryName)) {
-            addError("mediaFile.error.view.dirNameEmpty");
-        } else if (this.newDirectoryName.contains("/")) {
-            addError("mediaFile.error.view.dirNameInvalid");
-        } else {
-            try {
-
-                log.debug("Creating new directory - " + this.newDirectoryName);
-                MediaFileManager manager = WebloggerFactory.getWeblogger()
-                        .getMediaFileManager();
-
-                if (!getActionWeblog().hasMediaFileDirectory(
-                        this.newDirectoryName)) {
-                    // Create
-                    MediaFileDirectory dir = manager.createMediaFileDirectory(
-                            getActionWeblog(), this.newDirectoryName);
-                    // flush changes
-                    WebloggerFactory.getWeblogger().flush();
-                    addMessage("mediaFile.directoryCreate.success",
-                            this.newDirectoryName);
-
-                    // Switch to folder
-                    setDirectoryId(dir.getId());
-
-                    dirCreated = true;
-                } else {
-                    // already exists
-                    addMessage("mediaFile.directoryCreate.error.exists",
-                            this.newDirectoryName);
-                }
-
-            } catch (WebloggerException e) {
-                log.error("Error creating new directory", e);
-                addError("Error creating new directory");
-            }
-        }
-
-        if (dirCreated) {
-            // Refresh list of directories so the newly created directory is
-            // included.
-            refreshAllDirectories();
-        }
-        return execute();
-
-    }
-
-    /**
-     * Returns directory content in JSON format.
-     */
-    public String fetchDirectoryContentLight() {
-        execute();
-        return "success.json";
-    }
-
-    /**
-     * Fetches and displays list of media file for the given directory. The
-     * directory could be chosen by ID or path.
-     * 
-     * @return String The result of the action.
-     */
-    @SkipValidation
-    @Override
-    public String execute() {
-        MediaFileManager manager = WebloggerFactory.getWeblogger().getMediaFileManager();
-        try {
-            MediaFileDirectory directory;
-            if (StringUtils.isNotEmpty(this.directoryId)) {
-                directory = manager.getMediaFileDirectory(getActionWeblog(), this.directoryId);
-
-            } else if (StringUtils.isNotEmpty(this.directoryName)) {
-                directory = manager.getMediaFileDirectoryByName(
-                        getActionWeblog(), this.directoryName);
-
-            } else {
-                directory = manager.getDefaultMediaFileDirectory(getActionWeblog());
-            }
-            this.directoryId = directory.getId();
-            this.directoryName = directory.getName();
-
-            this.childFiles = new ArrayList<>();
-            this.childFiles.addAll(directory.getMediaFiles());
-
-            if ("type".equals(sortBy)) {
-                this.childFiles.sort(new MediaFileComparator(MediaFileComparatorType.TYPE));
-
-            } else if ("date_uploaded".equals(sortBy)) {
-                this.childFiles.sort(new MediaFileComparator(MediaFileComparatorType.DATE_UPLOADED));
-
-            } else {
-                // default to sort by name
-                sortBy = "name";
-                this.childFiles.sort(new MediaFileComparator(MediaFileComparatorType.NAME));
-            }
-
-            this.currentDirectory = directory;
-
-            // set current directory if valid
-            if (directory != null) {
-                setViewDirectoryId(directory.getId());
-            }
-
-            return SUCCESS;
-
-        } catch (FileIOException ex) {
-            log.error("Error viewing media file directory ", ex);
-            addError("MediaFile.error.view");
-        } catch (Exception e) {
-            log.error("Error viewing media file directory ", e);
-            addError("MediaFile.error.view");
-        }
-        return SUCCESS;
-    }
-
-    /**
-     * View the contents of another Media folder.
-     */
-    public String view() {
-        try {
-            MediaFileManager manager = WebloggerFactory.getWeblogger()
-                    .getMediaFileManager();
-            if (!StringUtils.isEmpty(viewDirectoryId)) {
-                setDirectoryId(viewDirectoryId);
-                setCurrentDirectory(manager.getMediaFileDirectory(getActionWeblog(), viewDirectoryId));
-            }
-        } catch (WebloggerException ex) {
-            log.error("Error looking up directory", ex);
-        }
-        return execute();
-    }
-
-    /**
-     * Save a media file.
-     * 
-     * @return String The result of the action.
-     */
-    public String search() {
-
-        boolean valSuccess = myValidate();
-
-        if (valSuccess) {
-            MediaFileFilter filter = new MediaFileFilter();
-            bean.copyTo(filter);
-            MediaFileManager manager = WebloggerFactory.getWeblogger().getMediaFileManager();
-            try {
-                List<MediaFile> rawResults = manager.searchMediaFiles(getActionWeblog(), filter);
-                boolean hasMore = false;
-                List<MediaFile> results = new ArrayList<>(rawResults);
-                if (results.size() > MediaFileSearchBean.PAGE_SIZE) {
-                    results.remove(results.size() - 1);
-                    hasMore = true;
-                }
-
-                this.pager = new MediaFilePager(bean.getPageNum(), results,
-                        hasMore);
-
-            } catch (Exception e) {
-                log.error("Error applying search criteria", e);
-                addError("Error applying search criteria - check Roller logs");
-            }
-
-        }
-
-        return SUCCESS;
-    }
-
-    /**
-     * Delete selected media files.
-     * 
-     */
-    public String deleteSelected() {
-        doDeleteSelected();
-        return execute();
-    }
-
-    /**
-     * Delete selected media file
-     * 
-     */
-    public String delete() {
-        doDeleteMediaFile();
-        return execute();
-    }
-
-    /**
-     * Delete folder
-     */
-    public String deleteFolder() {
-
-        try {
-            MediaFileManager manager = WebloggerFactory.getWeblogger().getMediaFileManager();
-            if (directoryId != null) {
-                log.debug("Deleting media file folder - " + directoryId + " ("
-                        + directoryName + ")");
-                MediaFileDirectory mediaFileDir = manager
-                        .getMediaFileDirectory(getActionWeblog(), directoryId);
-                manager.removeMediaFileDirectory(mediaFileDir);
-                refreshAllDirectories();
-                WebloggerFactory.getWeblogger().getWeblogManager()
-                        .saveWeblog(this.getActionWeblog());
-
-                // flush changes
-                WebloggerFactory.getWeblogger().flush();
-                WebloggerFactory.getWeblogger().release();
-                addMessage("mediaFile.deleteFolder.success");
-
-                // notify caches
-                CacheManager.invalidate(getActionWeblog());
-
-                // re-route to default folder
-                mediaFileDir = manager
-                        .getDefaultMediaFileDirectory(getActionWeblog());
-                setDirectoryId(mediaFileDir.getId());
-                setDirectoryName(mediaFileDir.getName());
-            } else {
-                log.error("(System error) No directory ID provided for media file folder delete.");
-            }
-        } catch (WebloggerException ex) {
-            log.error("Error deleting folder", ex);
-        }
-        return execute();
-    }
-
-    /**
-     * Include selected media file in gallery
-     * 
-     */
-    public String includeInGallery() {
-        doIncludeMediaFileInGallery();
-        return execute();
-    }
-
-    /**
-     * Move selected media files to a different directory
-     * 
-     */
-    public String moveSelected() {
-        doMoveSelected();
-        return execute();
-    }
-
-    public String getDirectoryId() {
-        return directoryId;
-    }
-
-    @StrutsParameter
-    public void setDirectoryId(String id) {
-        this.directoryId = id;
-    }
-
-    public List<MediaFile> getChildFiles() {
-        return childFiles;
-    }
-
-    public void setChildFiles(List<MediaFile> files) {
-        this.childFiles = files;
-    }
-
-    public String getNewDirectoryName() {
-        return newDirectoryName;
-    }
-
-    @StrutsParameter
-    public void setNewDirectoryName(String newDirectoryName) {
-        this.newDirectoryName = newDirectoryName;
-    }
-
-    public MediaFileDirectory getCurrentDirectory() {
-        return currentDirectory;
-    }
-
-    public void setCurrentDirectory(MediaFileDirectory currentDirectory) {
-        this.currentDirectory = currentDirectory;
-    }
-
-    public String getDirectoryName() {
-        return directoryName;
-    }
-
-    @StrutsParameter
-    public void setDirectoryName(String path) {
-        this.directoryName = path;
-    }
-
-    public String getSortBy() {
-        return sortBy;
-    }
-
-    @StrutsParameter
-    public void setSortBy(String sortBy) {
-        this.sortBy = sortBy;
-    }
-
-    /**
-     * Validates search input
-     */
-    public boolean myValidate() {
-        if (StringUtils.isEmpty(bean.getName())
-                && StringUtils.isEmpty(bean.getTags())
-                && StringUtils.isEmpty(bean.getType()) && bean.getSize() == 0) {
-            addError("MediaFile.error.search.empty");
-            return false;
-        }
-        return true;
-    }
-
-    @StrutsParameter(depth = 1)
-    public MediaFileSearchBean getBean() {
-        return bean;
-    }
-
-    public void setBean(MediaFileSearchBean b) {
-        this.bean = b;
-    }
-
-    public List<KeyValueObject> getFileTypes() {
-        return FILE_TYPES;
-    }
-
-    public List<KeyValueObject> getSizeFilterTypes() {
-        return SIZE_FILTER_TYPES;
-    }
-
-    public List<KeyValueObject> getSizeUnits() {
-        return SIZE_UNITS;
-    }
-
-    public List<KeyValueObject> getSortOptions() {
-        return SORT_OPTIONS;
-    }
-
-    public MediaFilePager getPager() {
-        return pager;
-    }
-
-    public void setPager(MediaFilePager pager) {
-        this.pager = pager;
-    }
-
-    public String getNewDirectoryPath() {
-        return newDirectoryPath;
-    }
-
-    public void setNewDirectoryPath(String newDirectoryPath) {
-        this.newDirectoryPath = newDirectoryPath;
-    }
-
-    public String getViewDirectoryId() {
-        return viewDirectoryId;
-    }
-
-    @StrutsParameter
-    public void setViewDirectoryId(String viewDirectoryId) {
-        this.viewDirectoryId = viewDirectoryId;
-    }
-
-}
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ *  contributor license agreements.  The ASF licenses this file to You
+ * under the Apache License, Version 2.0 (the "License"); you may not
+ * use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.  For additional information regarding
+ * copyright in this work, please see the NOTICE file in the top level
+ * directory of this distribution.
+ */
+package org.apache.roller.weblogger.ui.struts2.editor;
+
+import java.util.ArrayList;
+import java.util.Arrays;
+import java.util.List;
+
+import org.apache.commons.lang3.StringUtils;
+import org.apache.commons.logging.Log;
+import org.apache.commons.logging.LogFactory;
+import org.apache.roller.weblogger.WebloggerException;
+import org.apache.roller.weblogger.business.FileIOException;
+import org.apache.roller.weblogger.business.MediaFileManager;
+import org.apache.roller.weblogger.business.WebloggerFactory;
+import org.apache.roller.weblogger.pojos.MediaFile;
+import org.apache.roller.weblogger.pojos.MediaFileComparator;
+import org.apache.roller.weblogger.pojos.MediaFileComparator.MediaFileComparatorType;
+import org.apache.roller.weblogger.pojos.MediaFileDirectory;
+import org.apache.roller.weblogger.pojos.MediaFileFilter;
+import org.apache.roller.weblogger.ui.struts2.pagers.MediaFilePager;
+import org.apache.roller.weblogger.ui.struts2.util.KeyValueObject;
+import org.apache.roller.weblogger.util.cache.CacheManager;
+import org.apache.struts2.convention.annotation.AllowedMethods;
+import org.apache.struts2.interceptor.parameter.StrutsParameter;
+import org.apache.struts2.interceptor.validation.SkipValidation;
+
+/**
+ * View media files.
+ */
+@SuppressWarnings("serial")
+// TODO: make this work @AllowedMethods({"execute","view","search","delete","deleteSelected","deleteFolder","includeInGallery","moveSelected"})
+public class MediaFileView extends MediaFileBase {
+
+    private static Log log = LogFactory.getLog(MediaFileView.class);
+
+    private String directoryId;
+    private String directoryName;
+    private String sortBy;
+    private String newDirectoryName;
+
+    private List<MediaFile> childFiles;
+    private MediaFileDirectory currentDirectory;
+
+    // Search criteria - drop-down for file type
+    private static List<KeyValueObject> FILE_TYPES = null;
+
+    // Search criteria - drop-down for size filter
+    private static List<KeyValueObject> SIZE_FILTER_TYPES = null;
+
+    // Search criteria - drop-down for size unit
+    private static List<KeyValueObject> SIZE_UNITS = null;
+
+    // Sort options for search results.
+    private static List<KeyValueObject> SORT_OPTIONS = null;
+
+    // Pager for displaying search results.
+    private MediaFilePager pager;
+
+    // Path of new directory to be created.
+    private String newDirectoryPath;
+
+    // a new directory the user wishes to view
+    private String viewDirectoryId = null;
+
+    private MediaFileSearchBean bean = new MediaFileSearchBean();
+
+    public MediaFileView() {
+        this.actionName = "mediaFileView";
+        this.desiredMenu = "editor";
+        this.pageTitle = "mediaFileView.title";
+    }
+
+    /**
+     * Prepares view action
+     */
+    @Override
+    public void myPrepare() {
+
+        if (SIZE_FILTER_TYPES == null) {
+
+            SIZE_FILTER_TYPES = Arrays.asList(new KeyValueObject(
+                    "mediaFileView.gt", getText("mediaFileView.gt")),
+                    new KeyValueObject("mediaFileView.ge",
+                            getText("mediaFileView.ge")), new KeyValueObject(
+                            "mediaFileView.eq", getText("mediaFileView.eq")),
+                    new KeyValueObject("mediaFileView.le",
+                            getText("mediaFileView.le")), new KeyValueObject(
+                            "mediaFileView.lt", getText("mediaFileView.lt")));
+
+            FILE_TYPES = Arrays.asList(new KeyValueObject("mediaFileView.any",
+                    getText("mediaFileView.any")), new KeyValueObject(
+                    "mediaFileView.others", getText("mediaFileView.others")),
+                    new KeyValueObject("mediaFileView.image",
+                            getText("mediaFileView.image")),
+                    new KeyValueObject("mediaFileView.video",
+                            getText("mediaFileView.video")),
+                    new KeyValueObject("mediaFileView.audio",
+                            getText("mediaFileView.audio")));
+
+            SIZE_UNITS = Arrays.asList(new KeyValueObject(
+                    "mediaFileView.bytes", getText("mediaFileView.bytes")),
+                    new KeyValueObject("mediaFileView.kb",
+                            getText("mediaFileView.kb")), new KeyValueObject(
+                            "mediaFileView.mb", getText("mediaFileView.mb")));
+
+            SORT_OPTIONS = Arrays.asList(new KeyValueObject("name",
+                    getText("generic.name")), new KeyValueObject(
+                    "date_uploaded", getText("mediaFileView.date")),
+                    new KeyValueObject("type", getText("mediaFileView.type")));
+        }
+
+        refreshAllDirectories();
+    }
+
+    /**
+     * Create a new directory by name. All folders placed at the root.
+     */
+    public String createNewDirectory() {
+        boolean dirCreated = false;
+        if (StringUtils.isEmpty(this.newDirectoryName)) {
+            addError("mediaFile.error.view.dirNameEmpty");
+        } else if (this.newDirectoryName.contains("/")) {
+            addError("mediaFile.error.view.dirNameInvalid");
+        } else {
+            try {
+
+                log.debug("Creating new directory - " + this.newDirectoryName);
+                MediaFileManager manager = WebloggerFactory.getWeblogger()
+                        .getMediaFileManager();
+
+                if (!getActionWeblog().hasMediaFileDirectory(
+                        this.newDirectoryName)) {
+                    // Create
+                    MediaFileDirectory dir = manager.createMediaFileDirectory(
+                            getActionWeblog(), this.newDirectoryName);
+                    // flush changes
+                    WebloggerFactory.getWeblogger().flush();
+                    addMessage("mediaFile.directoryCreate.success",
+                            this.newDirectoryName);
+
+                    // Switch to folder
+                    setDirectoryId(dir.getId());
+
+                    dirCreated = true;
+                } else {
+                    // already exists
+                    addMessage("mediaFile.directoryCreate.error.exists",
+                            this.newDirectoryName);
+                }
+
+            } catch (WebloggerException e) {
+                log.error("Error creating new directory", e);
+                addError("Error creating new directory");
+            }
+        }
+
+        if (dirCreated) {
+            // Refresh list of directories so the newly created directory is
+            // included.
+            refreshAllDirectories();
+        }
+        return execute();
+
+    }
+
+    /**
+     * Returns directory content in JSON format.
+     */
+    public String fetchDirectoryContentLight() {
+        execute();
+        return "success.json";
+    }
+
+    /**
+     * Fetches and displays list of media file for the given directory. The
+     * directory could be chosen by ID or path.
+     * 
+     * @return String The result of the action.
+     */
+    @SkipValidation
+    @Override
+    public String execute() {
+        MediaFileManager manager = WebloggerFactory.getWeblogger().getMediaFileManager();
+        try {
+            MediaFileDirectory directory;
+            if (StringUtils.isNotEmpty(this.directoryId)) {
+                directory = manager.getMediaFileDirectory(getActionWeblog(), this.directoryId);
+
+            } else if (StringUtils.isNotEmpty(this.directoryName)) {
+                directory = manager.getMediaFileDirectoryByName(
+                        getActionWeblog(), this.directoryName);
+
+            } else {
+                directory = manager.getDefaultMediaFileDirectory(getActionWeblog());
+            }
+            this.directoryId = directory.getId();
+            this.directoryName = directory.getName();
+
+            this.childFiles = new ArrayList<>();
+            this.childFiles.addAll(directory.getMediaFiles());
+
+            if ("type".equals(sortBy)) {
+                this.childFiles.sort(new MediaFileComparator(MediaFileComparatorType.TYPE));
+
+            } else if ("date_uploaded".equals(sortBy)) {
+                this.childFiles.sort(new MediaFileComparator(MediaFileComparatorType.DATE_UPLOADED));
+
+            } else {
+                // default to sort by name
+                sortBy = "name";
+                this.childFiles.sort(new MediaFileComparator(MediaFileComparatorType.NAME));
+            }
+
+            this.currentDirectory = directory;
+
+            // set current directory if valid
+            if (directory != null) {
+                setViewDirectoryId(directory.getId());
+            }
+
+            return SUCCESS;
+
+        } catch (FileIOException ex) {
+            log.error("Error viewing media file directory ", ex);
+            addError("MediaFile.error.view");
+        } catch (Exception e) {
+            log.error("Error viewing media file directory ", e);
+            addError("MediaFile.error.view");
+        }
+        return SUCCESS;
+    }
+
+    /**
+     * View the contents of another Media folder.
+     */
+    public String view() {
+        try {
+            MediaFileManager manager = WebloggerFactory.getWeblogger()
+                    .getMediaFileManager();
+            if (!StringUtils.isEmpty(viewDirectoryId)) {
+                setDirectoryId(viewDirectoryId);
+                setCurrentDirectory(manager.getMediaFileDirectory(getActionWeblog(), viewDirectoryId));
+            }
+        } catch (WebloggerException ex) {
+            log.error("Error looking up directory", ex);
+        }
+        return execute();
+    }
+
+    /**
+     * Save a media file.
+     * 
+     * @return String The result of the action.
+     */
+    public String search() {
+
+        boolean valSuccess = myValidate();
+
+        if (valSuccess) {
+            MediaFileFilter filter = new MediaFileFilter();
+            bean.copyTo(filter);
+            MediaFileManager manager = WebloggerFactory.getWeblogger().getMediaFileManager();
+            try {
+                List<MediaFile> results = new ArrayList<>(
+                        manager.searchMediaFiles(getActionWeblog(), filter));
+                long totalCount = manager.getMediaFileCount(getActionWeblog(), filter);
+
+                this.pager = new MediaFilePager(bean.getPageNum(), results,
+                        totalCount, MediaFileSearchBean.PAGE_SIZE);
+
+            } catch (Exception e) {
+                log.error("Error applying search criteria", e);
+                addError("Error applying search criteria - check Roller logs");
+            }
+
+        }
+
+        return SUCCESS;
+    }
+
+    /**
+     * Delete selected media files.
+     * 
+     */
+    public String deleteSelected() {
+        doDeleteSelected();
+        return execute();
+    }
+
+    /**
+     * Delete selected media file
+     * 
+     */
+    public String delete() {
+        doDeleteMediaFile();
+        return execute();
+    }
+
+    /**
+     * Delete folder
+     */
+    public String deleteFolder() {
+
+        try {
+            MediaFileManager manager = WebloggerFactory.getWeblogger().getMediaFileManager();
+            if (directoryId != null) {
+                log.debug("Deleting media file folder - " + directoryId + " ("
+                        + directoryName + ")");
+                MediaFileDirectory mediaFileDir = manager
+                        .getMediaFileDirectory(getActionWeblog(), directoryId);
+                manager.removeMediaFileDirectory(mediaFileDir);
+                refreshAllDirectories();
+                WebloggerFactory.getWeblogger().getWeblogManager()
+                        .saveWeblog(this.getActionWeblog());
+
+                // flush changes
+                WebloggerFactory.getWeblogger().flush();
+                WebloggerFactory.getWeblogger().release();
+                addMessage("mediaFile.deleteFolder.success");
+
+                // notify caches
+                CacheManager.invalidate(getActionWeblog());
+
+                // re-route to default folder
+                mediaFileDir = manager
+                        .getDefaultMediaFileDirectory(getActionWeblog());
+                setDirectoryId(mediaFileDir.getId());
+                setDirectoryName(mediaFileDir.getName());
+            } else {
+                log.error("(System error) No directory ID provided for media file folder delete.");
+            }
+        } catch (WebloggerException ex) {
+            log.error("Error deleting folder", ex);
+        }
+        return execute();
+    }
+
+    /**
+     * Include selected media file in gallery
+     * 
+     */
+    public String includeInGallery() {
+        doIncludeMediaFileInGallery();
+        return execute();
+    }
+
+    /**
+     * Move selected media files to a different directory
+     * 
+     */
+    public String moveSelected() {
+        doMoveSelected();
+        return execute();
+    }
+
+    public String getDirectoryId() {
+        return directoryId;
+    }
+
+    @StrutsParameter
+    public void setDirectoryId(String id) {
+        this.directoryId = id;
+    }
+
+    public List<MediaFile> getChildFiles() {
+        return childFiles;
+    }
+
+    public void setChildFiles(List<MediaFile> files) {
+        this.childFiles = files;
+    }
+
+    public String getNewDirectoryName() {
+        return newDirectoryName;
+    }
+
+    @StrutsParameter
+    public void setNewDirectoryName(String newDirectoryName) {
+        this.newDirectoryName = newDirectoryName;
+    }
+
+    public MediaFileDirectory getCurrentDirectory() {
+        return currentDirectory;
+    }
+
+    public void setCurrentDirectory(MediaFileDirectory currentDirectory) {
+        this.currentDirectory = currentDirectory;
+    }
+
+    public String getDirectoryName() {
+        return directoryName;
+    }
+
+    @StrutsParameter
+    public void setDirectoryName(String path) {
+        this.directoryName = path;
+    }
+
+    public String getSortBy() {
+        return sortBy;
+    }
+
+    @StrutsParameter
+    public void setSortBy(String sortBy) {
+        this.sortBy = sortBy;
+    }
+
+    /**
+     * Validates search input
+     */
+    public boolean myValidate() {
+        if (StringUtils.isEmpty(bean.getName())
+                && StringUtils.isEmpty(bean.getTags())
+                && StringUtils.isEmpty(bean.getType()) && bean.getSize() == 0) {
+            addError("MediaFile.error.search.empty");
+            return false;
+        }
+        return true;
+    }
+
+    @StrutsParameter(depth = 1)
+    public MediaFileSearchBean getBean() {
+        return bean;
+    }
+
+    public void setBean(MediaFileSearchBean b) {
+        this.bean = b;
+    }
+
+    public List<KeyValueObject> getFileTypes() {
+        return FILE_TYPES;
+    }
+
+    public List<KeyValueObject> getSizeFilterTypes() {
+        return SIZE_FILTER_TYPES;
+    }
+
+    public List<KeyValueObject> getSizeUnits() {
+        return SIZE_UNITS;
+    }
+
+    public List<KeyValueObject> getSortOptions() {
+        return SORT_OPTIONS;
+    }
+
+    public MediaFilePager getPager() {
+        return pager;
+    }
+
+    public void setPager(MediaFilePager pager) {
+        this.pager = pager;
+    }
+
+    public String getNewDirectoryPath() {
+        return newDirectoryPath;
+    }
+
+    public void setNewDirectoryPath(String newDirectoryPath) {
+        this.newDirectoryPath = newDirectoryPath;
+    }
+
+    public String getViewDirectoryId() {
+        return viewDirectoryId;
+    }
+
+    @StrutsParameter
+    public void setViewDirectoryId(String viewDirectoryId) {
+        this.viewDirectoryId = viewDirectoryId;
+    }
+
+}
diff --git a/app/src/main/java/org/apache/roller/weblogger/ui/struts2/pagers/CommentsPager.java b/app/src/main/java/org/apache/roller/weblogger/ui/struts2/pagers/CommentsPager.java
index 7f1b0de427..629c3cba46 100644
--- a/app/src/main/java/org/apache/roller/weblogger/ui/struts2/pagers/CommentsPager.java
+++ b/app/src/main/java/org/apache/roller/weblogger/ui/struts2/pagers/CommentsPager.java
@@ -18,79 +18,17 @@
 
 package org.apache.roller.weblogger.ui.struts2.pagers;
 
-import java.util.HashMap;
 import java.util.List;
-import java.util.Map;
 import org.apache.roller.weblogger.pojos.WeblogEntryComment;
-import org.apache.roller.weblogger.util.URLUtilities;
 
 
 /**
  * Paging through a collection of comments.
  */
-public class CommentsPager {
-    
-    // the collection for the pager
-    private final List<WeblogEntryComment> items;
-    
-    // base url for the pager
-    private final String baseUrl;
-    
-    // what page we are on
-    private final int pageNum;
-    
-    // are there more items?
-    private final boolean moreItems;
-    
-    
-    public CommentsPager(String url, int page, List<WeblogEntryComment> comments, boolean hasMore) {
-        this.baseUrl = url;
-        this.pageNum = page;
-        this.items = comments;
-        this.moreItems = hasMore;
-    }
-    
-    
-    public String getNextLink() {
-        if(isMoreItems()) {
-            int nextPage = pageNum + 1;
-            Map<String, String> params = new HashMap<>();
-            params.put("bean.page", ""+nextPage);
-            return createURL(baseUrl, params);
-        }
-        return null;
-    }
-    
-    
-    public String getPrevLink() {
-        if (pageNum > 0) {
-            int prevPage = pageNum - 1;
-            Map<String, String> params = new HashMap<>();
-            params.put("bean.page", ""+prevPage);
-            return createURL(baseUrl, params);
-        }
-        return null;
-    }
-    
-    
-    private String createURL(String base, Map<String, String> params) {
-        String qString = URLUtilities.getQueryString(params);
-        
-        if(base.indexOf('?') != -1) {
-            // if base url already has params them just append our query string
-            return base + "&" + qString.substring(1);
-        } else {
-            return base + qString;
-        }
-    }
-    
-    
-    public List<WeblogEntryComment> getItems() {
-        return items;
-    }
+public class CommentsPager extends LinkPager<WeblogEntryComment> {
 
-    public boolean isMoreItems() {
-        return moreItems;
+    public CommentsPager(String url, int page, List<WeblogEntryComment> comments, long totalCount, int pageSize) {
+        super(url, page, comments, totalCount, pageSize);
     }
-    
+
 }
diff --git a/app/src/main/java/org/apache/roller/weblogger/ui/struts2/pagers/EntriesPager.java b/app/src/main/java/org/apache/roller/weblogger/ui/struts2/pagers/EntriesPager.java
index 58e1768c16..943473a9d7 100644
--- a/app/src/main/java/org/apache/roller/weblogger/ui/struts2/pagers/EntriesPager.java
+++ b/app/src/main/java/org/apache/roller/weblogger/ui/struts2/pagers/EntriesPager.java
@@ -18,79 +18,17 @@
 
 package org.apache.roller.weblogger.ui.struts2.pagers;
 
-import java.util.HashMap;
 import java.util.List;
-import java.util.Map;
 import org.apache.roller.weblogger.pojos.WeblogEntry;
-import org.apache.roller.weblogger.util.URLUtilities;
 
 
 /**
  * Paging through a collection of entries.
  */
-public class EntriesPager {
-    
-    // the collection for the pager
-    private final List<WeblogEntry> items;
-    
-    // base url for the pager
-    private final String baseUrl;
-    
-    // what page we are on
-    private final int pageNum;
-    
-    // are there more items?
-    private final boolean moreItems;
-    
-    
-    public EntriesPager(String url, int page, List<WeblogEntry> entries, boolean hasMore) {
-        this.baseUrl = url;
-        this.pageNum = page;
-        this.items = entries;
-        this.moreItems = hasMore;
-    }
-    
-    
-    public String getNextLink() {
-        if(isMoreItems()) {
-            int nextPage = pageNum + 1;
-            Map<String, String> params = new HashMap<>();
-            params.put("bean.page", ""+nextPage);
-            return createURL(baseUrl, params);
-        }
-        return null;
-    }
-    
-    
-    public String getPrevLink() {
-        if (pageNum > 0) {
-            int prevPage = pageNum - 1;
-            Map<String, String> params = new HashMap<>();
-            params.put("bean.page", ""+prevPage);
-            return createURL(baseUrl, params);
-        }
-        return null;
-    }
-    
-    
-    private String createURL(String base, Map<String, String> params) {
-        String qString = URLUtilities.getQueryString(params);
-        
-        if(base.indexOf('?') != -1) {
-            // if base url already has params them just append our query string
-            return base + "&" + qString.substring(1);
-        } else {
-            return base + qString;
-        }
-    }
-    
-    
-    public List<WeblogEntry> getItems() {
-        return items;
-    }
+public class EntriesPager extends LinkPager<WeblogEntry> {
 
-    public boolean isMoreItems() {
-        return moreItems;
+    public EntriesPager(String url, int page, List<WeblogEntry> entries, long totalCount, int pageSize) {
+        super(url, page, entries, totalCount, pageSize);
     }
-    
+
 }
diff --git a/app/src/main/java/org/apache/roller/weblogger/ui/struts2/pagers/LinkPager.java b/app/src/main/java/org/apache/roller/weblogger/ui/struts2/pagers/LinkPager.java
new file mode 100644
index 0000000000..d064130e53
--- /dev/null
+++ b/app/src/main/java/org/apache/roller/weblogger/ui/struts2/pagers/LinkPager.java
@@ -0,0 +1,110 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ *  contributor license agreements.  The ASF licenses this file to You
+ * under the Apache License, Version 2.0 (the "License"); you may not
+ * use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.  For additional information regarding
+ * copyright in this work, please see the NOTICE file in the top level
+ * directory of this distribution.
+ */
+
+package org.apache.roller.weblogger.ui.struts2.pagers;
+
+import java.util.HashMap;
+import java.util.List;
+import java.util.Map;
+import org.apache.roller.weblogger.util.URLUtilities;
+
+
+/**
+ * Paging through query results with links: first, previous (newer), next (older)
+ * and last page. Page numbers are zero-based in links and one-based for display.
+ */
+public class LinkPager<T> {
+
+    // the collection for the pager
+    private final List<T> items;
+
+    // base url for the pager
+    private final String baseUrl;
+
+    // what page we are on
+    private final int pageNum;
+
+    // rows matching the query across all pages
+    private final long totalCount;
+
+    private final int pageSize;
+
+
+    public LinkPager(String url, int page, List<T> items, long totalCount, int pageSize) {
+        this.baseUrl = url;
+        this.pageNum = page;
+        this.items = items;
+        this.totalCount = totalCount;
+        this.pageSize = pageSize;
+    }
+
+
+    public String getFirstLink() {
+        return pageNum > 0 ? pageLink(0) : null;
+    }
+
+    public String getPrevLink() {
+        return pageNum > 0 ? pageLink(pageNum - 1) : null;
+    }
+
+    public String getNextLink() {
+        return isMoreItems() ? pageLink(pageNum + 1) : null;
+    }
+
+    public String getLastLink() {
+        return isMoreItems() ? pageLink(getPageCount() - 1) : null;
+    }
+
+
+    private String pageLink(int page) {
+        Map<String, String> params = new HashMap<>();
+        params.put("bean.page", ""+page);
+        String qString = URLUtilities.getQueryString(params);
+
+        if(baseUrl.indexOf('?') != -1) {
+            // if base url already has params them just append our query string
+            return baseUrl + "&" + qString.substring(1);
+        } else {
+            return baseUrl + qString;
+        }
+    }
+
+
+    public List<T> getItems() {
+        return items;
+    }
+
+    public boolean isMoreItems() {
+        return (long) (pageNum + 1) * pageSize < totalCount;
+    }
+
+    public long getTotalCount() {
+        return totalCount;
+    }
+
+    /** One-based page number, for display. */
+    public int getPageNumber() {
+        return pageNum + 1;
+    }
+
+    /** Number of pages; at least 1, so an empty result reads "Page 1 of 1". */
+    public int getPageCount() {
+        return (int) Math.max(1, (totalCount + pageSize - 1) / pageSize);
+    }
+
+}
diff --git a/app/src/main/java/org/apache/roller/weblogger/ui/struts2/pagers/MediaFilePager.java b/app/src/main/java/org/apache/roller/weblogger/ui/struts2/pagers/MediaFilePager.java
index de328eb542..ecd3aeee79 100644
--- a/app/src/main/java/org/apache/roller/weblogger/ui/struts2/pagers/MediaFilePager.java
+++ b/app/src/main/java/org/apache/roller/weblogger/ui/struts2/pagers/MediaFilePager.java
@@ -1,80 +1,77 @@
-/*
- * Licensed to the Apache Software Foundation (ASF) under one or more
- *  contributor license agreements.  The ASF licenses this file to You
- * under the Apache License, Version 2.0 (the "License"); you may not
- * use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *     http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.  For additional information regarding
- * copyright in this work, please see the NOTICE file in the top level
- * directory of this distribution.
- */
-
-package org.apache.roller.weblogger.ui.struts2.pagers;
-
-import java.util.List;
-
-import org.apache.roller.weblogger.pojos.MediaFile;
-
-/**
- * A pager implementation for a collection of media files.
- *
- */
-public class MediaFilePager {
-    
-    // the collection for the pager
-    private final List<MediaFile> items;
-    
-    // What page we are on
-    private final int pageNum;
-    
-    // Are there more items?
-    private final boolean moreItems;
-    
-    public MediaFilePager(int page, List<MediaFile> mediaFiles, boolean hasMore) {
-        this.pageNum = page;
-        this.items = mediaFiles;
-        this.moreItems = hasMore;
-    }
-    
-    public List<MediaFile> getItems() {
-        return items;
-    }
-
-    public boolean isMoreItems() {
-        return moreItems;
-    }
-
-    /**
-     * Indicates whether there are more than one page of item.
-     * 
-     */
-    public boolean isJustOnePage() {
-    	return (pageNum == 0 && !moreItems);
-    }
-    
-    /**
-     * Has previous page?
-     * 
-     */
-    public boolean hasPrevious() {
-    	return (pageNum > 0);
-    }
-    
-    /**
-     * Has next page?
-     * 
-     */
-    public boolean hasNext() {
-    	return this.moreItems;
-    }
-    
-    
-
-}
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ *  contributor license agreements.  The ASF licenses this file to You
+ * under the Apache License, Version 2.0 (the "License"); you may not
+ * use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.  For additional information regarding
+ * copyright in this work, please see the NOTICE file in the top level
+ * directory of this distribution.
+ */
+
+package org.apache.roller.weblogger.ui.struts2.pagers;
+
+import java.util.List;
+
+import org.apache.roller.weblogger.pojos.MediaFile;
+
+/**
+ * A pager implementation for a collection of media files. Media file search
+ * is a POSTed form, so instead of links this exposes page numbers that the
+ * page resubmits the search form with.
+ */
+public class MediaFilePager {
+    
+    // the collection for the pager
+    private final List<MediaFile> items;
+    
+    // What page we are on
+    private final int pageNum;
+    
+    // media files matching the search across all pages
+    private final long totalCount;
+
+    private final int pageSize;
+    
+    public MediaFilePager(int page, List<MediaFile> mediaFiles, long totalCount, int pageSize) {
+        this.pageNum = page;
+        this.items = mediaFiles;
+        this.totalCount = totalCount;
+        this.pageSize = pageSize;
+    }
+    
+    public List<MediaFile> getItems() {
+        return items;
+    }
+
+    public boolean isMoreItems() {
+        return (long) (pageNum + 1) * pageSize < totalCount;
+    }
+
+    public long getTotalCount() {
+        return totalCount;
+    }
+
+    /** Zero-based page number, as the search form submits it. */
+    public int getPageNum() {
+        return pageNum;
+    }
+
+    /** One-based page number, for display. */
+    public int getPageNumber() {
+        return pageNum + 1;
+    }
+
+    /** Number of pages; at least 1, so an empty result reads "Page 1 of 1". */
+    public int getPageCount() {
+        return (int) Math.max(1, (totalCount + pageSize - 1) / pageSize);
+    }
+
+}
diff --git a/app/src/main/resources/ApplicationResources.properties b/app/src/main/resources/ApplicationResources.properties
index 598bd4c61a..38ca7a7d74 100644
--- a/app/src/main/resources/ApplicationResources.properties
+++ b/app/src/main/resources/ApplicationResources.properties
@@ -43,6 +43,14 @@ generic.toggle=Toggle
 generic.success=Success
 generic.looksGood=Looks good!
 generic.poweredBy=Powered by Apache Roller
+generic.first=First
+generic.newer=Newer
+generic.older=Older
+generic.last=Last
+generic.pageNumber=Page {0} of {1}
+generic.pagination=Pagination
+generic.chooseDate=Choose date
+generic.clearDate=Clear date
 
 # ------------------------------------------------------------- BookmarkForm.jsp
 
diff --git a/app/src/main/webapp/WEB-INF/jsps/editor/Comments.jsp b/app/src/main/webapp/WEB-INF/jsps/editor/Comments.jsp
index ad2240cc0e..718c63ffbc 100644
--- a/app/src/main/webapp/WEB-INF/jsps/editor/Comments.jsp
+++ b/app/src/main/webapp/WEB-INF/jsps/editor/Comments.jsp
@@ -84,46 +84,32 @@
 
         <div class="tablenav">
 
-            <div style="float:left;">
-                <s:text name="commentManagement.nowShowing">
-                    <s:param value="pager.items.size()"/>
-                </s:text>
-            </div>
-            <div style="float:right;">
-                <s:if test="firstComment.postTime != null">
-                    <s:text name="commentManagement.date.toStringFormat">
-                        <s:param value="firstComment.postTime"/>
-                    </s:text>
-                </s:if>
-                ---
-                <s:if test="lastComment.postTime != null">
-                    <s:text name="commentManagement.date.toStringFormat">
-                        <s:param value="lastComment.postTime"/>
+            <div class="d-flex flex-wrap justify-content-between gap-2">
+                <div>
+                    <s:text name="commentManagement.nowShowing">
+                        <s:param value="pager.items.size()"/>
                     </s:text>
-                </s:if>
+                </div>
+                <div class="text-body-secondary">
+                    <s:if test="firstComment.postTime != null">
+                        <s:text name="commentManagement.date.toStringFormat">
+                            <s:param value="firstComment.postTime"/>
+                        </s:text>
+                    </s:if>
+                    ---
+                    <s:if test="lastComment.postTime != null">
+                        <s:text name="commentManagement.date.toStringFormat">
+                            <s:param value="lastComment.postTime"/>
+                        </s:text>
+                    </s:if>
+                </div>
             </div>
-            <br/>
 
 
                 <%-- ============================================================= --%>
                 <%-- Next / previous links --%>
 
-            <nav>
-                <ul class="pager">
-                    <s:if test="pager.prevLink != null">
-                        <li class="previous">
-                            <a href="<s:property value="pager.prevLink" />">
-                                <span aria-hidden="true">&larr;</span>Newer</a>
-                        </li>
-                    </s:if>
-                    <s:if test="pager.nextLink != null">
-                        <li class="next">
-                            <a href="<s:property value="pager.nextLink"/>">Older
-                                <span aria-hidden="true">&rarr;</span></a>
-                        </li>
-                    </s:if>
-                </ul>
-            </nav>
+            <s:include value="/WEB-INF/jsps/editor/Pager.jsp"/>
 
         </div> <%-- class="tablenav" --%>
 
@@ -143,231 +129,218 @@
             </p>
         </s:if>
 
-        <table class="rollertable table table-striped" width="100%">
+        <div class="table-responsive">
+            <table class="rollertable table table-striped" width="100%">
 
-                <%-- ======================================================== --%>
-                <%-- Comment table header --%>
+                    <%-- ======================================================== --%>
+                    <%-- Comment table header --%>
 
-            <tr>
-                <s:if test="actionName == 'comments'">
+                <tr>
+                    <s:if test="actionName == 'comments'">
+                        <th class="rollertable" width="5%">
+                            <s:text name="commentManagement.columnApproved"/>
+                        </th>
+                    </s:if>
                     <th class="rollertable" width="5%">
-                        <s:text name="commentManagement.columnApproved"/>
+                        <s:text name="commentManagement.columnSpam"/>
                     </th>
-                </s:if>
-                <th class="rollertable" width="5%">
-                    <s:text name="commentManagement.columnSpam"/>
-                </th>
-                <th class="rollertable" width="5%">
-                    <s:text name="generic.delete"/>
-                </th>
-                <th class="rollertable">
-                    <s:text name="commentManagement.columnComment"/>
-                </th>
-            </tr>
-
-                <%-- ======================================================== --%>
-                <%-- Select ALL and NONE buttons --%>
-
-            <tr class="actionrow">
-                <s:if test="actionName == 'comments'">
-                    <td align="center">
-                        <s:text name="commentManagement.select"/><br/>
+                    <th class="rollertable" width="5%">
+                        <s:text name="generic.delete"/>
+                    </th>
+                    <th class="rollertable">
+                        <s:text name="commentManagement.columnComment"/>
+                    </th>
+                </tr>
 
-                        <span id="checkallapproved"><a href="#"><s:text name="generic.all"/></a></span><br/>
-                        <span id="clearallapproved"><a href="#"><s:text name="generic.none"/></a></span>
-                    </td>
-                </s:if>
-                <td align="center">
-                    <s:text name="commentManagement.select"/><br/>
-
-                    <span id="checkallspam"><a href="#"><s:text name="generic.all"/></a></span><br/>
-                    <span id="clearallspam"><a href="#"><s:text name="generic.none"/></a></span>
-                </td>
-                <td align="center">
-                    <s:text name="commentManagement.select"/><br/>
-
-                    <span id="checkalldelete"><a href="#"><s:text name="generic.all"/></a></span><br/>
-                    <span id="clearalldelete"><a href="#"><s:text name="generic.none"/></a></span>
-                </td>
-                <td align="right">
-                    <br/>
-                    <span class="pendingCommentBox">&nbsp;&nbsp;&nbsp;&nbsp;</span>
-                    <s:text name="commentManagement.pending"/>&nbsp;&nbsp;
-                    <span class="spamCommentBox">&nbsp;&nbsp;&nbsp;&nbsp;</span>
-                    <s:text name="commentManagement.spam"/>&nbsp;&nbsp;
-                </td>
-            </tr>
-
-
-                <%-- ========================================================= --%>
-                <%-- Loop through comments --%>
-
-                <%-- it is a bit funky to use checkbox list here, but using checkbox didn't work for me :( 
-                we are effectively just creating a checkbox list of 1 item for each iteration of our collection. --%>
-
-            <s:iterator var="comment" value="pager.items" status="rowstatus">
-                <tr>
+                    <%-- ======================================================== --%>
+                    <%-- Select ALL and NONE buttons --%>
+
+                <tr class="actionrow">
                     <s:if test="actionName == 'comments'">
-                        <%-- only blog admins (not the global admin) can approve blog comments --%>
-                        <td>
-                            <s:checkboxlist name="bean.approvedComments" cssClass="comment-select"
-                                            list="#comment" listKey="id" listValue="" />
+                        <td align="center">
+                            <s:text name="commentManagement.select"/><br/>
+
+                            <span id="checkallapproved"><a href="#"><s:text name="generic.all"/></a></span><br/>
+                            <span id="clearallapproved"><a href="#"><s:text name="generic.none"/></a></span>
                         </td>
                     </s:if>
-                    <td>
-                        <s:checkboxlist name="bean.spamComments" cssClass="comment-select"
-                                        list="#comment" listKey="id" listValue="" />
+                    <td align="center">
+                        <s:text name="commentManagement.select"/><br/>
+
+                        <span id="checkallspam"><a href="#"><s:text name="generic.all"/></a></span><br/>
+                        <span id="clearallspam"><a href="#"><s:text name="generic.none"/></a></span>
                     </td>
-                    <td>
-                        <s:checkboxlist name="bean.deleteComments" cssClass="comment-select"
-                                        list="#comment" listKey="id" listValue="" />
+                    <td align="center">
+                        <s:text name="commentManagement.select"/><br/>
+
+                        <span id="checkalldelete"><a href="#"><s:text name="generic.all"/></a></span><br/>
+                        <span id="clearalldelete"><a href="#"><s:text name="generic.none"/></a></span>
                     </td>
+                    <td align="right">
+                        <br/>
+                        <span class="pendingCommentBox">&nbsp;&nbsp;&nbsp;&nbsp;</span>
+                        <s:text name="commentManagement.pending"/>&nbsp;&nbsp;
+                        <span class="spamCommentBox">&nbsp;&nbsp;&nbsp;&nbsp;</span>
+                        <s:text name="commentManagement.spam"/>&nbsp;&nbsp;
+                    </td>
+                </tr>
 
-                        <%-- ======================================================== --%>
-                        <%-- Display comment details and text --%>
 
-                        <%-- <td> with style if comment is spam or pending --%>
-                    <s:if test="#comment.status.name() == 'SPAM'">
-                    <td class="spamcomment">
+                    <%-- ========================================================= --%>
+                    <%-- Loop through comments --%>
+
+                    <%-- it is a bit funky to use checkbox list here, but using checkbox didn't work for me :( 
+                    we are effectively just creating a checkbox list of 1 item for each iteration of our collection. --%>
+
+                <s:iterator var="comment" value="pager.items" status="rowstatus">
+                    <tr>
+                        <s:if test="actionName == 'comments'">
+                            <%-- only blog admins (not the global admin) can approve blog comments --%>
+                            <td>
+                                <s:checkboxlist name="bean.approvedComments" cssClass="comment-select"
+                                                list="#comment" listKey="id" listValue="" />
+                            </td>
                         </s:if>
-                        <s:elseif test="#comment.status.name() == 'PENDING'">
-                    <td class="pendingcomment">
-                        </s:elseif>
-                        <s:else>
-                    <td>
-                        </s:else>
-
-                            <%-- comment details table in table --%>
-                        <table class="innertable">
-                            <tr>
-                                <td class="viewbody">
-
-                                    <div class="viewdetails bot">
-
-                                        <div class="details">
-                                            <s:text name="commentManagement.entryTitled"/>&nbsp;:&nbsp;
-                                            <a href="<s:property value="#comment.weblogEntry.permalink" />">
-                                                <s:property value="#comment.weblogEntry.title"/></a>
-                                        </div>
+                        <td>
+                            <s:checkboxlist name="bean.spamComments" cssClass="comment-select"
+                                            list="#comment" listKey="id" listValue="" />
+                        </td>
+                        <td>
+                            <s:checkboxlist name="bean.deleteComments" cssClass="comment-select"
+                                            list="#comment" listKey="id" listValue="" />
+                        </td>
 
-                                        <div class="details">
-                                            <s:text name="commentManagement.commentBy"/>&nbsp;:&nbsp;
-                                            <s:if test="#comment.email != null && #comment.name != null">
-                                                <s:text name="commentManagement.commentByBoth">
-                                                    <s:param><s:property value="#comment.name"/></s:param>
-                                                    <s:param><s:property value="#comment.email"/></s:param>
-                                                    <s:param><s:property value="#comment.email"/></s:param>
-                                                    <s:param><s:property value="#comment.remoteHost"/></s:param>
-                                                </s:text>
-                                            </s:if>
-                                            <s:elseif test="#comment.email == null && #comment.name == null">
-                                                <s:text name="commentManagement.commentByIP">
-                                                    <s:param><s:property value="#comment.remoteHost"/></s:param>
-                                                </s:text>
-                                            </s:elseif>
-                                            <s:else>
-                                                <s:text name="commentManagement.commentByName">
-                                                    <s:param><s:property value="#comment.name"/></s:param>
-                                                    <s:param><s:property value="#comment.remoteHost"/></s:param>
-                                                </s:text>
-                                            </s:else>
-                                        </div>
+                            <%-- ======================================================== --%>
+                            <%-- Display comment details and text --%>
+
+                            <%-- <td> with style if comment is spam or pending --%>
+                        <s:if test="#comment.status.name() == 'SPAM'">
+                        <td class="spamcomment">
+                            </s:if>
+                            <s:elseif test="#comment.status.name() == 'PENDING'">
+                        <td class="pendingcomment">
+                            </s:elseif>
+                            <s:else>
+                        <td>
+                            </s:else>
+
+                                <%-- comment details table in table --%>
+                            <table class="innertable">
+                                <tr>
+                                    <td class="viewbody">
+
+                                        <div class="viewdetails bot">
 
-                                        <s:set var="safeCommentUrl" value="#comment.safeUrl"/>
-                                        <s:if test="#safeCommentUrl != null">
                                             <div class="details">
-                                                <s:text name="commentManagement.commentByURL"/>&nbsp;:&nbsp;
-                                                <a href="<s:property value="#safeCommentUrl" escapeHtml="true" />">
-                                                    <str:truncateNicely upper="60" appendToEnd="..."><s:property
-                                                            value="#safeCommentUrl" escapeHtml="true"/></str:truncateNicely></a>
+                                                <s:text name="commentManagement.entryTitled"/>&nbsp;:&nbsp;
+                                                <a href="<s:property value="#comment.weblogEntry.permalink" />">
+                                                    <s:property value="#comment.weblogEntry.title"/></a>
                                             </div>
-                                        </s:if>
-                                        <s:elseif test="#comment.url != null && #comment.url.trim().length() > 0">
+
                                             <div class="details">
-                                                <s:text name="commentManagement.commentByURL"/>&nbsp;:&nbsp;
-                                                <str:truncateNicely upper="60" appendToEnd="..."><s:property
-                                                        value="#comment.url" escapeHtml="true"/></str:truncateNicely>
+                                                <s:text name="commentManagement.commentBy"/>&nbsp;:&nbsp;
+                                                <s:if test="#comment.email != null && #comment.name != null">
+                                                    <s:text name="commentManagement.commentByBoth">
+                                                        <s:param><s:property value="#comment.name"/></s:param>
+                                                        <s:param><s:property value="#comment.email"/></s:param>
+                                                        <s:param><s:property value="#comment.email"/></s:param>
+                                                        <s:param><s:property value="#comment.remoteHost"/></s:param>
+                                                    </s:text>
+                                                </s:if>
+                                                <s:elseif test="#comment.email == null && #comment.name == null">
+                                                    <s:text name="commentManagement.commentByIP">
+                                                        <s:param><s:property value="#comment.remoteHost"/></s:param>
+                                                    </s:text>
+                                                </s:elseif>
+                                                <s:else>
+                                                    <s:text name="commentManagement.commentByName">
+                                                        <s:param><s:property value="#comment.name"/></s:param>
+                                                        <s:param><s:property value="#comment.remoteHost"/></s:param>
+                                                    </s:text>
+                                                </s:else>
                                             </div>
-                                        </s:elseif>
-
-                                        <div class="details">
-                                            <s:text name="commentManagement.postTime"/>&nbsp;:&nbsp;
-                                            <s:date name="#comment.postTime"/>
-                                        </div>
-
-                                    </div>
-                                    <div class="viewdetails bot">
-
-                                        <div class="details bot">
 
-                                            <s:if test="#comment.content.length() > 1000">
-                                                <div class="bot" id="comment-<s:property value="#comment.id"/>">
-                                                    <str:truncateNicely upper="1000" appendToEnd="...">
-                                                        <s:property value="#comment.content" escapeHtml="true"/>
-                                                    </str:truncateNicely>
-                                                </div>
-                                                <div id="link-<s:property value="#comment.id"/>">
-                                                    <a onclick='readMoreComment("<s:property
-                                                            value="#comment.id"/>")'><s:text
-                                                            name="commentManagement.readmore"/></a>
+                                            <s:set var="safeCommentUrl" value="#comment.safeUrl"/>
+                                            <s:if test="#safeCommentUrl != null">
+                                                <div class="details">
+                                                    <s:text name="commentManagement.commentByURL"/>&nbsp;:&nbsp;
+                                                    <a href="<s:property value="#safeCommentUrl" escapeHtml="true" />">
+                                                        <str:truncateNicely upper="60" appendToEnd="..."><s:property
+                                                                value="#safeCommentUrl" escapeHtml="true"/></str:truncateNicely></a>
                                                 </div>
                                             </s:if>
-                                            <s:else>
-                                                <span width="200px"
-                                                      id="comment-<s:property value="#comment.id"/>"><s:property
-                                                        value="#comment.content" escapeHtml="true"/></span>
-                                            </s:else>
-                                        </div>
+                                            <s:elseif test="#comment.url != null && #comment.url.trim().length() > 0">
+                                                <div class="details">
+                                                    <s:text name="commentManagement.commentByURL"/>&nbsp;:&nbsp;
+                                                    <str:truncateNicely upper="60" appendToEnd="..."><s:property
+                                                            value="#comment.url" escapeHtml="true"/></str:truncateNicely>
+                                                </div>
+                                            </s:elseif>
 
-                                        <s:if test="actionName == 'comments'">
                                             <div class="details">
-                                                <a id="editlink-<s:property value="#comment.id"/>"
-                                                   onclick='editComment("<s:property value="#comment.id"/>")'>
-                                                    <s:text name="generic.edit"/>
-                                                </a>
+                                                <s:text name="commentManagement.postTime"/>&nbsp;:&nbsp;
+                                                <s:date name="#comment.postTime"/>
                                             </div>
-                                            <div class="details">
-                                              <span id="savelink-<s:property value="#comment.id"/>"
-                                                    style="display: none">
-                                                   <a onclick='saveComment("<s:property value="#comment.id"/>")'><s:text
-                                                           name="generic.save"/></a> &nbsp;|&nbsp;
-                                              </span>
-                                                <span id="cancellink-<s:property value="#comment.id"/>"
-                                                      style="display: none">
-                                                   <a onclick='editCommentCancel("<s:property
-                                                           value="#comment.id"/>")'><s:text name="generic.cancel"/></a>
-                                              </span>
+
+                                        </div>
+                                        <div class="viewdetails bot">
+
+                                            <div class="details bot">
+
+                                                <s:if test="#comment.content.length() > 1000">
+                                                    <div class="bot" id="comment-<s:property value="#comment.id"/>">
+                                                        <str:truncateNicely upper="1000" appendToEnd="...">
+                                                            <s:property value="#comment.content" escapeHtml="true"/>
+                                                        </str:truncateNicely>
+                                                    </div>
+                                                    <div id="link-<s:property value="#comment.id"/>">
+                                                        <a onclick='readMoreComment("<s:property
+                                                                value="#comment.id"/>")'><s:text
+                                                                name="commentManagement.readmore"/></a>
+                                                    </div>
+                                                </s:if>
+                                                <s:else>
+                                                    <span width="200px"
+                                                          id="comment-<s:property value="#comment.id"/>"><s:property
+                                                            value="#comment.content" escapeHtml="true"/></span>
+                                                </s:else>
                                             </div>
-                                        </s:if>
 
-                                    </div>
-                            </tr>
-                        </table>
-                            <%-- end comment details table in table --%>
-                    </td>
-                </tr>
-            </s:iterator>
-        </table>
+                                            <s:if test="actionName == 'comments'">
+                                                <div class="details">
+                                                    <a id="editlink-<s:property value="#comment.id"/>"
+                                                       onclick='editComment("<s:property value="#comment.id"/>")'>
+                                                        <s:text name="generic.edit"/>
+                                                    </a>
+                                                </div>
+                                                <div class="details">
+                                                  <span id="savelink-<s:property value="#comment.id"/>"
+                                                        style="display: none">
+                                                       <a onclick='saveComment("<s:property value="#comment.id"/>")'><s:text
+                                                               name="generic.save"/></a> &nbsp;|&nbsp;
+                                                  </span>
+                                                    <span id="cancellink-<s:property value="#comment.id"/>"
+                                                          style="display: none">
+                                                       <a onclick='editCommentCancel("<s:property
+                                                               value="#comment.id"/>")'><s:text name="generic.cancel"/></a>
+                                                  </span>
+                                                </div>
+                                            </s:if>
+
+                                        </div>
+                                </tr>
+                            </table>
+                                <%-- end comment details table in table --%>
+                        </td>
+                    </tr>
+                </s:iterator>
+            </table>
+        </div>
 
         <%-- ============================================================= --%>
         <%-- Next / previous links --%>
 
-        <nav>
-            <ul class="pager">
-                <s:if test="pager.prevLink != null">
-                    <li class="previous">
-                        <a href="<s:property value="pager.prevLink" />">
-                            <span aria-hidden="true">&larr;</span>Newer</a>
-                    </li>
-                </s:if>
-                <s:if test="pager.nextLink != null">
-                    <li class="next">
-                        <a href="<s:property value="pager.nextLink"/>">Older
-                            <span aria-hidden="true">&rarr;</span></a>
-                    </li>
-                </s:if>
-            </ul>
-        </nav>
+        <s:include value="/WEB-INF/jsps/editor/Pager.jsp"/>
 
         <%-- ========================================================= --%>
         <%-- Save changes and cancel buttons --%>
diff --git a/app/src/main/webapp/WEB-INF/jsps/editor/CommentsSidebar.jsp b/app/src/main/webapp/WEB-INF/jsps/editor/CommentsSidebar.jsp
index 7e8cf2f744..50a10c1a82 100644
--- a/app/src/main/webapp/WEB-INF/jsps/editor/CommentsSidebar.jsp
+++ b/app/src/main/webapp/WEB-INF/jsps/editor/CommentsSidebar.jsp
@@ -29,7 +29,7 @@
 
 <p><s:text name="commentManagement.sidebarDescription"/></p>
 
-<s:form action="%{#mainAction}!query" id="commentsQuery" theme="bootstrap" cssClass="form-vertical">
+<s:form action="%{#mainAction}!query" id="commentsQuery" theme="bootstrap" cssClass="form-vertical roller-filter-form">
 
     <s:hidden name="salt"/>
     <s:hidden name="weblog"/>
@@ -39,24 +39,11 @@
 
     <s:textfield name="bean.searchString" label="%{getText('commentManagement.searchString')}" size="15"/>
 
-    <script>
-        $(function () {
-            $("#entries_bean_startDateString").datepicker();
-        });
-    </script>
-
     <%-- ========================================================= --%>
     <%-- filter by date --%>
 
-    <script>
-        // jQuery UI Date Picker
-        $(function () {
-            $("#commentsQuery_bean_startDateString").datepicker();
-        });
-    </script>
-    
     <div class="control-group">
-        <label for="bean.startDateString" class="col-form-label">
+        <label for="commentsQuery_bean_startDateString" class="col-form-label">
             <s:text name="commentManagement.startDate"/>
         </label>
         <div class="controls">
@@ -64,23 +51,21 @@
 
                 <s:textfield name="bean.startDateString" readonly="true"
                              theme="simple" cssClass="date-picker form-control"/>
-                <label for="bean.startDateString" class="input-group-text btn">
-                    <span class="bi bi-calendar"></span>
-                </label>
+                <button type="button" class="btn btn-outline-secondary date-picker-show"
+                        title="<s:text name='generic.chooseDate'/>" aria-label="<s:text name='generic.chooseDate'/>">
+                    <span class="bi bi-calendar" aria-hidden="true"></span>
+                </button>
+                <button type="button" class="btn btn-outline-secondary date-picker-clear"
+                        title="<s:text name='generic.clearDate'/>" aria-label="<s:text name='generic.clearDate'/>">
+                    <span class="bi bi-x-lg" aria-hidden="true"></span>
+                </button>
 
             </div>
         </div>
     </div>
 
-    <script>
-        // jQuery UI Date Picker
-        $(function () {
-            $("#commentsQuery_bean_endDateString").datepicker();
-        });
-    </script>
-
     <div class="control-group">
-        <label for="bean.endDateString" class="col-form-label">
+        <label for="commentsQuery_bean_endDateString" class="col-form-label">
             <s:text name="commentManagement.endDate"/>
         </label>
         <div class="controls">
@@ -88,16 +73,19 @@
 
                 <s:textfield name="bean.endDateString" readonly="true"
                              theme="simple" cssClass="date-picker form-control"/>
-                <label for="bean.endDateString" class="input-group-text btn">
-                    <span class="bi bi-calendar"></span>
-                </label>
+                <button type="button" class="btn btn-outline-secondary date-picker-show"
+                        title="<s:text name='generic.chooseDate'/>" aria-label="<s:text name='generic.chooseDate'/>">
+                    <span class="bi bi-calendar" aria-hidden="true"></span>
+                </button>
+                <button type="button" class="btn btn-outline-secondary date-picker-clear"
+                        title="<s:text name='generic.clearDate'/>" aria-label="<s:text name='generic.clearDate'/>">
+                    <span class="bi bi-x-lg" aria-hidden="true"></span>
+                </button>
 
             </div>
         </div>
     </div>
 
-    <br/>
-
     <%-- ========================================================= --%>
     <%-- filter by status--%>
 
diff --git a/app/src/main/webapp/WEB-INF/jsps/editor/Entries.jsp b/app/src/main/webapp/WEB-INF/jsps/editor/Entries.jsp
index 3e2fc8d26b..42935802a3 100644
--- a/app/src/main/webapp/WEB-INF/jsps/editor/Entries.jsp
+++ b/app/src/main/webapp/WEB-INF/jsps/editor/Entries.jsp
@@ -30,22 +30,7 @@
 <%-- ============================================================= --%>
 <%-- Next / previous links --%>
 
-<nav>
-    <ul class="pager">
-        <s:if test="pager.prevLink != null">
-            <li class="previous">
-                <a href='<s:property value="pager.prevLink" />'> 
-                    <span aria-hidden="true">&larr;</span>Newer</a>
-            </li>
-        </s:if>
-        <s:if test="pager.nextLink != null">
-            <li class="next">
-                <a href='<s:property value="pager.nextLink"/>'>Older
-                    <span aria-hidden="true">&rarr;</span></a>
-            </li>
-        </s:if>
-    </ul>
-</nav>
+<s:include value="/WEB-INF/jsps/editor/Pager.jsp"/>
 
 
 <%-- ============================================================= --%>
@@ -60,120 +45,107 @@
     <s:text name="weblogEntryQuery.scheduled" />&nbsp;&nbsp;
 </p>
 
-<table class="rollertable table table-striped" width="100%">
-
-<tr>
-    <th class="rollertable" width="3%"> </th>
-    <th class="rollertable" width="7%">
-        <s:text name="weblogEntryQuery.pubTime" />
-    </th>
-    <th class="rollertable" width="7%">
-        <s:text name="weblogEntryQuery.updateTime" />
-    </th>
-    <th class="rollertable">
-        <s:text name="weblogEntryQuery.title" />
-    </th>
-    <th class="rollertable" width="15%">
-        <s:text name="weblogEntryQuery.category" />
-    </th>
-    <th class="rollertable" width="3%"> </th>
-</tr>
-
-<s:iterator var="post" value="pager.items">
-    <%-- <td> with style if comment is spam or pending --%>
-    <s:if test="#post.status.name() == 'DRAFT'">
-        <tr class="draftentry"> 
-    </s:if>
-    <s:elseif test="#post.status.name() == 'PENDING'">
-        <tr class="pendingentry"> 
-    </s:elseif>
-    <s:elseif test="#post.status.name() == 'SCHEDULED'">
-        <tr class="scheduledentry"> 
-    </s:elseif>
-    <s:else>
-        <tr>
-    </s:else>
-
-    <td>
-        <s:url var="editUrl" action="entryEdit">
-            <s:param name="weblog" value="%{actionWeblog.handle}" />
-            <s:param name="bean.id" value="#post.id" />
-        </s:url>
-        <s:a href="%{editUrl}">
-            <span class="bi bi-pencil-square" data-bs-toggle="tooltip" data-bs-placement="top"
-                  title="<s:text name='generic.edit'/>">
-            </span>
-        </s:a>
-    </td>
-
-    <td>
-        <s:if test="#post.pubTime != null">
-            <s:text name="weblogEntryQuery.date.toStringFormat">
-                <s:param value="#post.pubTime" />
-            </s:text>
+<div class="table-responsive">
+    <table class="rollertable table table-striped" width="100%">
+
+    <tr>
+        <th class="rollertable" width="3%"> </th>
+        <th class="rollertable" width="7%">
+            <s:text name="weblogEntryQuery.pubTime" />
+        </th>
+        <th class="rollertable" width="7%">
+            <s:text name="weblogEntryQuery.updateTime" />
+        </th>
+        <th class="rollertable">
+            <s:text name="weblogEntryQuery.title" />
+        </th>
+        <th class="rollertable" width="15%">
+            <s:text name="weblogEntryQuery.category" />
+        </th>
+        <th class="rollertable" width="3%"> </th>
+    </tr>
+
+    <s:iterator var="post" value="pager.items">
+        <%-- <td> with style if comment is spam or pending --%>
+        <s:if test="#post.status.name() == 'DRAFT'">
+            <tr class="draftentry"> 
         </s:if>
-    </td>
+        <s:elseif test="#post.status.name() == 'PENDING'">
+            <tr class="pendingentry"> 
+        </s:elseif>
+        <s:elseif test="#post.status.name() == 'SCHEDULED'">
+            <tr class="scheduledentry"> 
+        </s:elseif>
+        <s:else>
+            <tr>
+        </s:else>
+
+        <td>
+            <s:url var="editUrl" action="entryEdit">
+                <s:param name="weblog" value="%{actionWeblog.handle}" />
+                <s:param name="bean.id" value="#post.id" />
+            </s:url>
+            <s:a href="%{editUrl}">
+                <span class="bi bi-pencil-square" data-bs-toggle="tooltip" data-bs-placement="top"
+                      title="<s:text name='generic.edit'/>">
+                </span>
+            </s:a>
+        </td>
+
+        <td>
+            <s:if test="#post.pubTime != null">
+                <s:text name="weblogEntryQuery.date.toStringFormat">
+                    <s:param value="#post.pubTime" />
+                </s:text>
+            </s:if>
+        </td>
     
-    <td>
-        <s:if test="#post.updateTime != null">
-            <s:text name="weblogEntryQuery.date.toStringFormat">
-                <s:param value="#post.updateTime" />
-            </s:text>
-        </s:if>
-    </td>
+        <td>
+            <s:if test="#post.updateTime != null">
+                <s:text name="weblogEntryQuery.date.toStringFormat">
+                    <s:param value="#post.updateTime" />
+                </s:text>
+            </s:if>
+        </td>
     
-    <td>
-        <s:if test="#post.status.name() == 'PUBLISHED'">
-            <a href='<s:property value="#post.permalink" />'>
+        <td>
+            <s:if test="#post.status.name() == 'PUBLISHED'">
+                <a href='<s:property value="#post.permalink" />'>
+                    <str:truncateNicely upper="80"><s:property value="#post.displayTitle" /></str:truncateNicely>
+                </a>
+            </s:if>
+            <s:else>
                 <str:truncateNicely upper="80"><s:property value="#post.displayTitle" /></str:truncateNicely>
-            </a>
-        </s:if>
-        <s:else>
-            <str:truncateNicely upper="80"><s:property value="#post.displayTitle" /></str:truncateNicely>
-        </s:else>
-    </td>
+            </s:else>
+        </td>
     
-    <td>
-        <s:property value="#post.category.name" />
-    </td>
-
-    <td>
-        <s:set var="postId" value="#post.id" />
-        <s:set var="postTitle" value="#post.title" />
-        <a href="#" class="entry-delete-link"
-            data-post-id="<s:property value="postId" />"
-            data-post-title="<s:property value="postTitle"/>">
-            <span class="bi bi-trash"
-                  data-bs-toggle="tooltip" data-bs-placement="top" title="<s:text name='generic.delete'/>">
-            </span>
-        </a>
-    </td>
+        <td>
+            <s:property value="#post.category.name" />
+        </td>
+
+        <td>
+            <s:set var="postId" value="#post.id" />
+            <s:set var="postTitle" value="#post.title" />
+            <a href="#" class="entry-delete-link"
+                data-post-id="<s:property value="postId" />"
+                data-post-title="<s:property value="postTitle"/>">
+                <span class="bi bi-trash"
+                      data-bs-toggle="tooltip" data-bs-placement="top" title="<s:text name='generic.delete'/>">
+                </span>
+            </a>
+        </td>
 
-    </tr>
-</s:iterator>
+        </tr>
+    </s:iterator>
 
-</table>
+    </table>
+</div>
 
 
 <%-- ============================================================= --%>
 <%-- Next / previous links --%>
 
-<nav>
-    <ul class="pager">
-        <s:if test="pager.prevLink != null">
-            <li class="previous">
-                <a href='<s:property value="pager.prevLink" />'>
-                    <span aria-hidden="true">&larr;</span> Older</a>
-            </li>
-        </s:if>
-        <s:if test="pager.nextLink != null">
-            <li class="next">
-                <a href='<s:property value="pager.nextLink"/>'>Newer
-                    <span aria-hidden="true">&rarr;</span></a>
-            </li>
-        </s:if>
-    </ul>
-</nav>
+<s:include value="/WEB-INF/jsps/editor/Pager.jsp"/>
 
 <s:if test="pager.items.isEmpty">
     <s:text name="weblogEntryQuery.noneFound" />
diff --git a/app/src/main/webapp/WEB-INF/jsps/editor/EntriesSidebar.jsp b/app/src/main/webapp/WEB-INF/jsps/editor/EntriesSidebar.jsp
index e58527d923..ae3b9bab6f 100644
--- a/app/src/main/webapp/WEB-INF/jsps/editor/EntriesSidebar.jsp
+++ b/app/src/main/webapp/WEB-INF/jsps/editor/EntriesSidebar.jsp
@@ -22,7 +22,7 @@
 
 <p><s:text name="weblogEntryQuery.sidebarDescription"/></p>
 
-<s:form action="entries" theme="bootstrap" cssClass="form-vertical">
+<s:form action="entries" theme="bootstrap" cssClass="form-vertical roller-filter-form">
     <s:hidden name="salt"/>
     <s:hidden name="weblog"/>
 
@@ -49,7 +49,7 @@
     <%-- filter by date --%>
 
     <div class="control-group">
-        <label for="bean.startDateString" class="col-form-label">
+        <label for="entries_bean_startDateString" class="col-form-label">
             <s:text name="weblogEntryQuery.label.startDate"/>
         </label>
         <div class="controls">
@@ -57,16 +57,21 @@
 
                 <s:textfield name="bean.startDateString" readonly="true"
                              theme="simple" cssClass="date-picker form-control"/>
-                <label for="bean.startDateString" class="input-group-text btn">
-                    <span class="bi bi-calendar"></span>
-                </label>
+                <button type="button" class="btn btn-outline-secondary date-picker-show"
+                        title="<s:text name='generic.chooseDate'/>" aria-label="<s:text name='generic.chooseDate'/>">
+                    <span class="bi bi-calendar" aria-hidden="true"></span>
+                </button>
+                <button type="button" class="btn btn-outline-secondary date-picker-clear"
+                        title="<s:text name='generic.clearDate'/>" aria-label="<s:text name='generic.clearDate'/>">
+                    <span class="bi bi-x-lg" aria-hidden="true"></span>
+                </button>
 
             </div>
         </div>
     </div>
 
     <div class="control-group">
-        <label for="bean.endDateString" class="col-form-label">
+        <label for="entries_bean_endDateString" class="col-form-label">
             <s:text name="weblogEntryQuery.label.endDate"/>
         </label>
         <div class="controls">
@@ -74,16 +79,19 @@
 
                 <s:textfield name="bean.endDateString" readonly="true"
                              theme="simple" cssClass="date-picker form-control"/>
-                <label for="bean.endDateString" class="input-group-text btn">
-                    <span class="bi bi-calendar"></span>
-                </label>
+                <button type="button" class="btn btn-outline-secondary date-picker-show"
+                        title="<s:text name='generic.chooseDate'/>" aria-label="<s:text name='generic.chooseDate'/>">
+                    <span class="bi bi-calendar" aria-hidden="true"></span>
+                </button>
+                <button type="button" class="btn btn-outline-secondary date-picker-clear"
+                        title="<s:text name='generic.clearDate'/>" aria-label="<s:text name='generic.clearDate'/>">
+                    <span class="bi bi-x-lg" aria-hidden="true"></span>
+                </button>
 
             </div>
         </div>
     </div>
 
-    <br/>
-
     <%-- ========================================================= --%>
     <%-- filter by status --%>
 
@@ -106,12 +114,4 @@
 
 </s:form>
 
-<script>
-
-    $(document).ready(function () {
-        $("#entries_bean_startDateString").datepicker();
-        $("#entries_bean_endDateString").datepicker();
-    });
-
-</script>
 
diff --git a/app/src/main/webapp/WEB-INF/jsps/editor/EntryEdit.jsp b/app/src/main/webapp/WEB-INF/jsps/editor/EntryEdit.jsp
index 59b6cd8582..e93b76ded7 100644
--- a/app/src/main/webapp/WEB-INF/jsps/editor/EntryEdit.jsp
+++ b/app/src/main/webapp/WEB-INF/jsps/editor/EntryEdit.jsp
@@ -150,7 +150,7 @@
     </div>
 
 
-    <div class="accordion" id="accordion">
+    <div class="accordion my-4" id="accordion">
 
             <%-- Weblog editor --%>
 
@@ -160,17 +160,16 @@
 
         <s:if test="!entryPlugins.isEmpty">
 
-            <div class="card" id="panel-plugins">
-                <div class="card-header">
-
-                    <h4 class="card-title">
-                        <a class="collapsed text-body text-decoration-none" data-bs-toggle="collapse" data-bs-target="#collapsePlugins" href="#">
-                            <s:text name="weblogEdit.pluginsToApply"/> </a>
-                    </h4>
-
-                </div>
-                <div id="collapsePlugins" class="collapse">
-                    <div class="card-body">
+            <div class="accordion-item" id="panel-plugins">
+                <h2 class="accordion-header">
+                    <button type="button" class="accordion-button collapsed"
+                            data-bs-toggle="collapse" data-bs-target="#collapsePlugins"
+                            aria-expanded="false" aria-controls="collapsePlugins">
+                        <s:text name="weblogEdit.pluginsToApply"/>
+                    </button>
+                </h2>
+                <div id="collapsePlugins" class="accordion-collapse collapse">
+                    <div class="accordion-body">
 
                         <s:checkboxlist name="bean.plugins" list="entryPlugins" listKey="name" listValue="name"/>
 
@@ -182,18 +181,16 @@
 
             <%-- Advanced settings --%>
 
-        <div class="card" id="panel-settings">
-            <div class="card-header">
-
-                <h4 class="card-title">
-                    <a class="collapsed text-body text-decoration-none" data-bs-toggle="collapse" data-bs-parent="#collapseAdvanced"
-                       href="#collapseAdvanced">
-                        <s:text name="weblogEdit.miscSettings"/> </a>
-                </h4>
-
-            </div>
-            <div id="collapseAdvanced" class="collapse">
-                <div class="card-body">
+        <div class="accordion-item" id="panel-settings">
+            <h2 class="accordion-header">
+                <button type="button" class="accordion-button collapsed"
+                        data-bs-toggle="collapse" data-bs-target="#collapseAdvanced"
+                        aria-expanded="false" aria-controls="collapseAdvanced">
+                    <s:text name="weblogEdit.miscSettings"/>
+                </button>
+            </h2>
+            <div id="collapseAdvanced" class="accordion-collapse collapse">
+                <div class="accordion-body">
 
                     <div class="row mb-3">
 
@@ -213,9 +210,10 @@
                             <div class="input-group w-auto d-inline-flex">
                                 <s:textfield name="bean.dateString" readonly="true" cssStyle="width:15em"
                                              theme="simple" cssClass="date-picker form-control"/>
-                                <label for="bean.dateString" class="input-group-text btn" style="width:3em">
-                                    <span class="bi bi-calendar"></span>
-                                </label>
+                                <button type="button" class="btn btn-outline-secondary date-picker-show"
+                                        title="<s:text name='generic.chooseDate'/>" aria-label="<s:text name='generic.chooseDate'/>">
+                                    <span class="bi bi-calendar" aria-hidden="true"></span>
+                                </button>
                             </div>
 
                             <s:property value="actionWeblog.timeZone"/>
@@ -257,43 +255,45 @@
     <%-- ================================================================== --%>
     <%-- The button box --%>
 
-    <%-- save draft --%>
-    <s:submit theme="simple" cssClass="btn btn-warning"
-              value="%{getText('weblogEdit.save')}"
-              action="%{#mainAction}!saveDraft"/>
+    <div class="d-flex flex-wrap gap-2 mt-4">
 
-    <s:if test="actionName == 'entryEdit'">
+        <%-- save draft --%>
+        <s:submit theme="simple" cssClass="btn btn-warning"
+                  value="%{getText('weblogEdit.save')}"
+                  action="%{#mainAction}!saveDraft"/>
 
-        <%-- preview mode --%>
-        <input class="btn btn-outline-secondary" type="button" name="fullPreview"
-               value="<s:text name='weblogEdit.fullPreviewMode' />"
-               onclick="fullPreviewMode()"/>
-    </s:if>
-    <s:if test="userAnAuthor">
+        <s:if test="actionName == 'entryEdit'">
 
-        <%-- publish --%>
-        <s:submit theme="simple" cssClass="btn btn-success"
-                  value="%{getText('weblogEdit.post')}"
-                  action="%{#mainAction}!publish"/>
-    </s:if>
-    <s:else>
+            <%-- preview mode --%>
+            <input class="btn btn-outline-secondary" type="button" name="fullPreview"
+                   value="<s:text name='weblogEdit.fullPreviewMode' />"
+                   onclick="fullPreviewMode()"/>
+        </s:if>
+        <s:if test="userAnAuthor">
 
-        <%-- submit for review --%>
-        <s:submit theme="simple" cssClass="btn btn-info"
-                  value="%{getText('weblogEdit.submitForReview')}"
-                  action="%{#mainAction}!publish"/>
-    </s:else>
+            <%-- publish --%>
+            <s:submit theme="simple" cssClass="btn btn-success"
+                      value="%{getText('weblogEdit.post')}"
+                      action="%{#mainAction}!publish"/>
+        </s:if>
+        <s:else>
 
-    <s:if test="actionName == 'entryEdit'">
+            <%-- submit for review --%>
+            <s:submit theme="simple" cssClass="btn btn-info"
+                      value="%{getText('weblogEdit.submitForReview')}"
+                      action="%{#mainAction}!publish"/>
+        </s:else>
+
+        <s:if test="actionName == 'entryEdit'">
 
-        <%-- delete --%>
-        <span style="float:right">
-            <input class="btn btn-danger entry-delete-button" type="button"
+            <%-- delete --%>
+            <input class="btn btn-danger entry-delete-button ms-auto" type="button"
                    value="<s:text name='weblogEdit.deleteEntry'/>"
                    data-post-id="<s:property value="entry.id" />"
                    data-post-title="<s:property value="entry.title"/>">
-        </span>
-    </s:if>
+        </s:if>
+
+    </div>
 
 </s:form>
 
@@ -363,10 +363,6 @@
 
 <script>
 
-    $(document).ready(function () {
-        $("#entry_bean_dateString").datepicker();
-    });
-
     function fullPreviewMode() {
         window.open('<s:property value="previewURL" />', 'roller-preview');
     }
diff --git a/app/src/main/webapp/WEB-INF/jsps/editor/EntryEditor.jsp b/app/src/main/webapp/WEB-INF/jsps/editor/EntryEditor.jsp
index 68ba4c2430..5e7e8999ec 100644
--- a/app/src/main/webapp/WEB-INF/jsps/editor/EntryEditor.jsp
+++ b/app/src/main/webapp/WEB-INF/jsps/editor/EntryEditor.jsp
@@ -30,19 +30,16 @@
 
 <%-- summary --%>
 
-<div class="card" id="panel-summary">
-    <div class="card-header">
-
-        <h4 class="card-title">
-            <a href="#" class="collapsed text-body text-decoration-none"
-               data-bs-toggle="collapse" data-bs-target="#collapseSummaryEditor">
-                <s:text name="weblogEdit.summary"/>
-            </a>
-        </h4>
-
-    </div>
-    <div id="collapseSummaryEditor" class="collapse">
-        <div class="card-body">
+<div class="accordion-item" id="panel-summary">
+    <h2 class="accordion-header">
+        <button type="button" class="accordion-button collapsed"
+                data-bs-toggle="collapse" data-bs-target="#collapseSummaryEditor"
+                aria-expanded="false" aria-controls="collapseSummaryEditor">
+            <s:text name="weblogEdit.summary"/>
+        </button>
+    </h2>
+    <div id="collapseSummaryEditor" class="accordion-collapse collapse">
+        <div class="accordion-body">
 
             <s:textarea id="edit_summary" name="bean.summary"
                         tabindex="6" rows="10" cssClass="col-sm-12" theme="simple"/>
diff --git a/app/src/main/webapp/WEB-INF/jsps/editor/MediaFilePager.jsp b/app/src/main/webapp/WEB-INF/jsps/editor/MediaFilePager.jsp
new file mode 100644
index 0000000000..1b5b12164a
--- /dev/null
+++ b/app/src/main/webapp/WEB-INF/jsps/editor/MediaFilePager.jsp
@@ -0,0 +1,83 @@
+<%--
+  Licensed to the Apache Software Foundation (ASF) under one or more
+   contributor license agreements.  The ASF licenses this file to You
+  under the Apache License, Version 2.0 (the "License"); you may not
+  use this file except in compliance with the License.
+  You may obtain a copy of the License at
+
+      http://www.apache.org/licenses/LICENSE-2.0
+
+  Unless required by applicable law or agreed to in writing, software
+  distributed under the License is distributed on an "AS IS" BASIS,
+  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+  See the License for the specific language governing permissions and
+  limitations under the License.  For additional information regarding
+  copyright in this work, please see the NOTICE file in the top level
+  directory of this distribution.
+--%>
+<%@ include file="/WEB-INF/jsps/taglibs-struts2.jsp" %>
+
+<%-- Media file search results pager: the same control as Pager.jsp, but search is a POSTed
+     form, so each button resubmits #mediaFileSearchForm with bean.pageNum set to its page
+     (see MediaFileSidebar.jsp). --%>
+
+<s:if test="pager.pageCount > 1">
+    <nav class="roller-pager" aria-label="<s:text name='generic.pagination'/>">
+
+        <s:if test="pager.pageNum > 0">
+            <button type="button" class="btn btn-outline-primary btn-sm media-search-page"
+                    data-page="0">
+                <span class="bi bi-chevron-double-left" aria-hidden="true"></span>
+                <s:text name="generic.first"/></button>
+        </s:if>
+        <s:else>
+            <span class="btn btn-outline-secondary btn-sm disabled" aria-disabled="true">
+                <span class="bi bi-chevron-double-left" aria-hidden="true"></span>
+                <s:text name="generic.first"/></span>
+        </s:else>
+
+        <s:if test="pager.pageNum > 0">
+            <button type="button" class="btn btn-outline-primary btn-sm media-search-page"
+                    data-page="<s:property value='pager.pageNum - 1'/>">
+                <span class="bi bi-chevron-left" aria-hidden="true"></span>
+                <s:text name="generic.newer"/></button>
+        </s:if>
+        <s:else>
+            <span class="btn btn-outline-secondary btn-sm disabled" aria-disabled="true">
+                <span class="bi bi-chevron-left" aria-hidden="true"></span>
+                <s:text name="generic.newer"/></span>
+        </s:else>
+
+        <span class="roller-pager-page">
+            <s:text name="generic.pageNumber">
+                <s:param value="pager.pageNumber"/>
+                <s:param value="pager.pageCount"/>
+            </s:text>
+        </span>
+
+        <s:if test="pager.moreItems">
+            <button type="button" class="btn btn-outline-primary btn-sm media-search-page"
+                    data-page="<s:property value='pager.pageNum + 1'/>">
+                <s:text name="generic.older"/>
+                <span class="bi bi-chevron-right" aria-hidden="true"></span></button>
+        </s:if>
+        <s:else>
+            <span class="btn btn-outline-secondary btn-sm disabled" aria-disabled="true">
+                <s:text name="generic.older"/>
+                <span class="bi bi-chevron-right" aria-hidden="true"></span></span>
+        </s:else>
+
+        <s:if test="pager.moreItems">
+            <button type="button" class="btn btn-outline-primary btn-sm media-search-page"
+                    data-page="<s:property value='pager.pageCount - 1'/>">
+                <s:text name="generic.last"/>
+                <span class="bi bi-chevron-double-right" aria-hidden="true"></span></button>
+        </s:if>
+        <s:else>
+            <span class="btn btn-outline-secondary btn-sm disabled" aria-disabled="true">
+                <s:text name="generic.last"/>
+                <span class="bi bi-chevron-double-right" aria-hidden="true"></span></span>
+        </s:else>
+
+    </nav>
+</s:if>
diff --git a/app/src/main/webapp/WEB-INF/jsps/editor/MediaFileSidebar.jsp b/app/src/main/webapp/WEB-INF/jsps/editor/MediaFileSidebar.jsp
index 4a11314b88..82c40c0d61 100644
--- a/app/src/main/webapp/WEB-INF/jsps/editor/MediaFileSidebar.jsp
+++ b/app/src/main/webapp/WEB-INF/jsps/editor/MediaFileSidebar.jsp
@@ -64,6 +64,8 @@
                     <s:hidden name="salt"/>
                     <s:hidden name="weblog"/>
                     <input type="hidden" name="mediaFileId" value=""/>
+                    <%-- set by the result pager buttons; a new search starts at page 0 --%>
+                    <input type="hidden" id="beanPageNum" name="bean.pageNum" value="0"/>
 
                     <s:textfield id="beanName" name="bean.name" size="20" maxlength="255"
                                  label="%{getText('generic.name')}"/>
@@ -120,6 +122,12 @@
         }
     }
 
+    // result pager buttons (MediaFilePager.jsp) resubmit this search for their page
+    $(document).on("click", ".media-search-page", function () {
+        $("#beanPageNum").val($(this).data("page"));
+        $("#mediaFileSearchForm").submit();
+    });
+
     $("#searchButton").ready(function () {
 
         maintainSearchButtonState();
diff --git a/app/src/main/webapp/WEB-INF/jsps/editor/MediaFileView.jsp b/app/src/main/webapp/WEB-INF/jsps/editor/MediaFileView.jsp
index b9880ad882..574c138045 100644
--- a/app/src/main/webapp/WEB-INF/jsps/editor/MediaFileView.jsp
+++ b/app/src/main/webapp/WEB-INF/jsps/editor/MediaFileView.jsp
@@ -53,7 +53,7 @@
 
         <s:if test="pager.items.size() > 0">
         <s:text name="mediaFileView.matchingResults">
-            <s:param value="pager.items.size()"/>
+            <s:param value="pager.totalCount"/>
         </s:text>
         </s:if>
         <s:else>
@@ -94,6 +94,8 @@
         </s:if>
     </ul>
 
+    <s:include value="/WEB-INF/jsps/editor/MediaFilePager.jsp"/>
+
 </s:elseif>
 
 <s:else>
@@ -275,6 +277,10 @@
 
         <div style="clear:left;"></div>
 
+        <s:if test="pager">
+            <s:include value="/WEB-INF/jsps/editor/MediaFilePager.jsp"/>
+        </s:if>
+
         <s:if test="(!pager && childFiles.size() > 0) || (pager && pager.items.size() > 0) || (currentDirectory.name != 'default' && !pager)">
 
             <div class="image-controls">
diff --git a/app/src/main/webapp/WEB-INF/jsps/editor/Pager.jsp b/app/src/main/webapp/WEB-INF/jsps/editor/Pager.jsp
new file mode 100644
index 0000000000..728bd81873
--- /dev/null
+++ b/app/src/main/webapp/WEB-INF/jsps/editor/Pager.jsp
@@ -0,0 +1,78 @@
+<%--
+  Licensed to the Apache Software Foundation (ASF) under one or more
+   contributor license agreements.  The ASF licenses this file to You
+  under the Apache License, Version 2.0 (the "License"); you may not
+  use this file except in compliance with the License.
+  You may obtain a copy of the License at
+
+      http://www.apache.org/licenses/LICENSE-2.0
+
+  Unless required by applicable law or agreed to in writing, software
+  distributed under the License is distributed on an "AS IS" BASIS,
+  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+  See the License for the specific language governing permissions and
+  limitations under the License.  For additional information regarding
+  copyright in this work, please see the NOTICE file in the top level
+  directory of this distribution.
+--%>
+<%@ include file="/WEB-INF/jsps/taglibs-struts2.jsp" %>
+
+<%-- First / Newer / Older / Last buttons for pages whose action has a LinkPager named "pager".
+     MediaFileView.jsp draws the same control with buttons that resubmit its search form. --%>
+
+<s:if test="pager.prevLink != null || pager.nextLink != null">
+    <nav class="roller-pager" aria-label="<s:text name='generic.pagination'/>">
+
+        <s:if test="pager.firstLink != null">
+            <a class="btn btn-outline-primary btn-sm" href="<s:property value='pager.firstLink'/>">
+                <span class="bi bi-chevron-double-left" aria-hidden="true"></span>
+                <s:text name="generic.first"/></a>
+        </s:if>
+        <s:else>
+            <span class="btn btn-outline-secondary btn-sm disabled" aria-disabled="true">
+                <span class="bi bi-chevron-double-left" aria-hidden="true"></span>
+                <s:text name="generic.first"/></span>
+        </s:else>
+
+        <s:if test="pager.prevLink != null">
+            <a class="btn btn-outline-primary btn-sm" href="<s:property value='pager.prevLink'/>">
+                <span class="bi bi-chevron-left" aria-hidden="true"></span>
+                <s:text name="generic.newer"/></a>
+        </s:if>
+        <s:else>
+            <span class="btn btn-outline-secondary btn-sm disabled" aria-disabled="true">
+                <span class="bi bi-chevron-left" aria-hidden="true"></span>
+                <s:text name="generic.newer"/></span>
+        </s:else>
+
+        <span class="roller-pager-page">
+            <s:text name="generic.pageNumber">
+                <s:param value="pager.pageNumber"/>
+                <s:param value="pager.pageCount"/>
+            </s:text>
+        </span>
+
+        <s:if test="pager.nextLink != null">
+            <a class="btn btn-outline-primary btn-sm" href="<s:property value='pager.nextLink'/>">
+                <s:text name="generic.older"/>
+                <span class="bi bi-chevron-right" aria-hidden="true"></span></a>
+        </s:if>
+        <s:else>
+            <span class="btn btn-outline-secondary btn-sm disabled" aria-disabled="true">
+                <s:text name="generic.older"/>
+                <span class="bi bi-chevron-right" aria-hidden="true"></span></span>
+        </s:else>
+
+        <s:if test="pager.lastLink != null">
+            <a class="btn btn-outline-primary btn-sm" href="<s:property value='pager.lastLink'/>">
+                <s:text name="generic.last"/>
+                <span class="bi bi-chevron-double-right" aria-hidden="true"></span></a>
+        </s:if>
+        <s:else>
+            <span class="btn btn-outline-secondary btn-sm disabled" aria-disabled="true">
+                <s:text name="generic.last"/>
+                <span class="bi bi-chevron-double-right" aria-hidden="true"></span></span>
+        </s:else>
+
+    </nav>
+</s:if>
diff --git a/app/src/main/webapp/WEB-INF/jsps/editor/TemplateEdit.jsp b/app/src/main/webapp/WEB-INF/jsps/editor/TemplateEdit.jsp
index ba35496294..ce721658cf 100644
--- a/app/src/main/webapp/WEB-INF/jsps/editor/TemplateEdit.jsp
+++ b/app/src/main/webapp/WEB-INF/jsps/editor/TemplateEdit.jsp
@@ -138,22 +138,20 @@
 
     <s:if test="template.custom">
 
-        <div class="accordion" id="accordion" style="margin-top:2em">
+        <div class="accordion my-4" id="accordion">
 
-        <div class="card" id="panel-plugins">
+        <div class="accordion-item" id="panel-plugins">
 
-            <div class="card-header">
+            <h2 class="accordion-header">
+                <button type="button" class="accordion-button collapsed"
+                        data-bs-toggle="collapse" data-bs-target="#collapseAdvanced"
+                        aria-expanded="false" aria-controls="collapseAdvanced">
+                    <s:text name="pageForm.advancedSettings"/>
+                </button>
+            </h2>
 
-                <h4 class="card-title">
-                    <a class="collapsed text-body text-decoration-none" data-bs-toggle="collapse" data-bs-target="#collapseAdvanced" href="#">
-                        <s:text name="pageForm.advancedSettings"/>
-                    </a>
-                </h4>
-
-            </div>
-
-            <div id="collapseAdvanced" class="collapse">
-                <div class="card-body">
+            <div id="collapseAdvanced" class="accordion-collapse collapse">
+                <div class="accordion-body">
 
                     <s:select name="bean.templateLanguage" list="templateLanguages" size="1"
                               label="%{getText('pageForm.templateLanguage')}"/>
diff --git a/app/src/main/webapp/WEB-INF/jsps/tiles/head.jsp b/app/src/main/webapp/WEB-INF/jsps/tiles/head.jsp
index 535d5824ed..5dd60291f6 100644
--- a/app/src/main/webapp/WEB-INF/jsps/tiles/head.jsp
+++ b/app/src/main/webapp/WEB-INF/jsps/tiles/head.jsp
@@ -32,6 +32,7 @@ You can override it with your own file via WEB-INF/tiles-def.xml
      the URL. Keeping the +1 here 404s and breaks the date picker/autocomplete. --%>
 <script src="<s:url value='/webjars/jquery-ui/1.14.2/jquery-ui.min.js' />"></script>
 <link href="<s:url value='/webjars/jquery-ui/1.14.2/jquery-ui.css' />" rel="stylesheet" />
+<script src="<s:url value='/roller-ui/scripts/date-fields.js' />"></script>
 
 <script src="<s:url value='/webjars/jquery-validation/1.21.0/jquery.validate.min.js' />"></script>
 
diff --git a/app/src/main/webapp/roller-ui/scripts/date-fields.js b/app/src/main/webapp/roller-ui/scripts/date-fields.js
new file mode 100644
index 0000000000..fd4e01a5e0
--- /dev/null
+++ b/app/src/main/webapp/roller-ui/scripts/date-fields.js
@@ -0,0 +1,35 @@
+/*
+* Licensed to the Apache Software Foundation (ASF) under one or more
+*  contributor license agreements.  The ASF licenses this file to You
+* under the Apache License, Version 2.0 (the "License"); you may not
+* use this file except in compliance with the License.
+* You may obtain a copy of the License at
+*
+*     http://www.apache.org/licenses/LICENSE-2.0
+*
+* Unless required by applicable law or agreed to in writing, software
+* distributed under the License is distributed on an "AS IS" BASIS,
+* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+* See the License for the specific language governing permissions and
+* limitations under the License.  For additional information regarding
+* copyright in this work, please see the NOTICE file in the top level
+* directory of this distribution.
+*/
+
+
+/*
+ * Date fields in the admin UI: a readonly input.date-picker inside an .input-group,
+ * optionally followed by a .date-picker-show button (opens the picker) and a
+ * .date-picker-clear button (empties the field, so a filter date can be removed).
+ */
+$(function () {
+    $(".date-picker").datepicker();
+
+    $(document).on("click", ".date-picker-show", function () {
+        $(this).closest(".input-group").find(".date-picker").datepicker("show");
+    });
+
+    $(document).on("click", ".date-picker-clear", function () {
+        $(this).closest(".input-group").find(".date-picker").val("").trigger("change");
+    });
+});
diff --git a/app/src/main/webapp/roller-ui/styles/roller.css b/app/src/main/webapp/roller-ui/styles/roller.css
index b8dea7b99a..f3dd485828 100644
--- a/app/src/main/webapp/roller-ui/styles/roller.css
+++ b/app/src/main/webapp/roller-ui/styles/roller.css
@@ -82,20 +82,34 @@ table.mm_table_actions td {
 
 /* edit entry page */
 
-/* Only the panel collapse toggles, not links in other toolbars that use
-   .panel-heading, such as the rich text editor's menus */
-.panel-heading a[data-toggle="collapse"]:after {
-    font-family:'Glyphicons Halflings';
-    content:"\e114";
-    float: right;
-    color: grey;
+/* Panel titles at the size of a section label, not a page heading */
+.accordion-button {
+    font-size: 1.1rem;
+    font-weight: 500;
 }
-.panel-heading a[data-toggle="collapse"].collapsed:after {
-    content:"\e080";
+
+/* entries and comments pages */
+
+/* Sidebar filter forms: the bootstrap theme adds no spacing to form-vertical groups */
+.roller-filter-form > div {
+    margin-bottom: 1rem;
+}
+
+.roller-filter-form input[type="submit"] {
+    margin-top: 0.5rem;
+}
+
+.roller-pager {
+    display: flex;
+    flex-wrap: wrap;
+    align-items: center;
+    gap: 0.75rem;
+    margin: 1rem 0;
 }
 
-#panel-plugins {      /* Kludge for EntryEditor.jsp */
-    margin-top: 6px;  /* Without this there no space between Summary and Plugins to Apply panels */
+.roller-pager-page {
+    color: var(--bs-secondary-color);
+    font-size: 0.875rem;
 }
 
 /* keep jQuery UI datepicker in front of other elements */
@@ -165,6 +179,8 @@ table.innertable tbody td {
 }
 table.innertable td.viewbody {
     padding: 5px;
+    /* spam often holds long unbroken URLs; let them wrap so the table fits the page */
+    overflow-wrap: anywhere;
 }
 table.innertable tr .bot {
     margin-bottom: 5px;
@@ -357,6 +373,12 @@ tr.rollertable_pending {
     opacity: 1;
 }
 
+/* Date fields are readonly only so the picker sets them; they are not locked */
+.form-control.date-picker[readonly] {
+    background-color: var(--bs-body-bg);
+    cursor: pointer;
+}
+
 /* Field hint icons sit next to the label rather than in the text flow. */
 .s2b_tooltip {
     margin-left: .35rem;
diff --git a/app/src/test/java/org/apache/roller/weblogger/business/CommentTest.java b/app/src/test/java/org/apache/roller/weblogger/business/CommentTest.java
index 075c1f141d..df55fd99b6 100644
--- a/app/src/test/java/org/apache/roller/weblogger/business/CommentTest.java
+++ b/app/src/test/java/org/apache/roller/weblogger/business/CommentTest.java
@@ -30,6 +30,7 @@
 import java.util.List;
 import org.junit.jupiter.api.AfterEach;
 import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
 
 import static org.junit.jupiter.api.Assertions.*;
 
@@ -85,6 +86,7 @@ public void tearDown() throws Exception {
     /**
      * Test basic persistence operations ... Create, Update, Delete
      */
+    @Test
     public void testCommentCRUD() throws Exception {
         
         WeblogEntryManager mgr = WebloggerFactory.getWeblogger().getWeblogEntryManager();
@@ -132,6 +134,7 @@ public void testCommentCRUD() throws Exception {
     /**
      * Test lookup mechanisms ... 
      */
+    @Test
     public void testCommentLookups() throws Exception {
         
         WeblogEntryManager mgr = WebloggerFactory.getWeblogger().getWeblogEntryManager();
@@ -149,6 +152,7 @@ public void testCommentLookups() throws Exception {
         comments = mgr.getComments(csc);
         assertNotNull(comments);
         assertEquals(3, comments.size());
+        assertEquals(3, mgr.getCommentCount(csc));
         
         // get all comments for entry
         testEntry = TestUtils.getManagedWeblogEntry(testEntry);
@@ -169,6 +173,7 @@ public void testCommentLookups() throws Exception {
         comments = mgr.getComments(csc);
         assertNotNull(comments);
         assertEquals(1, comments.size());
+        assertEquals(1, mgr.getCommentCount(csc));
         
         // get approved comments
         csc.setStatus(ApprovalStatus.APPROVED);
@@ -182,6 +187,8 @@ public void testCommentLookups() throws Exception {
         comments = mgr.getComments(csc);
         assertNotNull(comments);
         assertEquals(2, comments.size());
+        // the count ignores offset: it sizes the pager across all pages
+        assertEquals(3, mgr.getCommentCount(csc));
         
         // remove test comments
         TestUtils.teardownComment(comment1.getId());
@@ -197,6 +204,7 @@ public void testCommentLookups() throws Exception {
      * should delete all comments on that entry, and deleting a weblog should
      * delete all comments, etc.
      */
+    @Test
     public void testCommentParentDeletes() throws Exception {
         
         log.info("BEGIN");
diff --git a/app/src/test/java/org/apache/roller/weblogger/business/MediaFileTest.java b/app/src/test/java/org/apache/roller/weblogger/business/MediaFileTest.java
index 7e89295748..32515d5e25 100644
--- a/app/src/test/java/org/apache/roller/weblogger/business/MediaFileTest.java
+++ b/app/src/test/java/org/apache/roller/weblogger/business/MediaFileTest.java
@@ -505,6 +505,7 @@ public void testSearchMediaFile() throws Exception {
             filter1.setName("mytest.jpg");
             searchResults = mfMgr.searchMediaFiles(testWeblog, filter1);
             assertTrue(searchResults.isEmpty());
+            assertEquals(0, mfMgr.getMediaFileCount(testWeblog, filter1));
 
             MediaFileFilter filter2 = new MediaFileFilter();
             filter2.setName("test_home.jpg");
@@ -542,6 +543,7 @@ public void testSearchMediaFile() throws Exception {
             searchResults = mfMgr.searchMediaFiles(testWeblog, filter4);
             assertFalse(searchResults.isEmpty());
             assertEquals(1, searchResults.size());
+            assertEquals(1, mfMgr.getMediaFileCount(testWeblog, filter4));
             assertEquals("test_work.jpg", searchResults.get(0).getName());
 
             // search by size
@@ -576,6 +578,7 @@ public void testSearchMediaFile() throws Exception {
             searchResults = mfMgr.searchMediaFiles(testWeblog, filter9);
             assertFalse(searchResults.isEmpty());
             assertEquals(2, searchResults.size());
+            assertEquals(2, mfMgr.getMediaFileCount(testWeblog, filter9));
 
             MediaFileFilter filter10 = new MediaFileFilter();
             filter10.setSize(3000);
@@ -598,6 +601,7 @@ public void testSearchMediaFile() throws Exception {
             searchResults = mfMgr.searchMediaFiles(testWeblog, filter12);
             assertFalse(searchResults.isEmpty());
             assertEquals(2, searchResults.size());
+            assertEquals(2, mfMgr.getMediaFileCount(testWeblog, filter12));
 
         } finally {
             TestUtils.endSession(true);
@@ -668,6 +672,8 @@ public void testSearchMediaFilePaging() throws Exception {
             assertFalse(searchResults2.isEmpty());
             assertEquals(3, searchResults2.size());
             assertEquals("test_file13.jpg", searchResults2.get(0).getName());
+            // the count ignores start index and length: it sizes the pager across all pages
+            assertEquals(15, mfMgr.getMediaFileCount(testWeblog, filter2));
 
             MediaFileFilter filter3 = new MediaFileFilter();
             filter3.setSize(1000);
diff --git a/app/src/test/java/org/apache/roller/weblogger/business/WeblogEntryTest.java b/app/src/test/java/org/apache/roller/weblogger/business/WeblogEntryTest.java
index b9398d9262..04837304aa 100644
--- a/app/src/test/java/org/apache/roller/weblogger/business/WeblogEntryTest.java
+++ b/app/src/test/java/org/apache/roller/weblogger/business/WeblogEntryTest.java
@@ -209,6 +209,7 @@ public void testWeblogEntryLookups() throws Exception {
         assertNotNull(entries);
         assertEquals(5, entries.size());
         assertEquals(entry3, entries.get(0));
+        assertEquals(5, mgr.getWeblogEntryCount(wesc));
         
         // get all (non-future) PUBLISHED entries in category 
         WeblogEntrySearchCriteria wesc9 = new WeblogEntrySearchCriteria();
@@ -226,6 +227,7 @@ public void testWeblogEntryLookups() throws Exception {
         entries = mgr.getWeblogEntries(wesc2);
         assertNotNull(entries);
         assertEquals(3, entries.size());
+        assertEquals(3, mgr.getWeblogEntryCount(wesc2));
         
         // get all entries in date range
         WeblogEntrySearchCriteria wesc3 = new WeblogEntrySearchCriteria();
@@ -244,6 +246,8 @@ public void testWeblogEntryLookups() throws Exception {
         entries = mgr.getWeblogEntries(wesc4);
         assertNotNull(entries);
         assertEquals(2, entries.size());
+        // the count ignores maxResults: it sizes the pager across all pages
+        assertEquals(5, mgr.getWeblogEntryCount(wesc4));
         
         // get all entries in category
         WeblogEntrySearchCriteria wesc5 = new WeblogEntrySearchCriteria();
@@ -262,6 +266,7 @@ public void testWeblogEntryLookups() throws Exception {
         assertNotNull(entries);
         assertEquals(1, entries.size());
         assertEquals(entry2, entries.get(0));
+        assertEquals(5, mgr.getWeblogEntryCount(wesc6));
         
         // get all entries, limited by locale
         WeblogEntrySearchCriteria wesc7 = new WeblogEntrySearchCriteria();
@@ -271,6 +276,7 @@ public void testWeblogEntryLookups() throws Exception {
         assertNotNull(entries);
         assertEquals(4, entries.size());
         assertEquals(entry3, entries.get(0));
+        assertEquals(4, mgr.getWeblogEntryCount(wesc7));
         
         // get pinned entries only
         entries = mgr.getWeblogEntriesPinnedToMain(5);
@@ -575,6 +581,39 @@ public void testGetEntriesByTag() throws Exception {
     }
 
 
+    /**
+     * The entry count must match the rows getWeblogEntries() returns, even when a
+     * multi-tag search joins an entry once per matching tag, or the pager's last page is wrong.
+     */
+    @Test
+    public void testWeblogEntryCountWithMultipleTags() throws Exception {
+
+        WeblogEntryManager mgr = WebloggerFactory.getWeblogger().getWeblogEntryManager();
+
+        testWeblog = TestUtils.getManagedWebsite(testWeblog);
+        testUser = TestUtils.getManagedUser(testUser);
+        WeblogEntry entry1 = TestUtils.setupWeblogEntry("entry1", testWeblog, testUser);
+        entry1.addTag("counttagone");
+        entry1.addTag("counttagtwo");
+        mgr.saveWeblogEntry(entry1);
+        WeblogEntry entry2 = TestUtils.setupWeblogEntry("entry2", testWeblog, testUser);
+        entry2.addTag("counttagone");
+        mgr.saveWeblogEntry(entry2);
+        TestUtils.endSession(true);
+
+        testWeblog = TestUtils.getManagedWebsite(testWeblog);
+        WeblogEntrySearchCriteria wesc = new WeblogEntrySearchCriteria();
+        wesc.setWeblog(testWeblog);
+        wesc.setTags(Arrays.asList("counttagone", "counttagtwo"));
+        List<WeblogEntry> results = mgr.getWeblogEntries(wesc);
+        assertEquals(results.size(), mgr.getWeblogEntryCount(wesc));
+
+        TestUtils.teardownWeblogEntry(entry1.getId());
+        TestUtils.teardownWeblogEntry(entry2.getId());
+        TestUtils.endSession(true);
+    }
+
+
     @Test
     public void testRemoveEntryTagCascading() throws Exception {
 
diff --git a/app/src/test/java/org/apache/roller/weblogger/ui/EntryEditorAssetsTest.java b/app/src/test/java/org/apache/roller/weblogger/ui/EntryEditorAssetsTest.java
index 02d9fed204..5528aa9030 100644
--- a/app/src/test/java/org/apache/roller/weblogger/ui/EntryEditorAssetsTest.java
+++ b/app/src/test/java/org/apache/roller/weblogger/ui/EntryEditorAssetsTest.java
@@ -63,15 +63,24 @@ public void summernoteClosesItsIconElements() throws IOException {
 
     @Test
     public void panelChevronIsOnlyOnCollapseToggles() throws IOException {
+        // The entry editor's panels are Bootstrap 5 accordions, which draw their own
+        // chevron on .accordion-button. A roller.css chevron on ".panel-heading a"
+        // also lands on the rich text editor's menu items, so none may come back.
         String css = read(WEBAPP.resolve("roller-ui/styles/roller.css"));
-        Matcher rule = Pattern.compile("([^{}]*\\.panel-heading\\s+a[^{}]*:after)\\s*\\{").matcher(css);
-        boolean found = false;
-        while (rule.find()) {
-            found = true;
-            assertTrue(rule.group(1).contains("[data-toggle=\"collapse\"]"),
-                    "rule must be limited to collapse toggles: " + rule.group(1).trim());
+        assertFalse(Pattern.compile("\\.panel-heading[^{}]*:after").matcher(css).find(),
+                "roller.css must not add a chevron to .panel-heading links");
+
+        for (String jsp : new String[] {"EntryEdit.jsp", "EntryEditor.jsp"}) {
+            String page = read(WEBAPP.resolve("WEB-INF/jsps/editor/" + jsp));
+            Matcher toggle = Pattern.compile("<[^>]*data-bs-toggle=\"collapse\"[^>]*>").matcher(page);
+            boolean found = false;
+            while (toggle.find()) {
+                found = true;
+                assertTrue(toggle.group().contains("accordion-button"),
+                        jsp + " collapse toggle must be an accordion button: " + toggle.group());
+            }
+            assertTrue(found, jsp + " has collapsible panels");
         }
-        assertTrue(found, "roller.css still styles the panel collapse toggles");
     }
 
     private static String read(Path file) throws IOException {
diff --git a/app/src/test/java/org/apache/roller/weblogger/ui/struts2/pagers/LinkPagerTest.java b/app/src/test/java/org/apache/roller/weblogger/ui/struts2/pagers/LinkPagerTest.java
new file mode 100644
index 0000000000..01a0d2c49b
--- /dev/null
+++ b/app/src/test/java/org/apache/roller/weblogger/ui/struts2/pagers/LinkPagerTest.java
@@ -0,0 +1,98 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ *  contributor license agreements.  The ASF licenses this file to You
+ * under the Apache License, Version 2.0 (the "License"); you may not
+ * use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.  For additional information regarding
+ * copyright in this work, please see the NOTICE file in the top level
+ * directory of this distribution.
+ */
+
+package org.apache.roller.weblogger.ui.struts2.pagers;
+
+import java.util.Collections;
+
+import org.junit.jupiter.api.Test;
+
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertFalse;
+import static org.junit.jupiter.api.Assertions.assertNull;
+import static org.junit.jupiter.api.Assertions.assertTrue;
+
+
+public class LinkPagerTest {
+
+    private static final String URL = "/roller-ui/authoring/entries.rol?weblog=roller";
+
+    private static LinkPager<String> pager(int page, long total) {
+        return new LinkPager<>(URL, page, Collections.emptyList(), total, 30);
+    }
+
+    @Test
+    public void testEmptyResultIsOnePage() {
+        LinkPager<String> p = pager(0, 0);
+        assertEquals(1, p.getPageNumber());
+        assertEquals(1, p.getPageCount());
+        assertFalse(p.isMoreItems());
+        assertNull(p.getFirstLink());
+        assertNull(p.getPrevLink());
+        assertNull(p.getNextLink());
+        assertNull(p.getLastLink());
+    }
+
+    @Test
+    public void testFirstPage() {
+        LinkPager<String> p = pager(0, 65);
+        assertEquals(1, p.getPageNumber());
+        assertEquals(3, p.getPageCount());
+        assertTrue(p.isMoreItems());
+        assertNull(p.getFirstLink());
+        assertNull(p.getPrevLink());
+        assertEquals(URL + "&bean.page=1", p.getNextLink());
+        assertEquals(URL + "&bean.page=2", p.getLastLink());
+    }
+
+    @Test
+    public void testMiddlePage() {
+        LinkPager<String> p = pager(1, 65);
+        assertEquals(2, p.getPageNumber());
+        assertEquals(URL + "&bean.page=0", p.getFirstLink());
+        assertEquals(URL + "&bean.page=0", p.getPrevLink());
+        assertEquals(URL + "&bean.page=2", p.getNextLink());
+        assertEquals(URL + "&bean.page=2", p.getLastLink());
+    }
+
+    @Test
+    public void testLastPage() {
+        LinkPager<String> p = pager(2, 65);
+        assertEquals(3, p.getPageNumber());
+        assertFalse(p.isMoreItems());
+        assertEquals(URL + "&bean.page=0", p.getFirstLink());
+        assertEquals(URL + "&bean.page=1", p.getPrevLink());
+        assertNull(p.getNextLink());
+        assertNull(p.getLastLink());
+    }
+
+    @Test
+    public void testExactMultipleOfPageSize() {
+        // 60 rows is exactly two pages, not three
+        assertEquals(2, pager(0, 60).getPageCount());
+        assertFalse(pager(1, 60).isMoreItems());
+        assertEquals(URL + "&bean.page=1", pager(0, 60).getLastLink());
+    }
+
+    @Test
+    public void testBaseUrlWithoutQueryString() {
+        LinkPager<String> p = new LinkPager<>("/entries.rol", 0, Collections.emptyList(), 31, 30);
+        assertEquals("/entries.rol?bean.page=1", p.getNextLink());
+    }
+
+}

From 844b3913971a883a5e77bfb269ba4cc905212969 Mon Sep 17 00:00:00 2001
From: "David M. Johnson" <snoopdave@apache.org>
Date: Sun, 25 Oct 2026 14:09:55 -0400
Subject: [PATCH 2/4] Admin UI: space between stacked cards; smaller type scale

Bootstrap 5 sizes everything for a 16px root, which reads large in the
admin UI; a 15px root scales text, controls and spacing down together.
Page titles and sidebar headings get smaller sizes, and cards stacked in
a column (sidebar cards, setup steps) no longer touch.
---
 .../main/webapp/roller-ui/styles/roller.css   | 22 +++++++++++++++++++
 1 file changed, 22 insertions(+)

diff --git a/app/src/main/webapp/roller-ui/styles/roller.css b/app/src/main/webapp/roller-ui/styles/roller.css
index f3dd485828..8e4bde61db 100644
--- a/app/src/main/webapp/roller-ui/styles/roller.css
+++ b/app/src/main/webapp/roller-ui/styles/roller.css
@@ -30,9 +30,31 @@ p {
     margin: 1em;
 }
 
+/* Bootstrap 5 sizes everything for a 16px root, which reads large in this
+   dense admin UI; 15px scales text, controls and spacing down together. */
+html {
+    font-size: 15px;
+}
+
 .roller-page-title {
     margin-top: 0px;
     margin-bottom: 5px;
+    font-size: 1.75rem;
+}
+
+/* Section headings in sidebar cards */
+.roller-column-left h3 {
+    font-size: 1.25rem;
+}
+
+.roller-column-left h4 {
+    font-size: 1.1rem;
+}
+
+/* Cards stacked in a column (sidebar cards, setup steps, upload results)
+   would otherwise touch */
+.card + .card {
+    margin-top: 1rem;
 }
 
 .roller-column-right {

From 61821744fc5997ae2fd75e5e4d83d0c1ffe8877c Mon Sep 17 00:00:00 2001
From: "David M. Johnson" <snoopdave@apache.org>
Date: Sun, 25 Oct 2026 15:07:40 -0400
Subject: [PATCH 3/4] UI tests: open the entry editor's advanced settings by
 its accordion button

The accordion headers are Bootstrap 5 buttons now, not links.
---
 .../src/test/java/org/apache/roller/playwright/BaseIT.java      | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/it-playwright/src/test/java/org/apache/roller/playwright/BaseIT.java b/it-playwright/src/test/java/org/apache/roller/playwright/BaseIT.java
index 158f70f208..2c616cb75a 100644
--- a/it-playwright/src/test/java/org/apache/roller/playwright/BaseIT.java
+++ b/it-playwright/src/test/java/org/apache/roller/playwright/BaseIT.java
@@ -349,7 +349,7 @@ protected void insertMediaFileThroughDialog(String weblogHandle) {
      * stacked form rows.
      */
     protected void assertPublishingTimeOnOneLine() {
-        page.locator("a[href='#collapseAdvanced']").click();
+        page.locator("[data-bs-target='#collapseAdvanced']").click();
         assertThat(page.locator("#collapseAdvanced")).isVisible();
         double hours = page.locator("select[name='bean.hours']").boundingBox().y;
         double minutes = page.locator("select[name='bean.minutes']").boundingBox().y;

From 1a05350b80569ece6317745e9da48035a1664667 Mon Sep 17 00:00:00 2001
From: "David M. Johnson" <snoopdave@apache.org>
Date: Sun, 25 Oct 2026 20:41:26 -0400
Subject: [PATCH 4/4] UI tests: measure the publishing time selects after the
 accordion opens

The check read each select's position separately while the collapse was
still animating, so the y values drifted apart on slower runners.
---
 .../java/org/apache/roller/playwright/BaseIT.java   | 13 +++++++++----
 1 file changed, 9 insertions(+), 4 deletions(-)

diff --git a/it-playwright/src/test/java/org/apache/roller/playwright/BaseIT.java b/it-playwright/src/test/java/org/apache/roller/playwright/BaseIT.java
index 2c616cb75a..05859f2757 100644
--- a/it-playwright/src/test/java/org/apache/roller/playwright/BaseIT.java
+++ b/it-playwright/src/test/java/org/apache/roller/playwright/BaseIT.java
@@ -350,10 +350,15 @@ protected void insertMediaFileThroughDialog(String weblogHandle) {
      */
     protected void assertPublishingTimeOnOneLine() {
         page.locator("[data-bs-target='#collapseAdvanced']").click();
-        assertThat(page.locator("#collapseAdvanced")).isVisible();
-        double hours = page.locator("select[name='bean.hours']").boundingBox().y;
-        double minutes = page.locator("select[name='bean.minutes']").boundingBox().y;
-        double seconds = page.locator("select[name='bean.seconds']").boundingBox().y;
+        // Bootstrap adds "show" only when the collapse transition ends; read
+        // all three positions at once so the layout cannot move between reads
+        assertThat(page.locator("#collapseAdvanced.show:not(.collapsing)")).isVisible();
+        @SuppressWarnings("unchecked")
+        var ys = (java.util.List<Number>) page.evaluate("() => ['hours', 'minutes', 'seconds'].map(f =>"
+                + " document.querySelector(`select[name='bean.${f}']`).getBoundingClientRect().y)");
+        double hours = ys.get(0).doubleValue();
+        double minutes = ys.get(1).doubleValue();
+        double seconds = ys.get(2).doubleValue();
         org.junit.jupiter.api.Assertions.assertTrue(
                 Math.abs(hours - minutes) < 2 && Math.abs(minutes - seconds) < 2,
                 "publishing time selects should share a line, but sit at y=" + hours + ", " + minutes + ", " + seconds);