feat(tui): De-duplicate describe view (#530)

Reuse Describe component directly in the table view to avoid code
duplication.

Author: gtema

openstack_tui/src/components/compute/flavors.rs

@@ -87,6 +87,8 @@ impl<'a> Component for ComputeFlavors<'a> {
             KeyCode::End => self.cursor_last()?,
             KeyCode::PageUp => self.cursor_page_up()?,
             KeyCode::PageDown => self.cursor_page_down()?,
+            KeyCode::Left => self.cursor_left()?,
+            KeyCode::Right => self.cursor_right()?,
             KeyCode::Tab => self.key_tab()?,
             _ => {}
         }
@@ -99,7 +101,7 @@ impl<'a> Component for ComputeFlavors<'a> {
     fn draw(&mut self, f: &mut Frame<'_>, area: Rect) -> Result<()> {
         let areas = Layout::vertical([Constraint::Min(5), Constraint::Length(3)]).split(area);
 
-        self.render_content(TITLE, f, areas[0]);
+        self.render_content(TITLE, f, areas[0])?;
         self.render_footer(f, areas[1]);
         Ok(())
     }

openstack_tui/src/components/compute/servers.rs

@@ -108,6 +108,8 @@ impl<'a> Component for ComputeServers<'a> {
             KeyCode::End => self.cursor_last()?,
             KeyCode::PageUp => self.cursor_page_up()?,
             KeyCode::PageDown => self.cursor_page_down()?,
+            KeyCode::Left => self.cursor_left()?,
+            KeyCode::Right => self.cursor_right()?,
             KeyCode::Tab => self.key_tab()?,
             _ => {}
         }
@@ -120,7 +122,7 @@ impl<'a> Component for ComputeServers<'a> {
     fn draw(&mut self, f: &mut Frame<'_>, area: Rect) -> Result<()> {
         let areas = Layout::vertical([Constraint::Min(5), Constraint::Length(3)]).split(area);
 
-        self.render_content(TITLE, f, areas[0]);
+        self.render_content(TITLE, f, areas[0])?;
         self.render_footer(f, areas[1]);
         Ok(())
     }

openstack_tui/src/components/describe.rs

@@ -33,7 +33,8 @@ pub struct Describe {
     pub keymap: HashMap<KeyEvent, Action>,
     pub text: Vec<String>,
     pub last_events: Vec<KeyEvent>,
-    wrap: bool,
+    title: Option<String>,
+    is_focused: bool,
     max_row_length: u16,
     content_scroll: (u16, u16),
     content_size: Size,
@@ -43,7 +44,10 @@ pub struct Describe {
 
 impl Describe {
     pub fn new() -> Self {
-        Self::default()
+        Self {
+            is_focused: true,
+            ..Default::default()
+        }
     }
 
     pub fn keymap(mut self, keymap: HashMap<KeyEvent, Action>) -> Self {
@@ -57,11 +61,11 @@ impl Describe {
 
     pub fn render_tick(&mut self) {}
 
-    fn set_data(&mut self, data: Value) -> Result<()> {
+    pub fn set_data(&mut self, data: Value) -> Result<()> {
         if data.is_string() {
             self.text = data
                 .as_str()
-                .ok_or_eyre("Cannot treat data as string")?
+                .ok_or_eyre("Cannot access data as string")?
                 .split("\n")
                 .map(String::from)
                 .collect::<Vec<_>>();
@@ -87,6 +91,16 @@ impl Describe {
         Ok(())
     }
 
+    pub fn set_title(&mut self, title: Option<String>) -> Result<()> {
+        self.title = title;
+        Ok(())
+    }
+
+    pub fn set_focus(&mut self, focus: bool) -> Result<()> {
+        self.is_focused = focus;
+        Ok(())
+    }
+
     pub fn cursor_up(&mut self) -> Result<()> {
         if self.text.len() as u16 > self.content_size.height {
             self.content_scroll.0 = self.content_scroll.0.saturating_sub(1);
@@ -165,10 +179,21 @@ impl Describe {
 
     fn render(&mut self, f: &mut Frame<'_>, area: Rect) {
         let block = Block::default()
-            .title(Title::from(" Describe ").alignment(Alignment::Center))
+            .title(
+                Title::from(self.title.clone().unwrap_or(String::from(" Describe ")))
+                    .alignment(Alignment::Center),
+            )
+            .title_style(match self.is_focused {
+                true => Style::new().white(),
+                false => Style::default(),
+            })
             .borders(Borders::ALL)
             .padding(Padding::horizontal(1))
-            .border_style(Style::default().fg(PALETTES[0].c900));
+            .border_style(Style::default().fg(PALETTES[0].c900))
+            .style(match self.is_focused {
+                true => Style::new(),
+                false => Style::new().gray(),
+            });
         self.content_size = block.inner(area).as_size();
 
         let text: Vec<Line> = self.text.clone().into_iter().map(Line::from).collect();

openstack_tui/src/components/image/images.rs

@@ -100,6 +100,8 @@ impl<'a> Component for Images<'a> {
             KeyCode::End => self.cursor_last()?,
             KeyCode::PageUp => self.cursor_page_up()?,
             KeyCode::PageDown => self.cursor_page_down()?,
+            KeyCode::Left => self.cursor_left()?,
+            KeyCode::Right => self.cursor_right()?,
             KeyCode::Tab => self.key_tab()?,
             _ => {}
         }
@@ -109,7 +111,7 @@ impl<'a> Component for Images<'a> {
     fn draw(&mut self, f: &mut Frame<'_>, area: Rect) -> Result<()> {
         let areas = Layout::vertical([Constraint::Min(5), Constraint::Length(3)]).split(area);
 
-        self.render_content(TITLE, f, areas[0]);
+        self.render_content(TITLE, f, areas[0])?;
         self.render_footer(f, areas[1]);
         Ok(())
     }

openstack_tui/src/components/network/networks.rs

@@ -90,6 +90,8 @@ impl<'a> Component for NetworkNetworks<'a> {
             KeyCode::End => self.cursor_last()?,
             KeyCode::PageUp => self.cursor_page_up()?,
             KeyCode::PageDown => self.cursor_page_down()?,
+            KeyCode::Left => self.cursor_left()?,
+            KeyCode::Right => self.cursor_right()?,
             KeyCode::Tab => self.key_tab()?,
             KeyCode::Enter => {
                 if let Some(command_tx) = self.get_command_tx() {
@@ -113,7 +115,7 @@ impl<'a> Component for NetworkNetworks<'a> {
     fn draw(&mut self, f: &mut Frame<'_>, area: Rect) -> Result<()> {
         let areas = Layout::vertical([Constraint::Min(5), Constraint::Length(3)]).split(area);
 
-        self.render_content(TITLE, f, areas[0]);
+        self.render_content(TITLE, f, areas[0])?;
         self.render_footer(f, areas[1]);
         Ok(())
     }

openstack_tui/src/components/network/subnets.rs

@@ -91,6 +91,8 @@ impl<'a> Component for NetworkSubnets<'a> {
             KeyCode::End => self.cursor_last()?,
             KeyCode::PageUp => self.cursor_page_up()?,
             KeyCode::PageDown => self.cursor_page_down()?,
+            KeyCode::Left => self.cursor_left()?,
+            KeyCode::Right => self.cursor_right()?,
             KeyCode::Tab => self.key_tab()?,
             KeyCode::Char('0') => {
                 return Ok(Some(Action::NetworkSubnetFilter(NetworkSubnetFilters {
@@ -109,7 +111,7 @@ impl<'a> Component for NetworkSubnets<'a> {
     fn draw(&mut self, f: &mut Frame<'_>, area: Rect) -> Result<()> {
         let areas = Layout::vertical([Constraint::Min(5), Constraint::Length(3)]).split(area);
 
-        self.render_content(TITLE, f, areas[0]);
+        self.render_content(TITLE, f, areas[0])?;
         self.render_footer(f, areas[1]);
         Ok(())
     }