Merged in DSC-1207 (pull request DSpace#796)

[DSC-1207] fixed “search tools” button behavior Approved-by: Andrea Barbasso Approved-by: Vincenzo Mecca
4Science · Sep 20, 2023 · baa76ca · baa76ca
2 parents 3495ce0 + 7ef26d4
commit baa76ca
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/app/shared/search/search-sidebar/search-sidebar.component.html b/src/app/shared/search/search-sidebar/search-sidebar.component.html
@@ -1,4 +1,4 @@
-<div>
+<div class="bg-white">
     <div id="sidebar-options" class="d-block d-md-none search-controls clearfix">
         <small class="results">{{resultCount}} {{"search.sidebar.results" | translate}}</small>
         <button (click)="toggleSidebar.emit()"

diff --git a/src/app/shared/sidebar/sidebar.service.ts b/src/app/shared/sidebar/sidebar.service.ts
@@ -35,11 +35,11 @@ export class SidebarService {
    * @returns {Observable<boolean>} Emits true if the user's screen size is mobile or when the state in the store is currently collapsed
    */
   get isCollapsed(): Observable<boolean> {
-    return observableCombineLatest(
+    return observableCombineLatest([
       this.isXsOrSm$,
       this.isCollapsedInStore
-    ).pipe(
-      map(([mobile, store]) => mobile || store)
+    ]).pipe(
+      map(([mobile, store]) => mobile && store)
     );
   }