cff961e issue: Fix and clean up selection state — radicle-tui

Radicle terminal user interface

issue: Fix and clean up selection state

Erik Kundt committed 2 years ago

commit cff961e0b12e1b7980f8a257ba715118444f6b65
parent 608032c5734b6ed447f6a52f5ff8a713beb0516d

2 files changed +39 -47

modified bin/commands/issue/flux/select.rs

@@ -53,7 +53,6 @@ impl Default for UIState {

 #[derive(Clone, Debug)]
 pub struct State {
     issues: Vec<IssueItem>,
     selected: Option<IssueItem>,
     mode: Mode,
     search: StateValue<String>,
     ui: UIState,

@@ -73,11 +72,8 @@ impl TryFrom<&Context> for State {

             }
         }
         let selected = items.first().cloned();
         Ok(Self {
             issues: items,
             selected,
             mode: context.mode.clone(),
             search: StateValue::new(context.filter.to_string()),
             ui: UIState::default(),

@@ -87,7 +83,6 @@ impl TryFrom<&Context> for State {

 pub enum Action {
     Exit { selection: Option<Selection> },
     Select { item: IssueItem },
     PageSize(usize),
     OpenSearch,
     UpdateSearch { value: String },

@@ -103,10 +98,6 @@ impl store::State<Action, Selection> for State {

     fn handle_action(&mut self, action: Action) -> Option<Exit<Selection>> {
         match action {
             Action::Exit { selection } => Some(Exit { value: selection }),
             Action::Select { item } => {
                 self.selected = Some(item);
                 None
             }
             Action::PageSize(size) => {
                 self.ui.page_size = size;
                 None

modified bin/commands/issue/flux/select/ui.rs

@@ -30,7 +30,6 @@ use crate::tui_issue::common::Mode;

 use super::{Action, State};
 pub struct ListPageProps {
     selected: Option<IssueItem>,
     mode: Mode,
     show_search: bool,
     show_help: bool,

@@ -39,7 +38,6 @@ pub struct ListPageProps {

 impl From<&State> for ListPageProps {
     fn from(state: &State) -> Self {
         Self {
             selected: state.selected.clone(),
             mode: state.mode.clone(),
             show_search: state.ui.show_search,
             show_help: state.ui.show_help,

@@ -105,32 +103,6 @@ impl<'a> Widget<State, Action> for ListPage<'a> {

                 Key::Esc | Key::Ctrl('c') => {
                     let _ = self.action_tx.send(Action::Exit { selection: None });
                 }
                 Key::Char('\n') => {
                     if let Some(selected) = &self.props.selected {
                         let operation = match self.props.mode {
                             Mode::Operation => Some(IssueOperation::Show.to_string()),
                             Mode::Id => None,
                         };
                         let _ = self.action_tx.send(Action::Exit {
                             selection: Some(Selection {
                                 operation,
                                 ids: vec![selected.id],
                                 args: vec![],
                             }),
                         });
                     }
                 }
                 Key::Char('e') => {
                     if let Some(selected) = &self.props.selected {
                         let _ = self.action_tx.send(Action::Exit {
                             selection: Some(Selection {
                                 operation: Some(IssueOperation::Edit.to_string()),
                                 ids: vec![selected.id],
                                 args: vec![],
                             }),
                         });
                     }
                 }
                 Key::Char('/') => {
                     let _ = self.action_tx.send(Action::OpenSearch);
                 }

@@ -197,6 +169,7 @@ impl<'a> Render<()> for ListPage<'a> {

 }
 struct IssuesProps {
     mode: Mode,
     issues: Vec<IssueItem>,
     search: String,
     stats: HashMap<String, usize>,

@@ -250,6 +223,7 @@ impl From<&State> for IssuesProps {

         ]);
         Self {
             mode: state.mode.clone(),
             issues,
             search: state.search.read(),
             widths: [

@@ -343,18 +317,45 @@ impl Widget<State, Action> for Issues {

             Key::End => {
                 self.table.end(self.props.issues.len());
             }
             Key::Char('\n') => {
                 let operation = match self.props.mode {
                     Mode::Operation => Some(IssueOperation::Show.to_string()),
                     Mode::Id => None,
                 };
                 self.table
                     .selected()
                     .and_then(|selected| self.props.issues.get(selected))
                     .and_then(|issue| {
                         self.action_tx
                             .send(Action::Exit {
                                 selection: Some(Selection {
                                     operation,
                                     ids: vec![issue.id],
                                     args: vec![],
                                 }),
                             })
                             .ok()
                     });
             }
             Key::Char('e') => {
                 self.table
                     .selected()
                     .and_then(|selected| self.props.issues.get(selected))
                     .and_then(|issue| {
                         self.action_tx
                             .send(Action::Exit {
                                 selection: Some(Selection {
                                     operation: Some(IssueOperation::Edit.to_string()),
                                     ids: vec![issue.id],
                                     args: vec![],
                                 }),
                             })
                             .ok()
                     });
             }
             _ => {}
         }
         self.table
             .selected()
             .and_then(|selected| self.props.issues.get(selected))
             .and_then(|issue| {
                 self.action_tx
                     .send(Action::Select {
                         item: issue.clone(),
                     })
                     .ok()
             });
     }
 }