Fix new errors after merge

Joseph Lyons created 3 years ago

Change summary

Cargo.lock                             |  1 -
crates/feedback/Cargo.toml             |  1 -
crates/feedback/src/feedback_editor.rs | 20 ++++++++------------
crates/workspace/src/workspace.rs      |  2 +-
crates/zed/src/zed.rs                  |  3 +--
5 files changed, 10 insertions(+), 17 deletions(-)

Detailed changes

Cargo.lock

@@ -2040,7 +2040,6 @@ dependencies = [
  "search",
  "serde",
  "settings",
- "smallvec",
  "sysinfo",
  "theme",
  "tree-sitter-markdown",

crates/feedback/Cargo.toml

@@ -25,7 +25,6 @@ project = { path = "../project" }
 search = { path = "../search" }
 serde = { version = "1.0", features = ["derive", "rc"] }
 settings = { path = "../settings" }
-smallvec = { version = "1.6", features = ["union"] }
 sysinfo = "0.27.1"
 theme = { path = "../theme" }
 tree-sitter-markdown = { git = "https://github.com/MDeiml/tree-sitter-markdown", rev = "330ecab87a3e3a7211ac69bbadc19eabecdb1cca" }

crates/feedback/src/feedback_editor.rs

@@ -7,18 +7,18 @@ use futures::AsyncReadExt;
 use gpui::{
     actions,
     elements::{ChildView, Flex, Label, MouseEventHandler, ParentElement, Stack, Text},
-    serde_json, AnyViewHandle, CursorStyle, Element, ElementBox, Entity, ModelHandle, MouseButton,
-    MutableAppContext, PromptLevel, RenderContext, Task, View, ViewContext, ViewHandle,
+    serde_json, AnyViewHandle, AppContext, CursorStyle, Element, ElementBox, Entity, ModelHandle,
+    MouseButton, MutableAppContext, PromptLevel, RenderContext, Task, View, ViewContext,
+    ViewHandle,
 };
 use isahc::Request;
 use language::Buffer;
 use postage::prelude::Stream;
 
 use lazy_static::lazy_static;
-use project::{Project, ProjectEntryId, ProjectPath};
+use project::Project;
 use serde::Serialize;
 use settings::Settings;
-use smallvec::SmallVec;
 use workspace::{
     item::{Item, ItemHandle},
     searchable::{SearchableItem, SearchableItemHandle},
@@ -345,16 +345,12 @@ impl Item for FeedbackEditor {
             .boxed()
     }
 
-    fn to_item_events(_: &Self::Event) -> Vec<workspace::item::ItemEvent> {
-        Vec::new()
+    fn for_each_project_item(&self, cx: &AppContext, f: &mut dyn FnMut(usize, &dyn project::Item)) {
+        self.editor.for_each_project_item(cx, f)
     }
 
-    fn project_path(&self, _: &gpui::AppContext) -> Option<ProjectPath> {
-        None
-    }
-
-    fn project_entry_ids(&self, _: &gpui::AppContext) -> SmallVec<[ProjectEntryId; 3]> {
-        SmallVec::new()
+    fn to_item_events(_: &Self::Event) -> Vec<workspace::item::ItemEvent> {
+        Vec::new()
     }
 
     fn is_singleton(&self, _: &gpui::AppContext) -> bool {

crates/workspace/src/workspace.rs

@@ -99,7 +99,7 @@ actions!(
         ToggleRightSidebar,
         NewTerminal,
         NewSearch,
-        Feedback
+        Feedback,
         ShowNotif,
     ]
 );

crates/zed/src/zed.rs

@@ -20,8 +20,7 @@ use gpui::{
     },
     impl_actions,
     platform::{WindowBounds, WindowOptions},
-    AssetSource, AsyncAppContext, ClipboardItem, PromptLevel, TitlebarOptions, ViewContext,
-    WindowKind,
+    AssetSource, AsyncAppContext, PromptLevel, TitlebarOptions, ViewContext, WindowKind,
 };
 use language::Rope;
 use lazy_static::lazy_static;