Search UI overhaul

cozy/app_controller.py

@@ -51,12 +51,12 @@ def __init__(self, gtk_app, main_window_builder, main_window):
 
         self.whats_new_window: WhatsNewWindow = WhatsNewWindow()
 
-        self.library_view: LibraryView = LibraryView(main_window_builder)
         self.app_view: AppView = AppView(main_window_builder)
-        self.search_view: SearchView = SearchView()
-        self.book_detail_view: BookDetailView = BookDetailView(main_window_builder)
         self.headerbar: Headerbar = Headerbar(main_window_builder)
+        self.library_view: LibraryView = LibraryView(main_window_builder)
+        self.book_detail_view: BookDetailView = BookDetailView(main_window_builder)
         self.media_controller: MediaController = MediaController(main_window_builder)
+        self.search_view: SearchView = SearchView(main_window_builder, self.headerbar)
 
         self.library_view_model = inject.instance(LibraryViewModel)
         self.app_view_model = inject.instance(AppViewModel)
@@ -68,7 +68,7 @@ def __init__(self, gtk_app, main_window_builder, main_window):
         self.settings_view_model = inject.instance(SettingsViewModel)
         self.player = inject.instance(Player)
 
-        self._connect_popovers()
+        self._connect_search_button()
 
         self.search_view_model.add_listener(self._on_open_view)
         self.book_detail_view_model.add_listener(self._on_open_view)
@@ -125,8 +125,11 @@ def open_library(self):
         self.library_view_model.open_library()
         self.app_view_model.view = View.LIBRARY_FILTER
 
-    def _connect_popovers(self):
-        self.headerbar.search_button.set_popover(self.search_view.popover)
+    def _connect_search_button(self):
+        self.headerbar.search_button.connect(
+            "notify::active",
+            self.search_view.on_state_changed
+        )
 
     def _on_open_view(self, event, data):
         if event == OpenView.AUTHOR:

cozy/extensions/set.py

@@ -1,16 +1,5 @@
 import re
-from typing import Set
 
 
-def split_strings_to_set(set_to_split: Set[str]):
-    finished = set()
-    for entry in set_to_split:
-        results = re.split(",|;|/|&", entry)
-        results = {
-            entry.strip()
-            for entry in results
-        }
-
-        finished.update(results)
-
-    return finished
+def split_strings_to_set(set_to_split: set[str]) -> set[str]:
+    return {entry.strip() for item in set_to_split for entry in re.split(",|;|/|&", item)}

cozy/ui/delete_book_view.py

@@ -1,14 +1,14 @@
 from gi.repository import Adw, Gtk
 
 from cozy.ext import inject
-from cozy.control.artwork_cache import ArtworkCache
+from cozy.model.book import Book
+from cozy.ui.widgets.book_row import BookRow
 
 
 class DeleteBookView(Adw.MessageDialog):
     main_window = inject.attr("MainWindow")
-    artwork_cache: ArtworkCache = inject.attr(ArtworkCache)
 
-    def __init__(self, callback, book):
+    def __init__(self, callback, book: Book):
         super().__init__(
             heading=_("Delete Audiobook?"),
             body=_("The audiobook will be removed from your disk and from Cozy's library."),
@@ -22,22 +22,8 @@ def __init__(self, callback, book):
         self.add_response("delete", _("Remove Audiobook"))
         self.set_response_appearance("delete", Adw.ResponseAppearance.DESTRUCTIVE)
 
-        book_row = Adw.ActionRow(
-            title=book.name,
-            subtitle=book.author,
-            selectable=False,
-            use_markup=False,
-        )
-        album_art = Gtk.Picture(margin_top=6, margin_bottom=6)
-
-        paintable = self.artwork_cache.get_cover_paintable(book, 1, 48)
-        if paintable:
-            album_art.set_paintable(paintable)
-            book_row.add_prefix(album_art)
-
         list_box = Gtk.ListBox(margin_top=12, css_classes=["boxed-list"])
-        list_box.append(book_row)
+        list_box.append(BookRow(book))
         self.set_extra_child(list_box)
 
         self.connect("response", callback, book)
-

cozy/ui/headerbar.py

@@ -6,19 +6,22 @@
 from cozy.ui.widgets.progress_popover import ProgressPopover
 from cozy.view_model.headerbar_view_model import HeaderbarViewModel, HeaderBarState
 
-from gi.repository import Adw, Gtk
+from gi.repository import Adw, Gtk, GObject
 
 log = logging.getLogger("Headerbar")
 
 COVER_SIZE = 45
 
 
 @Gtk.Template.from_resource('/com/github/geigi/cozy/headerbar.ui')
-class Headerbar(Adw.Bin):
+class Headerbar(Gtk.Box):
     __gtype_name__ = "Headerbar"
 
     headerbar: Adw.HeaderBar = Gtk.Template.Child()
 
+    search_bar: Gtk.SearchBar = Gtk.Template.Child()
+    search_entry: Gtk.SearchEntry = Gtk.Template.Child()
+
     show_sidebar_button: Gtk.ToggleButton = Gtk.Template.Child()
     search_button: Gtk.MenuButton = Gtk.Template.Child()
     menu_button: Gtk.MenuButton = Gtk.Template.Child()
@@ -44,6 +47,9 @@ def __init__(self, main_window_builder: Gtk.Builder):
         self.progress_popover = ProgressPopover()
         self.progress_menu_button.set_popover(self.progress_popover)
 
+        self.search_bar.connect_entry(self.search_entry)
+        self.search_bar.set_key_capture_widget(self.header_container)
+
         self._headerbar_view_model: HeaderbarViewModel = inject.instance(HeaderbarViewModel)
         self._connect_view_model()
         self._connect_widgets()
@@ -64,6 +70,7 @@ def _on_sort_stack_changed(self, widget, _):
         page = widget.props.visible_child_name
 
         self.show_sidebar_button.set_visible(page != "recent")
+        self.search_button.set_active(False)
 
     def _on_mobile_view(self, widget, _):
         if widget.props.reveal: