chore: Release

Cargo.lock

@@ -3001,7 +3001,7 @@ dependencies = [
 
 [[package]]
 name = "letterbox-notmuch"
-version = "0.10.13"
+version = "0.12.0"
 dependencies = [
  "itertools",
  "log",
@@ -3016,14 +3016,14 @@ dependencies = [
 
 [[package]]
 name = "letterbox-procmail2notmuch"
-version = "0.10.13"
+version = "0.12.0"
 dependencies = [
  "anyhow",
 ]
 
 [[package]]
 name = "letterbox-server"
-version = "0.10.13"
+version = "0.12.0"
 dependencies = [
  "ammonia",
  "anyhow",
@@ -3065,7 +3065,7 @@ dependencies = [
 
 [[package]]
 name = "letterbox-shared"
-version = "0.10.13"
+version = "0.12.0"
 dependencies = [
  "build-info",
  "letterbox-notmuch",
@@ -3074,7 +3074,7 @@ dependencies = [
 
 [[package]]
 name = "letterbox-web"
-version = "0.10.13"
+version = "0.12.0"
 dependencies = [
  "build-info",
  "build-info-build",

Cargo.toml

@@ -8,7 +8,7 @@ authors = ["Bill Thiede <git@xinu.tv>"]
 edition = "2021"
 license = "UNLICENSED"
 publish = ["xinu"]
-version = "0.10.13"
+version = "0.12.0"
 repository = "https://git.z.xinu.tv/wathiede/letterbox"
 
 [profile.dev]

server/Cargo.toml

@@ -47,8 +47,8 @@ urlencoding = "2.1.3"
 #xtracing = { path = "../../xtracing" }
 #xtracing = { git = "http://git-private.h.xinu.tv/wathiede/xtracing.git" }
 xtracing = { version = "0.3.0", registry = "xinu" }
-letterbox-notmuch = { version = "0.10.13", path = "../notmuch", registry = "xinu" }
-letterbox-shared = { version = "0.10.13", path = "../shared", registry = "xinu" }
+letterbox-notmuch = { version = "0.12.0", path = "../notmuch", registry = "xinu" }
+letterbox-shared = { version = "0.12.0", path = "../shared", registry = "xinu" }
 
 [build-dependencies]
 build-info-build = "0.0.40"

shared/Cargo.toml

@@ -12,5 +12,5 @@ version.workspace = true
 
 [dependencies]
 build-info = "0.0.40"
-letterbox-notmuch = { version = "0.10.13", path = "../notmuch", registry = "xinu" }
+letterbox-notmuch = { version = "0.12.0", path = "../notmuch", registry = "xinu" }
 serde = { version = "1.0.147", features = ["derive"] }

web/Cargo.toml

@@ -33,8 +33,8 @@ wasm-bindgen = "=0.2.100"
 uuid = { version = "1.13.1", features = [
   "js",
 ] } # direct dep to set js feature, prevents Rng issues
-letterbox-shared = { version = "0.10.13", path = "../shared", registry = "xinu" }
-letterbox-notmuch = { version = "0.10.13", path = "../notmuch", registry = "xinu" }
+letterbox-shared = { version = "0.12.0", path = "../shared", registry = "xinu" }
+letterbox-notmuch = { version = "0.12.0", path = "../notmuch", registry = "xinu" }
 seed_hooks = { version = "0.4.0", registry = "xinu" }
 strum_macros = "0.27.1"
 

web/src/state.rs

@@ -616,9 +616,6 @@ pub fn update(msg: Msg, model: &mut Model, orders: &mut impl Orders<Msg>) {
             orders.send_msg(Msg::CatchupRequest { query });
         }
         Msg::CatchupKeepUnread => {
-            if let Some(thread_id) = current_thread_id(&model.context) {
-                orders.send_msg(Msg::SetUnread(thread_id, true));
-            };
             orders.send_msg(Msg::CatchupNext);
         }
         Msg::CatchupMarkAsRead => {
@@ -633,7 +630,15 @@ pub fn update(msg: Msg, model: &mut Model, orders: &mut impl Orders<Msg>) {
                 orders.send_msg(Msg::GoToSearchResults);
                 return;
             };
-            let Some(idx) = catchup.items.iter().position(|i| !i.seen) else {
+            let Some(thread_id) = current_thread_id(&model.context) else {
+                return;
+            };
+            let Some(idx) = catchup
+                .items
+                .iter()
+                .inspect(|i| info!("i {i:?} thread_id {thread_id}"))
+                .position(|i| i.id == thread_id)
+            else {
                 // All items have been seen
                 orders.send_msg(Msg::CatchupExit);
                 orders.send_msg(Msg::GoToSearchResults);
@@ -726,6 +731,7 @@ pub struct Catchup {
     pub items: Vec<CatchupItem>,
 }
 
+#[derive(Debug)]
 pub struct CatchupItem {
     pub id: String,
     pub seen: bool,
@@ -759,6 +765,7 @@ pub enum Msg {
     NextPage,
     PreviousPage,
     GoToSearchResults,
+
     UpdateQuery(String),
     SearchQuery(String),
 

web/src/view/mod.rs

@@ -12,7 +12,7 @@ use web_sys::{HtmlElement, HtmlInputElement};
 use crate::{
     api::urls,
     graphql::{front_page_query::*, show_thread_query::*},
-    state::{unread_query, CatchupItem, Context, Model, Msg, RefreshingState, Tag, Version},
+    state::{CatchupItem, Context, Model, Msg, RefreshingState, Tag, Version},
 };
 
 // TODO(wathiede): create a QueryString enum that wraps single and multiple message ids and thread
@@ -468,29 +468,22 @@ fn search_toolbar(
     if let Some(tri) = tri.get() {
         tri.set_indeterminate(indeterminate);
     }
-    nav![
-        C!["py-4", "flex", "w-full", "justify-between"],
-        div![
-            C!["gap-2", "flex", IF!(show_bulk_edit => "hidden")],
-            div![button![
+    let catchup = div![button![
         tw_classes::button(),
-                attrs! {At::Title => "Mark as read"},
+        attrs! {At::Title => "Catch up"},
         span![i![C!["far", "fa-eye"]]],
         span![C!["pl-2", "hidden", "md:inline"], "Catch-up"],
         ev(Ev::Click, |_| Msg::CatchupStart)
-            ]],
-        ],
-        div![
-            C!["gap-2", "flex", IF!(!show_bulk_edit => "hidden")],
-            div![
+    ]];
+    let tristate_input = div![
         C!["flex", "items-center", "mr-4"],
         input![
-                    tri,
+            &tri,
             C![&tw_classes::CHECKBOX],
             attrs! {
                 At::Type=>"checkbox",
-                        At::Checked=>all_selected,
-                    }
+            },
+            IF!(all_selected=>attrs!{At::Checked=>true})
         ],
         ev(Ev::Input, move |_| {
             if all_selected {
@@ -499,7 +492,18 @@ fn search_toolbar(
                 Msg::SelectionSetAll
             }
         }),
+    ];
+    nav![
+        C!["py-4", "flex", "w-full", "justify-between"],
+        div![
+            C!["gap-2", "flex", IF!(show_bulk_edit => "hidden")],
+            &tristate_input,
+            &catchup
         ],
+        div![
+            C!["gap-2", "flex", IF!(!show_bulk_edit => "hidden")],
+            &tristate_input,
+            &catchup,
             div![
                 button![
                     tw_classes::button(),
@@ -1208,9 +1212,11 @@ fn view_header(
     let query = Url::decode_uri_component(query).unwrap_or("".to_string());
 
     nav![
-        C!["flex", "px-4", "pt-4", "overflow-hidden"],
-        a![
-            C![IF![is_error => "bg-red-500"], "rounded-r-none"],
+        C!["flex", "flex-col"],
+        div![
+            C!["flex-auto", "flex"],
+            button![
+                C![IF![is_error => "bg-red-500"], "rounded-none"],
                 tw_classes::button(),
                 span![i![C![
                     "fa-solid",
@@ -1219,24 +1225,42 @@ fn view_header(
                 ]]],
                 ev(Ev::Click, |_| Msg::RefreshStart),
             ],
-        a![
+            button![
                 tw_classes::button(),
-            C!["px-4", "rounded-none"],
-            attrs! {
-                At::Href => urls::search(unread_query(), 0)
-            },
-            "Unread",
-        ],
-        a![
-            tw_classes::button(),
-            C!["px-4", "rounded-none"],
-            attrs! {
-                At::Href => urls::search("", 0)
-            },
+                C!["grow", "rounded-none"],
                 "All",
+                ev(Ev::Click, |_| Msg::SearchQuery(String::new())),
             ],
+            button![
+                tw_classes::button(),
+                C!["grow", "rounded-none"],
+                span![i![C!["far", "fa-envelope"]]],
+                " Unread",
+                ev(Ev::Click, |_| Msg::SearchQuery("is:unread".to_string())),
+            ],
+            button![
+                tw_classes::button(),
+                C!["grow", "rounded-none"],
+                span![i![C!["far", "fa-envelope"]]],
+                " News",
+                ev(Ev::Click, |_| Msg::SearchQuery(
+                    "is:unread is:news".to_string()
+                )),
+            ],
+            button![
+                tw_classes::button(),
+                C!["grow", "rounded-none"],
+                span![i![C!["far", "fa-envelope"]]],
+                " Mail",
+                ev(Ev::Click, |_| Msg::SearchQuery(
+                    "is:unread is:mail".to_string()
+                )),
+            ],
+        ],
+        div![
+            C!["flex-auto", "flex"],
             input![
-            C!["grow", "pl-2", "text-black", "rounded-r"],
+                C!["grow", "text-black", "m-2", "p-1"],
                 attrs! {
                     At::Placeholder => "Search";
                     At::AutoFocus => auto_focus_search.as_at_value();
@@ -1251,6 +1275,7 @@ fn view_header(
                 }),
             ]
         ]
+    ]
 }
 
 pub fn view_tags(tags: &Option<Vec<Tag>>) -> Node<Msg> {
@@ -1555,13 +1580,13 @@ fn render_news_post_header(post: &ShowThreadQueryThreadOnNewsPost) -> Node<Msg>
                 } else {
                     "fa-envelope-open"
                 },
-            ]]
-        ],
+            ]],
             ev(Ev::Click, move |e| {
                 e.stop_propagation();
                 Msg::SetUnread(id, !is_unread)
             })
         ]
+    ]
 }
 fn reading_progress(ratio: f64) -> Node<Msg> {
     let percent = ratio * 100.;