oldnews 0.6.0__py3-none-any.whl → 0.7.0__py3-none-any.whl

oldnews/widgets/_after_highlight.py

@@ -45,7 +45,7 @@ def options_after_highlight[T](
     highlight = (len(options) - highlight - 1) if direction == "previous" else highlight
     return (
         option
-        for option in [*options[highlight:], *options[0:highlight]][1:]
+        for option in [*options[highlight:], *options[:highlight]][1:]
         if option_filter(option)
     )
 

oldnews/widgets/navigation.py

@@ -7,7 +7,7 @@ from __future__ import annotations
 ##############################################################################
 # Python imports.
 from dataclasses import dataclass
-from typing import cast
+from typing import Callable, Iterable, Iterator, cast
 
 ##############################################################################
 # OldAs imports.
@@ -157,41 +157,86 @@ class Navigation(EnhancedOptionList):
         self._expanded = get_navigation_state()
         """The IDs of the folders that are expanded."""
 
-    def _add_subscriptions(self, parent_folder: str) -> None:
-        """Add the subscriptions for a given parent folder.
+    @staticmethod
+    def _key(attr: str) -> Callable[[object], str]:
+        """Create a key to use with `sorted`.
 
         Args:
-            parent_folder: The parent folder to add the subscriptions for.
+            attr: The attribute to sort on.
+
+        Returns:
+            A function to get a `casefold` version of the attribute.
+        """
+
+        def _casefold(value: object) -> str:
+            return cast(str, getattr(value, attr)).casefold()
+
+        return _casefold
+
+    def _viewable(
+        self, subscriptions: Iterable[Subscription]
+    ) -> Iterator[SubscriptionView]:
+        """Given a iterable of subscriptions, make them viewable.
+
+        Args:
+            subscriptions: The subscriptions to make viewable.
+
+        Yields:
+            Views of the subscriptions.
         """
-        for subscription in self.subscriptions:
-            if any(
-                category.id == parent_folder for category in subscription.categories
-            ):
-                self.add_option(SubscriptionView(subscription, self.unread))
+        yield from (
+            SubscriptionView(subscription, self.unread)
+            for subscription in sorted(subscriptions, key=self._key("title"))
+        )
 
-    def _add_folder(self, folder: Folder) -> None:
-        """Add the given folder to the navigation.
+    def _gather_subscriptions_for_folder(
+        self, parent_folder: Folder
+    ) -> Iterator[SubscriptionView]:
+        """Gather the subscriptions for a given parent folder.
 
         Args:
-            folder: The folder to add.
+            parent_folder: The parent folder to add the subscriptions for.
+
+        Yields:
+            The subscriptions within that folder.
+        """
+        yield from self._viewable(
+            subscription
+            for subscription in self.subscriptions
+            if parent_folder in subscription.categories
+        )
+
+    def _gather_folders(self) -> Iterator[FolderView | SubscriptionView]:
+        """Gather up all the folders and their subscriptions.
+
+        Yields:
+            Folder and subscription options.
+        """
+        for folder in sorted(self.folders, key=self._key("name")):
+            yield FolderView(
+                folder, expanded := folder.id in self._expanded, self.unread
+            )
+            if expanded:
+                yield from self._gather_subscriptions_for_folder(folder)
+
+    def _gather_folderless_subscrtiptions(self) -> Iterator[SubscriptionView]:
+        """Gather up all the subscriptions that don't live in a folder.
+
+        Yields:
+            Subscription options for folderless subscriptions.
         """
-        self.add_option(
-            FolderView(folder, expanded := folder.id in self._expanded, self.unread)
+        yield from self._viewable(
+            subscription
+            for subscription in self.subscriptions
+            if not subscription.categories
         )
-        if expanded:
-            self._add_subscriptions(folder.id)
 
     def _refresh_navigation(self) -> None:
         """Refresh the content of the navigation widget."""
         with self.preserved_highlight:
-            self.clear_options()
-            # First off, add subscriptions that lack a folder.
-            for subscription in self.subscriptions:
-                if not subscription.categories:
-                    self.add_option(SubscriptionView(subscription, self.unread))
-            # Now add all the subscriptions that are within folders.
-            for folder in self.folders:
-                self._add_folder(folder)
+            self.set_options(
+                (*self._gather_folderless_subscrtiptions(), *self._gather_folders())
+            )
 
     def _watch_folders(self) -> None:
         """React to the folders being updated."""
@@ -205,6 +250,25 @@ class Navigation(EnhancedOptionList):
         """React to the unread data being updated."""
         self._refresh_navigation()
 
+    @work(thread=True)
+    def _save_state(self, state: set[str]) -> None:
+        """Save the folder expanded/collapsed state.
+
+        Args:
+            state: The state to save.
+        """
+        save_navigation_state(state)
+
+    def _set_expansion(self, new_state: set[str]) -> None:
+        """Set the new navigation state.
+
+        Args:
+            new_state: The new state to set.
+        """
+        self._expanded = new_state
+        self._save_state(new_state)
+        self._refresh_navigation()
+
     def _action_toggle_folder(self) -> None:
         """Action that toggles the expanded state of a folder."""
         if self.highlighted is None:
@@ -215,26 +279,15 @@ class Navigation(EnhancedOptionList):
             self.notify("Only folders can be collapsed/expanded", severity="warning")
             return
         if option.folder.id is not None:
-            self._expanded ^= {option.folder.id}
-            self._save_state()
-            self._refresh_navigation()
+            self._set_expansion(self._expanded ^ {option.folder.id})
 
     def _action_expand_all(self) -> None:
         """Action that expands all folders."""
-        self._expanded = {folder.id for folder in self.folders}
-        self._save_state()
-        self._refresh_navigation()
+        self._set_expansion({folder.id for folder in self.folders})
 
     def _action_collapse_all(self) -> None:
         """Action that collapses all folders."""
-        self._expanded = set()
-        self._save_state()
-        self._refresh_navigation()
-
-    @work(thread=True)
-    def _save_state(self) -> None:
-        """Save the folder expanded/collapsed state."""
-        save_navigation_state(self._expanded)
+        self._set_expansion(set())
 
     @on(EnhancedOptionList.OptionSelected)
     def _handle_selection(self, message: EnhancedOptionList.OptionSelected) -> None:

{oldnews-0.6.0.dist-info → oldnews-0.7.0.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: oldnews
-Version: 0.6.0
+Version: 0.7.0
 Summary: A terminal-based client for TheOldReader
 Keywords: atom,client,Google Reader,RSS,TheOldReader,terminal,news-reader,news
 Author: Dave Pearson
@@ -15,7 +15,7 @@ Classifier: Programming Language :: Python :: 3.13
 Classifier: Programming Language :: Python :: 3.14
 Classifier: Typing :: Typed
 Requires-Dist: html-to-markdown>=2.15.0
-Requires-Dist: oldas>=0.6.0
+Requires-Dist: oldas>=0.10.0
 Requires-Dist: textual>=6.3.0
 Requires-Dist: textual-autocomplete>=4.0.6
 Requires-Dist: textual-enhanced>=1.2.0

{oldnews-0.6.0.dist-info → oldnews-0.7.0.dist-info}/RECORD

@@ -29,11 +29,11 @@ oldnews/screens/main.py,sha256=AuCL54UC-DHl-fyRTr1_lSS2hCBXQKizQirpWpLBazs,28618
 oldnews/screens/new_subscription.py,sha256=bk01RXWu3kvJbtm88o1mrYpll73Op3PWWl7ggIH29kk,4011
 oldnews/sync.py,sha256=OZv5JvImvzPAjV6cuCbiSoIeVy0TnfWAq-rL-Xs_ffk,10877
 oldnews/widgets/__init__.py,sha256=5VSjKswHxv2W8g0O-LWlUFS-gy7iQrIM8PGvSBhDNLU,438
-oldnews/widgets/_after_highlight.py,sha256=4GgMx0SYU05K3K7xFk22N6ZSn5wewnAjA062mQT6jBg,1904
+oldnews/widgets/_after_highlight.py,sha256=cUJXUXElKfDrTsmy2WzKqGcxu67mlTamRBSi-VM_ddQ,1903
 oldnews/widgets/article_content.py,sha256=pv8zobd3eTsALHYmP854dd_enqy32QXGoWXEILYnWfA,3553
 oldnews/widgets/article_list.py,sha256=dSmSGlk1X94ryy0bJ6ejc2WXO0XMvTWyTxEtR1cL_5k,7428
-oldnews/widgets/navigation.py,sha256=QE_jyavY-A4mugEmFVOo1Ed7g9GfK4wEkrnjO5YQLtA,11019
-oldnews-0.6.0.dist-info/WHEEL,sha256=XV0cjMrO7zXhVAIyyc8aFf1VjZ33Fen4IiJk5zFlC3g,80
-oldnews-0.6.0.dist-info/entry_points.txt,sha256=FxY6Y4IsHZubhtdd0QJG3p2kCTcXe0e-Ib_AW0qkotE,51
-oldnews-0.6.0.dist-info/METADATA,sha256=vVA5ZPZ3r4E1rGLVg9ibHGeZwJ62j-zAakuoVRZ6_m4,2917
-oldnews-0.6.0.dist-info/RECORD,,
+oldnews/widgets/navigation.py,sha256=8GMjJtLFnvc9GJH7_ufChLsywPQno7HwWmBIrEnDjgo,12433
+oldnews-0.7.0.dist-info/WHEEL,sha256=e_m4S054HL0hyR3CpOk-b7Q7fDX6BuFkgL5OjAExXas,80
+oldnews-0.7.0.dist-info/entry_points.txt,sha256=FxY6Y4IsHZubhtdd0QJG3p2kCTcXe0e-Ib_AW0qkotE,51
+oldnews-0.7.0.dist-info/METADATA,sha256=BHH_29q28IZdWENsHZ7DpnqXBxoNDznqhJK2Vg-3JQk,2918
+oldnews-0.7.0.dist-info/RECORD,,

{oldnews-0.6.0.dist-info → oldnews-0.7.0.dist-info}/WHEEL

@@ -1,4 +1,4 @@
 Wheel-Version: 1.0
-Generator: uv 0.9.26
+Generator: uv 0.9.27
 Root-Is-Purelib: true
 Tag: py3-none-any