feat(View): introduce State parameter for draw_screen

Drawing a screen would require information about the screen's state,
which shouldn't be available in the View itself.

Pass state to the `draw_screen` function.

Signed-off-by: Ivin Joel Abraham <[email protected]>

Author: ivinjabraham

src/app.rs

@@ -51,10 +51,11 @@ impl App {
             })
     }
 
-    #[allow(dead_code)]
+    #[allow(dead_code, unreachable_code)]
     pub fn run(&mut self) -> Result<()> {
         loop {
-            self.get_current_view().draw_screen();
+            self.get_current_view()
+                .draw_screen(self.get_current_viewmodel().state());
 
             if let Event::Key(key) = event::read()? {
                 if key.kind == KeyEventKind::Release {

src/model/screens/mod.rs

@@ -1,3 +1,5 @@
+use crate::viewmodels::ViewModelState;
+
 pub mod bookmarked;
 pub mod details_actions;
 pub mod edit_config;
@@ -14,8 +16,8 @@ pub enum View {
 }
 
 impl View {
-    #[allow(dead_code)]
-    pub fn draw_screen(&self) {
+    #[allow(dead_code, unused_variables)]
+    pub fn draw_screen(&self, state: ViewModelState) {
         match self {
             View::MailingListSelection => todo!(),
             View::BookmarkedPatchsets => todo!(),