From ad355475820206834d6557194c439abedb15b2cf Mon Sep 17 00:00:00 2001 From: William Casarin Date: Sun, 3 Aug 2025 16:13:45 -0700 Subject: [PATCH] refactor: collapse client label settings; drop CLI/settings toggles MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The "top vs bottom" client label setting was cluttering the UI and codebase with toggles that added little value. This consolidates client label handling into one option, removes unused CLI/settings knobs, and makes NoteView’s API consistent and fluent. Result: fewer knobs, less branching, and a clearer, more predictable UI. Now client labels are only shown in one place: selected notes. - Drop `--show-client` arg in notedeck and `--show-note-client=top|bottom` args in notedeck_columns - Remove `NotedeckOptions::ShowClient` and related CLI parsing - Delete `ShowSourceClientOption` enum, settings UI, and `SettingsAction::SetShowSourceClient` - Collapse `NoteOptions::{ClientNameTop, ClientNameBottom}` into a single `NoteOptions::ClientName` - Add `NoteOptions::{Framed, UnreadIndicator}` - Move “framed” and unread indicator into flags (no more ad‑hoc bools) - Add new NoteView builder methods: `.client_name()`, `.frame()`, `.unread_indicator()`, and `.selected_style()` - CLI flags for showing client labels have been removed - `ClientNameTop`/`ClientNameBottom` replaced with `ClientName` - API using `framed` or `show_unread_indicator` booleans must now use the new flag setters Signed-off-by: William Casarin --- crates/notedeck/src/args.rs | 2 - crates/notedeck/src/options.rs | 3 - crates/notedeck_columns/src/app.rs | 13 +-- crates/notedeck_columns/src/args.rs | 6 -- crates/notedeck_columns/src/ui/mod.rs | 1 - crates/notedeck_columns/src/ui/settings.rs | 115 --------------------- crates/notedeck_columns/src/ui/thread.rs | 7 ++ crates/notedeck_ui/src/note/contents.rs | 7 +- crates/notedeck_ui/src/note/mod.rs | 87 +++++++++------- crates/notedeck_ui/src/note/options.rs | 13 ++- 10 files changed, 67 insertions(+), 187 deletions(-) diff --git a/crates/notedeck/src/args.rs b/crates/notedeck/src/args.rs index 42b7157a..bd6caed7 100644 --- a/crates/notedeck/src/args.rs +++ b/crates/notedeck/src/args.rs @@ -124,8 +124,6 @@ impl Args { res.options.set(NotedeckOptions::UseKeystore, true); } else if arg == "--relay-debug" { res.options.set(NotedeckOptions::RelayDebug, true); - } else if arg == "--show-client" { - res.options.set(NotedeckOptions::ShowClient, true); } else if arg == "--notebook" { res.options.set(NotedeckOptions::FeatureNotebook, true); } else { diff --git a/crates/notedeck/src/options.rs b/crates/notedeck/src/options.rs index 0ca7fd77..1c01644c 100644 --- a/crates/notedeck/src/options.rs +++ b/crates/notedeck/src/options.rs @@ -20,9 +20,6 @@ bitflags! { /// Use keystore? const UseKeystore = 1 << 4; - /// Show client on notes? - const ShowClient = 1 << 5; - /// Simulate is_compiled_as_mobile ? const Mobile = 1 << 6; diff --git a/crates/notedeck_columns/src/app.rs b/crates/notedeck_columns/src/app.rs index f7846257..86c0293b 100644 --- a/crates/notedeck_columns/src/app.rs +++ b/crates/notedeck_columns/src/app.rs @@ -10,7 +10,7 @@ use crate::{ subscriptions::{SubKind, Subscriptions}, support::Support, timeline::{self, kind::ListKind, thread::Threads, TimelineCache, TimelineKind}, - ui::{self, DesktopSidePanel, ShowSourceClientOption, SidePanelAction}, + ui::{self, DesktopSidePanel, SidePanelAction}, view_state::ViewState, Result, }; @@ -591,17 +591,6 @@ fn get_note_options(args: ColumnsArgs, settings_handler: &mut SettingsHandler) - NoteOptions::HideMedia, args.is_flag_set(ColumnsFlag::NoMedia), ); - note_options.set( - NoteOptions::ClientNameTop, - ShowSourceClientOption::Top == settings_handler.show_source_client().into() - || args.is_flag_set(ColumnsFlag::ShowNoteClientTop), - ); - note_options.set( - NoteOptions::ClientNameBottom, - ShowSourceClientOption::Bottom == settings_handler.show_source_client().into() - || args.is_flag_set(ColumnsFlag::ShowNoteClientBottom), - ); - note_options.set( NoteOptions::RepliesNewestFirst, settings_handler.show_replies_newest_first(), diff --git a/crates/notedeck_columns/src/args.rs b/crates/notedeck_columns/src/args.rs index 43f2b523..c5296fca 100644 --- a/crates/notedeck_columns/src/args.rs +++ b/crates/notedeck_columns/src/args.rs @@ -11,8 +11,6 @@ pub enum ColumnsFlag { Textmode, Scramble, NoMedia, - ShowNoteClientTop, - ShowNoteClientBottom, } pub struct ColumnsArgs { @@ -54,10 +52,6 @@ impl ColumnsArgs { res.clear_flag(ColumnsFlag::SinceOptimize); } else if arg == "--scramble" { res.set_flag(ColumnsFlag::Scramble); - } else if arg == "--show-note-client=top" { - res.set_flag(ColumnsFlag::ShowNoteClientTop); - } else if arg == "--show-note-client=bottom" { - res.set_flag(ColumnsFlag::ShowNoteClientBottom); } else if arg == "--no-media" { res.set_flag(ColumnsFlag::NoMedia); } else if arg == "--filter" { diff --git a/crates/notedeck_columns/src/ui/mod.rs b/crates/notedeck_columns/src/ui/mod.rs index 2d73c567..bfb922cd 100644 --- a/crates/notedeck_columns/src/ui/mod.rs +++ b/crates/notedeck_columns/src/ui/mod.rs @@ -26,7 +26,6 @@ pub use preview::{Preview, PreviewApp, PreviewConfig}; pub use profile::ProfileView; pub use relay::RelayView; pub use settings::SettingsView; -pub use settings::ShowSourceClientOption; pub use side_panel::{DesktopSidePanel, SidePanelAction}; pub use thread::ThreadView; pub use timeline::TimelineView; diff --git a/crates/notedeck_columns/src/ui/settings.rs b/crates/notedeck_columns/src/ui/settings.rs index d944d23f..31e846b7 100644 --- a/crates/notedeck_columns/src/ui/settings.rs +++ b/crates/notedeck_columns/src/ui/settings.rs @@ -10,7 +10,6 @@ use notedeck::{ SettingsHandler, DEFAULT_NOTE_BODY_FONT_SIZE, }; use notedeck_ui::{NoteOptions, NoteView}; -use strum::Display; use crate::{nav::RouterAction, Damus, Route}; @@ -21,89 +20,9 @@ const MAX_ZOOM: f32 = 3.0; const ZOOM_STEP: f32 = 0.1; const RESET_ZOOM: f32 = 1.0; -#[derive(Clone, Copy, PartialEq, Eq, Display)] -pub enum ShowSourceClientOption { - Hide, - Top, - Bottom, -} - -impl From for String { - fn from(show_option: ShowSourceClientOption) -> Self { - match show_option { - ShowSourceClientOption::Hide => "hide".to_string(), - ShowSourceClientOption::Top => "top".to_string(), - ShowSourceClientOption::Bottom => "bottom".to_string(), - } - } -} - -impl From for ShowSourceClientOption { - fn from(note_options: NoteOptions) -> Self { - if note_options.contains(NoteOptions::ClientNameTop) { - ShowSourceClientOption::Top - } else if note_options.contains(NoteOptions::ClientNameBottom) { - ShowSourceClientOption::Bottom - } else { - ShowSourceClientOption::Hide - } - } -} - -impl From for ShowSourceClientOption { - fn from(s: String) -> Self { - match s.to_lowercase().as_str() { - "hide" => Self::Hide, - "top" => Self::Top, - "bottom" => Self::Bottom, - _ => Self::Hide, // default fallback - } - } -} - -impl ShowSourceClientOption { - pub fn set_note_options(self, note_options: &mut NoteOptions) { - match self { - Self::Hide => { - note_options.set(NoteOptions::ClientNameTop, false); - note_options.set(NoteOptions::ClientNameBottom, false); - } - Self::Bottom => { - note_options.set(NoteOptions::ClientNameTop, false); - note_options.set(NoteOptions::ClientNameBottom, true); - } - Self::Top => { - note_options.set(NoteOptions::ClientNameTop, true); - note_options.set(NoteOptions::ClientNameBottom, false); - } - } - } - - fn label(&self, i18n: &mut Localization) -> String { - match self { - Self::Hide => tr!( - i18n, - "Hide", - "Option in settings section to hide the source client label in note display" - ), - Self::Top => tr!( - i18n, - "Top", - "Option in settings section to show the source client label at the top of the note" - ), - Self::Bottom => tr!( - i18n, - "Bottom", - "Option in settings section to show the source client label at the bottom of the note" - ), - } - } -} - pub enum SettingsAction { SetZoomFactor(f32), SetTheme(ThemePreference), - SetShowSourceClient(ShowSourceClientOption), SetLocale(LanguageIdentifier), SetRepliestNewestFirst(bool), SetNoteBodyFontSize(f32), @@ -131,11 +50,6 @@ impl SettingsAction { ctx.set_zoom_factor(zoom_factor); settings.set_zoom_factor(zoom_factor); } - Self::SetShowSourceClient(option) => { - option.set_note_options(&mut app.note_options); - - settings.set_show_source_client(option); - } Self::SetTheme(theme) => { ctx.set_theme(theme); settings.set_theme(theme); @@ -551,35 +465,6 @@ impl<'a> SettingsView<'a> { )); } }); - - ui.horizontal_wrapped(|ui| { - ui.label(richtext_small(tr!( - self.note_context.i18n, - "Source client:", - "Label for Source client, others settings section", - ))); - - for option in [ - ShowSourceClientOption::Hide, - ShowSourceClientOption::Top, - ShowSourceClientOption::Bottom, - ] { - let mut current: ShowSourceClientOption = - self.settings.show_source_client.clone().into(); - - if ui - .selectable_value( - &mut current, - option, - RichText::new(option.label(self.note_context.i18n)) - .text_style(NotedeckTextStyle::Small.text_style()), - ) - .changed() - { - action = Some(SettingsAction::SetShowSourceClient(option)); - } - } - }); }); action diff --git a/crates/notedeck_columns/src/ui/thread.rs b/crates/notedeck_columns/src/ui/thread.rs index e0632b58..faa8131b 100644 --- a/crates/notedeck_columns/src/ui/thread.rs +++ b/crates/notedeck_columns/src/ui/thread.rs @@ -279,6 +279,12 @@ enum ThreadNoteType { Reply, } +impl ThreadNoteType { + fn is_selected(&self) -> bool { + matches!(self, ThreadNoteType::Selected { .. }) + } +} + struct ThreadNotes<'a> { notes: Vec>, selected_index: usize, @@ -313,6 +319,7 @@ impl<'a> ThreadNote<'a> { ) -> NoteResponse { let inner = notedeck_ui::padding(8.0, ui, |ui| { NoteView::new(note_context, &self.note, self.options(flags), jobs) + .selected_style(self.note_type.is_selected()) .unread_indicator(self.unread_and_have_replies) .show(ui) }); diff --git a/crates/notedeck_ui/src/note/contents.rs b/crates/notedeck_ui/src/note/contents.rs index 18c42a1c..c72fa654 100644 --- a/crates/notedeck_ui/src/note/contents.rs +++ b/crates/notedeck_ui/src/note/contents.rs @@ -129,11 +129,6 @@ fn render_note_contents( options: NoteOptions, jobs: &mut JobsCache, ) -> NoteResponse { - if options.contains(NoteOptions::ClientNameTop) { - let before_date = false; - render_client_name(ui, note_context.note_cache, note, before_date); - } - let response = render_undecorated_note_contents(ui, note_context, txn, note, options, jobs); ui.horizontal_wrapped(|ui| { @@ -163,7 +158,7 @@ fn note_bottom_metadata_ui( secondary_label(ui, time_format(i18n, note.created_at())); } - if options.contains(NoteOptions::ClientNameBottom) { + if options.contains(NoteOptions::ClientName) { render_client_name(ui, note_cache, note, show_full_date); } } diff --git a/crates/notedeck_ui/src/note/mod.rs b/crates/notedeck_ui/src/note/mod.rs index 69176b68..a1c4afef 100644 --- a/crates/notedeck_ui/src/note/mod.rs +++ b/crates/notedeck_ui/src/note/mod.rs @@ -39,10 +39,8 @@ pub struct NoteView<'a, 'd> { note_context: &'a mut NoteContext<'d>, parent: Option, note: &'a nostrdb::Note<'a>, - framed: bool, flags: NoteOptions, jobs: &'a mut JobsCache, - show_unread_indicator: bool, } pub struct NoteResponse { @@ -89,13 +87,9 @@ impl<'a, 'd> NoteView<'a, 'd> { pub fn new( note_context: &'a mut NoteContext<'d>, note: &'a nostrdb::Note<'a>, - mut flags: NoteOptions, + flags: NoteOptions, jobs: &'a mut JobsCache, ) -> Self { - flags.set(NoteOptions::ActionBar, true); - flags.set(NoteOptions::HasNotePreviews, true); - - let framed = false; let parent: Option = None; Self { @@ -103,9 +97,7 @@ impl<'a, 'd> NoteView<'a, 'd> { parent, note, flags, - framed, jobs, - show_unread_indicator: false, } } @@ -118,91 +110,121 @@ impl<'a, 'd> NoteView<'a, 'd> { .options_button(true) .is_preview(true) .full_date(false) + .client_name(false) } + pub fn selected_style(self, selected: bool) -> Self { + self.wide(selected) + .full_date(selected) + .client_name(selected) + } + + #[inline] pub fn textmode(mut self, enable: bool) -> Self { self.options_mut().set(NoteOptions::Textmode, enable); self } + #[inline] + pub fn client_name(mut self, enable: bool) -> Self { + self.options_mut().set(NoteOptions::ClientName, enable); + self + } + + #[inline] pub fn full_date(mut self, enable: bool) -> Self { self.options_mut().set(NoteOptions::FullCreatedDate, enable); self } + #[inline] pub fn actionbar(mut self, enable: bool) -> Self { self.options_mut().set(NoteOptions::ActionBar, enable); self } + #[inline] pub fn hide_media(mut self, enable: bool) -> Self { self.options_mut().set(NoteOptions::HideMedia, enable); self } + #[inline] pub fn frame(mut self, enable: bool) -> Self { - self.framed = enable; + self.options_mut().set(NoteOptions::Framed, enable); self } + #[inline] pub fn truncate(mut self, enable: bool) -> Self { self.options_mut().set(NoteOptions::Truncate, enable); self } + #[inline] pub fn small_pfp(mut self, enable: bool) -> Self { self.options_mut().set(NoteOptions::SmallPfp, enable); self } + #[inline] pub fn medium_pfp(mut self, enable: bool) -> Self { self.options_mut().set(NoteOptions::MediumPfp, enable); self } + #[inline] pub fn note_previews(mut self, enable: bool) -> Self { self.options_mut().set(NoteOptions::HasNotePreviews, enable); self } + #[inline] pub fn selectable_text(mut self, enable: bool) -> Self { self.options_mut().set(NoteOptions::SelectableText, enable); self } + #[inline] pub fn wide(mut self, enable: bool) -> Self { self.options_mut().set(NoteOptions::Wide, enable); self } + #[inline] pub fn options_button(mut self, enable: bool) -> Self { self.options_mut().set(NoteOptions::OptionsButton, enable); self } + #[inline] + pub fn unread_indicator(mut self, enable: bool) -> Self { + self.options_mut().set(NoteOptions::UnreadIndicator, enable); + self + } + + #[inline] pub fn options(&self) -> NoteOptions { self.flags } + #[inline] pub fn options_mut(&mut self) -> &mut NoteOptions { &mut self.flags } + #[inline] pub fn parent(mut self, parent: NoteKey) -> Self { self.parent = Some(parent); self } + #[inline] pub fn is_preview(mut self, is_preview: bool) -> Self { self.options_mut().set(NoteOptions::IsPreview, is_preview); self } - pub fn unread_indicator(mut self, show_unread_indicator: bool) -> Self { - self.show_unread_indicator = show_unread_indicator; - self - } - fn textmode_ui(&mut self, ui: &mut egui::Ui) -> egui::Response { let txn = self.note.txn().expect("todo: implement non-db notes"); @@ -340,7 +362,7 @@ impl<'a, 'd> NoteView<'a, 'd> { pub fn show(&mut self, ui: &mut egui::Ui) -> NoteResponse { if self.options().contains(NoteOptions::Textmode) { NoteResponse::new(self.textmode_ui(ui)) - } else if self.framed { + } else if self.options().contains(NoteOptions::Framed) { egui::Frame::new() .fill(ui.visuals().noninteractive().weak_bg_fill) .inner_margin(egui::Margin::same(8)) @@ -368,7 +390,6 @@ impl<'a, 'd> NoteView<'a, 'd> { i18n: &mut Localization, note: &Note, profile: &Result, nostrdb::Error>, - show_unread_indicator: bool, flags: NoteOptions, ) { let horiz_resp = ui @@ -383,19 +404,17 @@ impl<'a, 'd> NoteView<'a, 'd> { }) .response; - if !show_unread_indicator { - return; + if flags.contains(NoteOptions::UnreadIndicator) { + let radius = 4.0; + let circle_center = { + let mut center = horiz_resp.rect.right_center(); + center.x += radius + 4.0; + center + }; + + ui.painter() + .circle_filled(circle_center, radius, crate::colors::PINK); } - - let radius = 4.0; - let circle_center = { - let mut center = horiz_resp.rect.right_center(); - center.x += radius + 4.0; - center - }; - - ui.painter() - .circle_filled(circle_center, radius, crate::colors::PINK); } fn wide_ui( @@ -426,7 +445,6 @@ impl<'a, 'd> NoteView<'a, 'd> { self.note_context.i18n, self.note, profile, - self.show_unread_indicator, self.flags, ); }) @@ -515,14 +533,7 @@ impl<'a, 'd> NoteView<'a, 'd> { let mut note_action: Option = pfp_resp.into_action(self.note.pubkey()); ui.with_layout(egui::Layout::top_down(egui::Align::LEFT), |ui| { - NoteView::note_header( - ui, - self.note_context.i18n, - self.note, - profile, - self.show_unread_indicator, - self.flags, - ); + NoteView::note_header(ui, self.note_context.i18n, self.note, profile, self.flags); ui.horizontal_wrapped(|ui| 's: { ui.spacing_mut().item_spacing.x = if is_narrow(ui.ctx()) { 1.0 } else { 2.0 }; diff --git a/crates/notedeck_ui/src/note/options.rs b/crates/notedeck_ui/src/note/options.rs index f00edf36..56b9b756 100644 --- a/crates/notedeck_ui/src/note/options.rs +++ b/crates/notedeck_ui/src/note/options.rs @@ -23,13 +23,18 @@ bitflags! { /// will end with a ... and a "Show more" button. const Truncate = 1 << 11; /// Show note's client in the note content - const ClientNameTop = 1 << 12; - const ClientNameBottom = 1 << 13; + const ClientName = 1 << 12; - const RepliesNewestFirst = 1 << 14; + const RepliesNewestFirst = 1 << 13; /// Show note's full created at date at the bottom - const FullCreatedDate = 1 << 15; + const FullCreatedDate = 1 << 14; + + /// Note has a framed border + const Framed = 1 << 15; + + /// Note has a framed border + const UnreadIndicator = 1 << 16; } }