From a268bafb2b1190904192dc802452c38c1e2cd25b Mon Sep 17 00:00:00 2001 From: Alexander Blanchard Date: Tue, 25 Jun 2024 15:51:14 +0100 Subject: [PATCH 1/3] fix: Add toolbar to Summon Results Add serch tools to top of Summon Results --- .../themes/responsive/Summon/list.tpl | 67 ++++++++----------- 1 file changed, 28 insertions(+), 39 deletions(-) diff --git a/code/web/interface/themes/responsive/Summon/list.tpl b/code/web/interface/themes/responsive/Summon/list.tpl index f4dcb947bf..79e1ecfb88 100644 --- a/code/web/interface/themes/responsive/Summon/list.tpl +++ b/code/web/interface/themes/responsive/Summon/list.tpl @@ -1,4 +1,4 @@ -

{translate text='Events Search Results' isPublicFacing=true}

+

{translate text='Articles & Databases Search Results'}

{* Recommendations *} {if !empty($topRecommendations)} @@ -7,9 +7,13 @@ {/foreach} {/if} - {* Information about the search *}
- + {* User's viewing mode toggle switch *} + {if !empty($showSearchToolsAtTop)} + {include file="Search/search-toolbar.tpl"} + {else} + {include file="Search/results-displayMode-toggle.tpl"} + {/if} {if !empty($replacementTerm)}
{translate text="Showing Results for" isPublicFacing=true} {$replacementTerm}
@@ -38,12 +42,8 @@
{/if} - {* User's viewing mode toggle switch *} - {include file="Events/results-displayMode-toggle.tpl"} -
- {* End Listing Options *} {if !empty($subpage)} {include file=$subpage} @@ -51,39 +51,29 @@ {$pageContent} {/if} - {if $displayMode == 'covers'} - {if $recordEnd < $recordCount} - -
- -
- - {/if} - {else} - {if !empty($pageLinks.all)}
{$pageLinks.all}
{/if} - {/if} + {if !empty($pageLinks.all)}
{$pageLinks.all}
{/if} - {if $showSearchTools || ($loggedIn && count($userPermissions) > 0)} -
- {translate text='Search Tools' isPublicFacing=true} - {if !empty($showSearchTools) && (empty($offline) || $enableEContentWhileOffline)} - {translate text='Email this Search' isPublicFacing=true} - {if !empty($enableSavedSearches)} - {if !empty($savedSearch)} - {translate text="Remove Saved Search" isPublicFacing=true} - {else} - {translate text='Save Search' isPublicFacing=true} + {if !empty($showSearchTools) && !$showSearchToolsAtTop} +
+ {translate text='Search Tools' isPublicFacing=true} + {translate text='Get RSS Feed' isPublicFacing=true} + {if empty($offline) || $enableEContentWhileOffline} + {translate text='Email this Search' isPublicFacing=true} + {if !empty($enableSavedSearches)} + {if !empty($savedSearch)} + {translate text="Remove Saved Search" isPublicFacing=true} + {else} + {translate text='Save Search' isPublicFacing=true} + {/if} {/if} {/if} - {*{translate text='Export To CSV' isPublicFacing=true}*} - {/if} - {if !empty($loggedIn) && (in_array('Administer All Collection Spotlights', $userPermissions) || in_array('Administer Library Collection Spotlights', $userPermissions))} - {translate text='Create Spotlight' isAdminFacing=true} - {/if} - {if !empty($loggedIn) && (in_array('Administer All Browse Categories', $userPermissions) || in_array('Administer Library Browse Categories', $userPermissions) || in_array('Administer Selected Browse Category Groups', $userPermissions))} - {translate text='Add To Browse' isPublicFacing=true} - {/if} -
+ {if !empty($loggedIn) && (in_array('Administer All Collection Spotlights', $userPermissions) || in_array('Administer Library Collection Spotlights', $userPermissions))} + {translate text='Create Spotlight' isAdminFacing=true} + {/if} + {if !empty($loggedIn) && (in_array('Administer All Browse Categories', $userPermissions) || in_array('Administer Library Browse Categories', $userPermissions) || in_array('Administer Selected Browse Category Groups', $userPermissions))} + {translate text='Add To Browse' isPublicFacing=true} + {/if} +
{/if}
@@ -105,6 +95,5 @@ Globals.opac = 1; {* set to true to keep opac browsers from storing browse mode *} {/if} $('#'+AspenDiscovery.Searches.displayMode).parent('label').addClass('active'); {* show user which one is selected *} - {rdelim}); - \ No newline at end of file + From f1823db0b356f06f14f9dcf3030254eeb04ee5ba Mon Sep 17 00:00:00 2001 From: Alexander Blanchard Date: Mon, 22 Jul 2024 09:59:17 +0100 Subject: [PATCH 2/3] refactor: remove option from top of results Remove the covers and lists options view from top of resutls for Summon. --- .../responsive/Search/results-displayMode-toggle.tpl | 8 -------- .../interface/themes/responsive/Search/search-toolbar.tpl | 8 -------- 2 files changed, 16 deletions(-) diff --git a/code/web/interface/themes/responsive/Search/results-displayMode-toggle.tpl b/code/web/interface/themes/responsive/Search/results-displayMode-toggle.tpl index 52f8723105..552e3e9bf8 100644 --- a/code/web/interface/themes/responsive/Search/results-displayMode-toggle.tpl +++ b/code/web/interface/themes/responsive/Search/results-displayMode-toggle.tpl @@ -15,14 +15,6 @@
-
- - -
-
- - -
{if $showSearchTools || ($loggedIn && count($userPermissions) > 0)}
From cf08f687c971abe351ce2061a0dd885f1b341453 Mon Sep 17 00:00:00 2001 From: Alexander Blanchard Date: Mon, 22 Jul 2024 10:03:12 +0100 Subject: [PATCH 3/3] chore: update change log --- code/web/release_notes/24.08.00.MD | 1 + 1 file changed, 1 insertion(+) diff --git a/code/web/release_notes/24.08.00.MD b/code/web/release_notes/24.08.00.MD index 5f26f8054d..3cfa227a48 100644 --- a/code/web/release_notes/24.08.00.MD +++ b/code/web/release_notes/24.08.00.MD @@ -39,6 +39,7 @@ // alexander ### Summon Updates - Adjust code to ensure that the filter applies correctly no matter what page of results users are on when they apply it. (*AB*) +- Remove unimplemented option to change search results from List to Covers mode. (*AB*) // jacob