Remove more commented-out code (#4150)

Marshall Bowers created 2 years ago

This PR removes some more commented-out code from various spots in the
codebase.

Release Notes:

- N/A

Change summary

crates/storybook/src/stories/text.rs | 63 ------------------------------
crates/workspace/src/workspace.rs    | 12 -----
2 files changed, 75 deletions(-)

Detailed changes

crates/storybook/src/stories/text.rs

@@ -110,66 +110,3 @@ impl Render for TextStory {
             ).into_element()
     }
 }
-
-// TODO: Check all were updated to new style and remove
-
-// impl Render for TextStory {
-//     type Element = Div;
-
-//     fn render(&mut self, cx: &mut gpui::ViewContext<Self>) -> Self::Element {
-//         v_flex()
-//             .bg(blue())
-//             .child(
-//                 div()
-//                     .flex()
-//                     .child(div().max_w_96().bg(white()).child(concat!(
-//         "max-width: 96. The quick brown fox jumps over the lazy dog. ",
-//         "Meanwhile, the lazy dog decided it was time for a change. ",
-//         "He started daily workout routines, ate healthier and became the fastest dog in town.",
-//     ))),
-//             )
-//             .child(div().h_5())
-//             .child(div().flex().flex_col().w_96().bg(white()).child(concat!(
-//         "flex-col. width: 96; The quick brown fox jumps over the lazy dog. ",
-//         "Meanwhile, the lazy dog decided it was time for a change. ",
-//         "He started daily workout routines, ate healthier and became the fastest dog in town.",
-//     )))
-//             .child(div().h_5())
-//             .child(
-//                 div()
-//                     .flex()
-//                     .child(div().min_w_96().bg(white()).child(concat!(
-//     "min-width: 96. The quick brown fox jumps over the lazy dog. ",
-//     "Meanwhile, the lazy dog decided it was time for a change. ",
-//     "He started daily workout routines, ate healthier and became the fastest dog in town.",
-// ))))
-//             .child(div().h_5())
-//             .child(div().flex().w_96().bg(white()).child(div().overflow_hidden().child(concat!(
-//         "flex-row. width 96. overflow-hidden. The quick brown fox jumps over the lazy dog. ",
-//         "Meanwhile, the lazy dog decided it was time for a change. ",
-//         "He started daily workout routines, ate healthier and became the fastest dog in town.",
-//     ))))
-//             // NOTE: When rendering text in a horizontal flex container,
-//             // Taffy will not pass width constraints down from the parent.
-//             // To fix this, render text in a parent with overflow: hidden
-//                     .child(div().h_5())
-//                     .child(div().flex().w_96().bg(red()).child(concat!(
-//                 "flex-row. width 96. The quick brown fox jumps over the lazy dog. ",
-//                 "Meanwhile, the lazy dog decided it was time for a change. ",
-//                 "He started daily workout routines, ate healthier and became the fastest dog in town.",
-//             ))).child(
-//                 InteractiveText::new(
-//                     "interactive",
-//                     StyledText::new("Hello world, how is it going?").with_highlights(&cx.text_style(), [
-//                         (6..11, HighlightStyle {
-//                             background_color: Some(green()),
-//                             ..Default::default()
-//                         }),
-//                     ]),
-//                 )
-//                 .on_click(vec![2..4, 1..3, 7..9], |range_ix, _cx| {
-//                     println!("Clicked range {range_ix}");
-//                 })
-//             )
-//     }
-// }

crates/workspace/src/workspace.rs

@@ -669,18 +669,6 @@ impl Workspace {
 
         cx.defer(|this, cx| {
             this.update_window_title(cx);
-            // todo! @nate - these are useful for testing notifications
-            // this.show_error(
-            //     &anyhow::anyhow!("what happens if this message is very very very very very long"),
-            //     cx,
-            // );
-
-            // this.show_notification(1, cx, |cx| {
-            //     cx.new_view(|_cx| {
-            //         simple_message_notification::MessageNotification::new(format!("Error:"))
-            //             .with_click_message("click here because!")
-            //     })
-            // });
         });
         Workspace {
             weak_self: weak_handle.clone(),