From 59b4165136f6fb4bf53ba71de627f7bf1bde1c89 Mon Sep 17 00:00:00 2001
From: Jan Hohenheim <jan@hohenheim.ch>
Date: Thu, 16 Feb 2023 19:59:26 +0100
Subject: [PATCH 1/6] Implement dialog appear one letter at a time

---
 Cargo.lock                      |  7 +++++
 Cargo.toml                      |  1 +
 src/world_interaction/dialog.rs | 51 ++++++++++++++++++++++++---------
 3 files changed, 45 insertions(+), 14 deletions(-)

diff --git a/Cargo.lock b/Cargo.lock
index 5cd3e862..c0101592 100644
--- a/Cargo.lock
+++ b/Cargo.lock
@@ -2001,6 +2001,7 @@ dependencies = [
  "serde",
  "strum",
  "strum_macros",
+ "unicode-segmentation",
  "winit",
 ]
 
@@ -4137,6 +4138,12 @@ dependencies = [
  "tinyvec",
 ]
 
+[[package]]
+name = "unicode-segmentation"
+version = "1.10.1"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "1dd624098567895118886609431a7c3b8f516e41d30e0643f03d94592a147e36"
+
 [[package]]
 name = "unicode-width"
 version = "0.1.10"
diff --git a/Cargo.toml b/Cargo.toml
index 56745350..2cc5d3af 100644
--- a/Cargo.toml
+++ b/Cargo.toml
@@ -64,6 +64,7 @@ polyanya = "0.2.0"
 bevy_pathmesh = "0.3.0"
 bitflags = "1.3.2"
 iyes_progress = "0.7.1"
+unicode-segmentation = "1.10.1"
 
 # keep the following in sync with Bevy's dependencies
 winit = { version = "0.27", default-features = false }
diff --git a/src/world_interaction/dialog.rs b/src/world_interaction/dialog.rs
index 960d0a97..d9d66ee3 100644
--- a/src/world_interaction/dialog.rs
+++ b/src/world_interaction/dialog.rs
@@ -11,6 +11,7 @@ use bevy::prelude::*;
 use bevy_egui::egui::Color32;
 use bevy_egui::{egui, EguiContext, EguiPlugin};
 use serde::{Deserialize, Serialize};
+use unicode_segmentation::UnicodeSegmentation;
 
 mod resources;
 
@@ -93,13 +94,27 @@ fn show_dialog(
     mut egui_context: ResMut<EguiContext>,
     mut actions_frozen: ResMut<ActionsFrozen>,
     actions: Res<Actions>,
+    time: Res<Time>,
+    mut elapsed_time: Local<f32>,
 ) -> Result<()> {
     let mut current_dialog = match current_dialog {
         Some(current_dialog) => current_dialog,
-        None => return Ok(()),
+        None => {
+            *elapsed_time = 0.0;
+            return Ok(());
+        }
     };
     let height = 150.;
     let current_page = current_dialog.fetch_current_page()?;
+
+    const LETTERS_PER_SECOND: f32 = 50.;
+    let letters_to_display = (LETTERS_PER_SECOND * *elapsed_time) as usize;
+    let text: String = current_page
+        .text
+        .graphemes(true)
+        .take(letters_to_display)
+        .collect();
+
     egui::TopBottomPanel::bottom("Dialog")
         .resizable(false)
         .exact_height(height)
@@ -110,22 +125,26 @@ fn show_dialog(
         .show(egui_context.ctx_mut(), |ui| {
             ui.vertical_centered_justified(|ui| {
                 ui.add_space(5.);
-                ui.label(current_page.text.clone());
+                ui.label(&text);
                 ui.add_space(8.);
-                present_choices(
-                    ui,
-                    &mut commands,
-                    &mut current_dialog,
-                    &active_conditions,
-                    &mut condition_writer,
-                    &mut actions_frozen,
-                    &actions.ui,
-                    current_page.next_page,
-                )
-                .expect("Failed to present dialog choices");
-                ui.add_space(5.);
+                if text == current_page.text {
+                    present_choices(
+                        ui,
+                        &mut commands,
+                        &mut current_dialog,
+                        &active_conditions,
+                        &mut condition_writer,
+                        &mut actions_frozen,
+                        &actions.ui,
+                        current_page.next_page,
+                        &mut elapsed_time,
+                    )
+                    .expect("Failed to present dialog choices");
+                    ui.add_space(5.);
+                }
             });
         });
+    *elapsed_time += time.delta_seconds();
     Ok(())
 }
 
@@ -139,11 +158,13 @@ fn present_choices(
     actions_frozen: &mut ActionsFrozen,
     actions: &UiActions,
     next_page: NextPage,
+    elapsed_time: &mut f32,
 ) -> Result<()> {
     match next_page {
         NextPage::Continue(next_page_id) => {
             if ui.button("1. Continue").clicked() || actions.numbered_choice[1] {
                 current_dialog.current_page = next_page_id;
+                *elapsed_time = 0.0;
             }
         }
         NextPage::Choice(choices) => {
@@ -166,6 +187,7 @@ fn present_choices(
                 condition_writer.send(ConditionAddEvent(choice_id.clone()));
                 current_dialog.last_choice = Some(choice_id);
                 current_dialog.current_page = choice.next_page_id;
+                *elapsed_time = 0.0;
             }
         }
         NextPage::SameAs(other_page_id) => {
@@ -179,6 +201,7 @@ fn present_choices(
                 actions_frozen,
                 actions,
                 next_page,
+                elapsed_time,
             )?;
         }
         NextPage::Exit => {

From 3aa50dd015f85b15ae4ad06322e5593dd5fc02bf Mon Sep 17 00:00:00 2001
From: Jan Hohenheim <jan@hohenheim.ch>
Date: Thu, 16 Feb 2023 20:13:11 +0100
Subject: [PATCH 2/6] Parametrize talking speed

---
 src/world_interaction/dialog.rs           |  5 +++--
 src/world_interaction/dialog/resources.rs | 22 +++++++++++++++++++---
 2 files changed, 22 insertions(+), 5 deletions(-)

diff --git a/src/world_interaction/dialog.rs b/src/world_interaction/dialog.rs
index d9d66ee3..2e3d5cd2 100644
--- a/src/world_interaction/dialog.rs
+++ b/src/world_interaction/dialog.rs
@@ -107,8 +107,9 @@ fn show_dialog(
     let height = 150.;
     let current_page = current_dialog.fetch_current_page()?;
 
-    const LETTERS_PER_SECOND: f32 = 50.;
-    let letters_to_display = (LETTERS_PER_SECOND * *elapsed_time) as usize;
+    const BASE_LETTERS_PER_SECOND: f32 = 60.;
+    let letters_to_display =
+        (BASE_LETTERS_PER_SECOND * current_page.talking_speed * *elapsed_time) as usize;
     let text: String = current_page
         .text
         .graphemes(true)
diff --git a/src/world_interaction/dialog/resources.rs b/src/world_interaction/dialog/resources.rs
index 78a21f31..7874f1c2 100644
--- a/src/world_interaction/dialog/resources.rs
+++ b/src/world_interaction/dialog/resources.rs
@@ -14,7 +14,7 @@ pub struct DialogEvent {
     pub page: Option<PageId>,
 }
 
-#[derive(Debug, Clone, Eq, PartialEq, Resource, Serialize, Deserialize)]
+#[derive(Debug, Clone, PartialEq, Resource, Serialize, Deserialize)]
 pub struct CurrentDialog {
     pub source: Entity,
     pub id: DialogId,
@@ -35,7 +35,7 @@ impl CurrentDialog {
     }
 }
 
-#[derive(Debug, Clone, Eq, PartialEq, Serialize, Deserialize, TypeUuid, Default)]
+#[derive(Debug, Clone, PartialEq, Serialize, Deserialize, TypeUuid, Default)]
 #[uuid = "f7c10043-7196-4ead-a4dd-040c33798a62"]
 pub struct Dialog {
     pub initial_page: Vec<InitialPage>,
@@ -59,12 +59,28 @@ impl InitialPage {
     }
 }
 
-#[derive(Debug, Clone, Eq, PartialEq, Serialize, Deserialize, Default)]
+#[derive(Debug, Clone, PartialEq, Serialize, Deserialize)]
 pub struct Page {
     pub text: String,
+    #[serde(default = "get_default_talking_speed")]
+    pub talking_speed: f32,
     pub next_page: NextPage,
 }
 
+fn get_default_talking_speed() -> f32 {
+    1.
+}
+
+impl Default for Page {
+    fn default() -> Self {
+        Self {
+            text: default(),
+            talking_speed: get_default_talking_speed(),
+            next_page: default(),
+        }
+    }
+}
+
 #[derive(Debug, Clone, Eq, PartialEq, Serialize, Deserialize)]
 pub enum NextPage {
     /// There is only one automatic option for the next page

From a1370d5455a0790c5d56d3948ad82cf993613c1f Mon Sep 17 00:00:00 2001
From: Jan Hohenheim <jan@hohenheim.ch>
Date: Thu, 16 Feb 2023 21:05:25 +0100
Subject: [PATCH 3/6] Style dialog window

---
 src/world_interaction/dialog.rs | 23 ++++++++++++++++++-----
 1 file changed, 18 insertions(+), 5 deletions(-)

diff --git a/src/world_interaction/dialog.rs b/src/world_interaction/dialog.rs
index 2e3d5cd2..eb9f4648 100644
--- a/src/world_interaction/dialog.rs
+++ b/src/world_interaction/dialog.rs
@@ -8,7 +8,9 @@ pub use crate::world_interaction::dialog::resources::{
 use crate::GameState;
 use anyhow::{Context, Result};
 use bevy::prelude::*;
-use bevy_egui::egui::Color32;
+use bevy::window::WindowId;
+use bevy::winit::WinitWindows;
+use bevy_egui::egui::{Color32, Pos2};
 use bevy_egui::{egui, EguiContext, EguiPlugin};
 use serde::{Deserialize, Serialize};
 use unicode_segmentation::UnicodeSegmentation;
@@ -96,6 +98,7 @@ fn show_dialog(
     actions: Res<Actions>,
     time: Res<Time>,
     mut elapsed_time: Local<f32>,
+    windows: NonSend<WinitWindows>,
 ) -> Result<()> {
     let mut current_dialog = match current_dialog {
         Some(current_dialog) => current_dialog,
@@ -104,7 +107,6 @@ fn show_dialog(
             return Ok(());
         }
     };
-    let height = 150.;
     let current_page = current_dialog.fetch_current_page()?;
 
     const BASE_LETTERS_PER_SECOND: f32 = 60.;
@@ -116,15 +118,26 @@ fn show_dialog(
         .take(letters_to_display)
         .collect();
 
-    egui::TopBottomPanel::bottom("Dialog")
+    let total_window_size = windows
+        .get_window(WindowId::primary())
+        .context("Failed to get primary window")?
+        .inner_size();
+    let dialog_size = egui::Vec2::new(500., 150.);
+    egui::Window::new("Dialog")
         .resizable(false)
-        .exact_height(height)
+        .fixed_pos(Pos2::new(30., 400.))
+        .fixed_size(dialog_size)
+        .default_size(dialog_size)
+        .anchor(egui::Align2::CENTER_BOTTOM, egui::Vec2::new(0., -30.))
+        .collapsible(false)
+        .title_bar(false)
         .frame(egui::Frame {
             fill: Color32::from_black_alpha(250),
             ..default()
         })
         .show(egui_context.ctx_mut(), |ui| {
-            ui.vertical_centered_justified(|ui| {
+            ui.style_mut().visuals.button_frame = false;
+            ui.vertical(|ui| {
                 ui.add_space(5.);
                 ui.label(&text);
                 ui.add_space(8.);

From c6ea11a1d361dfbb5476a412c79b074e98bac4bb Mon Sep 17 00:00:00 2001
From: Jan Hohenheim <jan@hohenheim.ch>
Date: Thu, 16 Feb 2023 22:51:06 +0100
Subject: [PATCH 4/6] Style dialog

---
 assets/dialogs/follower.dlg.ron |  1 +
 src/world_interaction/dialog.rs | 72 ++++++++++++++++++---------------
 2 files changed, 41 insertions(+), 32 deletions(-)

diff --git a/assets/dialogs/follower.dlg.ron b/assets/dialogs/follower.dlg.ron
index dbfdfa49..57658831 100644
--- a/assets/dialogs/follower.dlg.ron
+++ b/assets/dialogs/follower.dlg.ron
@@ -54,6 +54,7 @@
         ),
         "page:exit": (
             text: "Goodbye.\n*The creature\'s gaze shifts ever so slightly. It now looks just past you into the void.*",
+            talking_speed: 2.,
             next_page: Exit,
         ),
         "page:main-choice-unnest": (
diff --git a/src/world_interaction/dialog.rs b/src/world_interaction/dialog.rs
index eb9f4648..07e2f5bc 100644
--- a/src/world_interaction/dialog.rs
+++ b/src/world_interaction/dialog.rs
@@ -2,15 +2,13 @@ use crate::file_system_interaction::asset_loading::DialogAssets;
 use crate::player_control::actions::{Actions, ActionsFrozen, UiActions};
 use crate::util::log_error::log_errors;
 use crate::world_interaction::condition::{ActiveConditions, ConditionAddEvent, ConditionId};
+use crate::world_interaction::dialog::resources::Page;
 pub use crate::world_interaction::dialog::resources::{
     CurrentDialog, Dialog, DialogEvent, DialogId, InitialPage, NextPage,
 };
 use crate::GameState;
 use anyhow::{Context, Result};
 use bevy::prelude::*;
-use bevy::window::WindowId;
-use bevy::winit::WinitWindows;
-use bevy_egui::egui::{Color32, Pos2};
 use bevy_egui::{egui, EguiContext, EguiPlugin};
 use serde::{Deserialize, Serialize};
 use unicode_segmentation::UnicodeSegmentation;
@@ -98,7 +96,6 @@ fn show_dialog(
     actions: Res<Actions>,
     time: Res<Time>,
     mut elapsed_time: Local<f32>,
-    windows: NonSend<WinitWindows>,
 ) -> Result<()> {
     let mut current_dialog = match current_dialog {
         Some(current_dialog) => current_dialog,
@@ -108,40 +105,35 @@ fn show_dialog(
         }
     };
     let current_page = current_dialog.fetch_current_page()?;
+    let dialog_text = create_dialog_rich_text(&current_page, *elapsed_time);
 
-    const BASE_LETTERS_PER_SECOND: f32 = 60.;
-    let letters_to_display =
-        (BASE_LETTERS_PER_SECOND * current_page.talking_speed * *elapsed_time) as usize;
-    let text: String = current_page
-        .text
-        .graphemes(true)
-        .take(letters_to_display)
-        .collect();
-
-    let total_window_size = windows
-        .get_window(WindowId::primary())
-        .context("Failed to get primary window")?
-        .inner_size();
     let dialog_size = egui::Vec2::new(500., 150.);
     egui::Window::new("Dialog")
-        .resizable(false)
-        .fixed_pos(Pos2::new(30., 400.))
-        .fixed_size(dialog_size)
-        .default_size(dialog_size)
+        .resizable(true)
         .anchor(egui::Align2::CENTER_BOTTOM, egui::Vec2::new(0., -30.))
         .collapsible(false)
+        .resizable(false)
         .title_bar(false)
         .frame(egui::Frame {
-            fill: Color32::from_black_alpha(250),
+            fill: egui::Color32::from_black_alpha(220),
+            inner_margin: egui::style::Margin::same(25.),
+            rounding: egui::Rounding::same(30.0),
             ..default()
         })
         .show(egui_context.ctx_mut(), |ui| {
-            ui.style_mut().visuals.button_frame = false;
+            ui.set_width(dialog_size.x);
+            ui.set_height(dialog_size.y);
+
+            let style = ui.style_mut();
+            style.visuals.button_frame = false;
+            style.visuals.override_text_color = Some(egui::Color32::WHITE);
             ui.vertical(|ui| {
                 ui.add_space(5.);
-                ui.label(&text);
-                ui.add_space(8.);
-                if text == current_page.text {
+                ui.label(dialog_text.clone());
+                if dialog_text.text() == current_page.text {
+                    ui.add_space(3.);
+                    ui.separator();
+                    ui.add_space(8.);
                     present_choices(
                         ui,
                         &mut commands,
@@ -154,7 +146,6 @@ fn show_dialog(
                         &mut elapsed_time,
                     )
                     .expect("Failed to present dialog choices");
-                    ui.add_space(5.);
                 }
             });
         });
@@ -176,7 +167,8 @@ fn present_choices(
 ) -> Result<()> {
     match next_page {
         NextPage::Continue(next_page_id) => {
-            if ui.button("1. Continue").clicked() || actions.numbered_choice[1] {
+            let text = create_choice_rich_text(0, "Continue");
+            if ui.button(text).clicked() || actions.numbered_choice[1] {
                 current_dialog.current_page = next_page_id;
                 *elapsed_time = 0.0;
             }
@@ -191,9 +183,8 @@ fn present_choices(
                 })
                 .enumerate()
             {
-                let index = index + 1;
-                let text = format!("{}. {}", index, choice.text);
-                if ui.button(text).clicked() || actions.numbered_choice[index] {
+                let text = create_choice_rich_text(index, &choice.text);
+                if ui.button(text).clicked() || actions.numbered_choice[index + 1] {
                     picked_choice = Some((choice_id.clone(), choice.clone()));
                 }
             }
@@ -219,7 +210,8 @@ fn present_choices(
             )?;
         }
         NextPage::Exit => {
-            if ui.button("1. Exit").clicked() || actions.numbered_choice[1] {
+            let text = create_choice_rich_text(0, "Exit");
+            if ui.button(text).clicked() || actions.numbered_choice[1] {
                 commands.remove_resource::<CurrentDialog>();
                 actions_frozen.unfreeze();
             }
@@ -228,6 +220,22 @@ fn present_choices(
     Ok(())
 }
 
+fn create_dialog_rich_text(page: &Page, elapsed_time: f32) -> egui::RichText {
+    const BASE_LETTERS_PER_SECOND: f32 = 60.;
+    let letters_to_display = (BASE_LETTERS_PER_SECOND * page.talking_speed * elapsed_time) as usize;
+    let text: String = page.text.graphemes(true).take(letters_to_display).collect();
+    egui::RichText::new(text)
+        .color(egui::Color32::from_gray(250))
+        .size(16.)
+}
+
+fn create_choice_rich_text(index: usize, text: &str) -> egui::RichText {
+    let text = format!("{}. {}", index + 1, text);
+    egui::RichText::new(&text)
+        .color(egui::Color32::from_gray(220))
+        .size(14.)
+}
+
 fn was_just_picked(current_dialog: &CurrentDialog, choice_id: &ConditionId) -> bool {
     current_dialog
         .last_choice

From 6aa4246526a3bbff16844a96ccd6d8d395ce07bc Mon Sep 17 00:00:00 2001
From: Jan Hohenheim <jan@hohenheim.ch>
Date: Thu, 16 Feb 2023 23:16:28 +0100
Subject: [PATCH 5/6] Implement pause menu

---
 src/ingame_menu.rs                         | 51 ++++++++++++++++++++++
 src/lib.rs                                 |  6 ++-
 src/player_control/actions.rs              |  3 ++
 src/player_control/actions/game_control.rs |  3 ++
 4 files changed, 62 insertions(+), 1 deletion(-)
 create mode 100644 src/ingame_menu.rs

diff --git a/src/ingame_menu.rs b/src/ingame_menu.rs
new file mode 100644
index 00000000..41e2cc42
--- /dev/null
+++ b/src/ingame_menu.rs
@@ -0,0 +1,51 @@
+use crate::player_control::actions::{Actions, ActionsFrozen};
+use crate::GameState;
+use bevy::prelude::*;
+use bevy_egui::{egui, EguiContext};
+
+/// Handles the pause menu accessed while playing the game via ESC.
+pub struct IngameMenuPlugin;
+
+impl Plugin for IngameMenuPlugin {
+    fn build(&self, app: &mut App) {
+        {
+            app.add_system_set(SystemSet::on_update(GameState::Playing).with_system(handle_pause));
+        }
+    }
+}
+
+fn handle_pause(
+    mut time: ResMut<Time>,
+    actions: Res<Actions>,
+    mut actions_frozen: ResMut<ActionsFrozen>,
+    mut egui_context: ResMut<EguiContext>,
+    mut paused: Local<bool>,
+) {
+    let toggled = actions.ui.toggle_pause;
+    if *paused {
+        if toggled {
+            *paused = false;
+            time.unpause();
+            actions_frozen.unfreeze();
+        } else {
+            egui::CentralPanel::default()
+                .frame(egui::Frame {
+                    fill: egui::Color32::from_black_alpha(240),
+                    ..default()
+                })
+                .show(egui_context.ctx_mut(), |ui| {
+                    ui.vertical_centered_justified(|ui| {
+                        ui.visuals_mut().override_text_color = Some(egui::Color32::WHITE);
+                        ui.add_space(100.0);
+                        ui.heading("Game Paused");
+                        ui.separator();
+                        ui.label("Press ESC to resume");
+                    });
+                });
+        }
+    } else if toggled {
+        *paused = true;
+        time.pause();
+        actions_frozen.freeze();
+    }
+}
diff --git a/src/lib.rs b/src/lib.rs
index bfd05c07..a8c1df42 100644
--- a/src/lib.rs
+++ b/src/lib.rs
@@ -15,6 +15,7 @@ pub mod bevy_config;
 #[cfg(feature = "dev")]
 pub mod dev;
 pub mod file_system_interaction;
+pub mod ingame_menu;
 pub mod level_instantiation;
 pub mod menu;
 pub mod movement;
@@ -27,6 +28,7 @@ pub use crate::bevy_config::BevyConfigPlugin;
 #[cfg(feature = "dev")]
 use crate::dev::DevPlugin;
 use crate::file_system_interaction::FileSystemInteractionPlugin;
+use crate::ingame_menu::IngameMenuPlugin;
 use crate::level_instantiation::LevelInstantiationPlugin;
 use crate::menu::MenuPlugin;
 use crate::movement::MovementPlugin;
@@ -57,6 +59,7 @@ enum GameState {
 /// - [`FileSystemInteractionPlugin`]: Handles the loading and saving of games.
 /// - [`ShaderPlugin`]: Handles the shaders.
 /// - [`DevPlugin`]: Handles the dev tools.
+/// - [`IngameMenuPlugin`]: Handles the ingame menu accessed via ESC.
 pub struct GamePlugin;
 
 impl Plugin for GamePlugin {
@@ -69,7 +72,8 @@ impl Plugin for GamePlugin {
             .add_plugin(WorldInteractionPlugin)
             .add_plugin(LevelInstantiationPlugin)
             .add_plugin(FileSystemInteractionPlugin)
-            .add_plugin(ShaderPlugin);
+            .add_plugin(ShaderPlugin)
+            .add_plugin(IngameMenuPlugin);
         #[cfg(feature = "dev")]
         app.add_plugin(DevPlugin);
     }
diff --git a/src/player_control/actions.rs b/src/player_control/actions.rs
index 4b976e9c..031be10f 100644
--- a/src/player_control/actions.rs
+++ b/src/player_control/actions.rs
@@ -72,7 +72,9 @@ pub struct CameraActions {
 #[derive(Debug, Clone, PartialEq, Reflect, FromReflect, Default, Serialize, Deserialize)]
 #[reflect(Serialize, Deserialize)]
 pub struct UiActions {
+    #[cfg(feature = "dev")]
     pub toggle_editor: bool,
+    pub toggle_pause: bool,
     pub numbered_choice: [bool; 10],
 }
 
@@ -85,6 +87,7 @@ pub fn set_actions(
 ) {
     *actions = default();
     actions.ui.toggle_editor = GameControl::ToggleEditor.just_pressed(&keyboard_input);
+    actions.ui.toggle_pause = GameControl::TogglePause.just_pressed(&keyboard_input);
     for i in 0..=9 {
         actions.ui.numbered_choice[i] =
             GameControl::NumberedChoice(i as u16).just_pressed(&keyboard_input);
diff --git a/src/player_control/actions/game_control.rs b/src/player_control/actions/game_control.rs
index 9fedfd89..dd108eee 100644
--- a/src/player_control/actions/game_control.rs
+++ b/src/player_control/actions/game_control.rs
@@ -8,6 +8,7 @@ pub enum GameControl {
     Sprint,
     Jump,
     ToggleEditor,
+    TogglePause,
     Interact,
     NumberedChoice(u16),
 }
@@ -149,6 +150,8 @@ generate_bindings! {
             #[cfg(target_os = "linux")] 0x10,
             #[cfg(target_arch = "wasm32")] 0x10,
         ),
+    ],
+    GameControl::TogglePause => [
         // Esc
         ScanCode(
             #[cfg(target_os = "macos")] 53,

From 561c310c5b8ee64c46767c5ff57f1346eccd10e0 Mon Sep 17 00:00:00 2001
From: Jan Hohenheim <jan@hohenheim.ch>
Date: Thu, 16 Feb 2023 23:17:30 +0100
Subject: [PATCH 6/6] Make clippy happy

---
 src/world_interaction/dialog.rs | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/world_interaction/dialog.rs b/src/world_interaction/dialog.rs
index 07e2f5bc..3f6027c0 100644
--- a/src/world_interaction/dialog.rs
+++ b/src/world_interaction/dialog.rs
@@ -86,6 +86,7 @@ fn set_current_dialog(
     Ok(())
 }
 
+#[allow(clippy::too_many_arguments)]
 fn show_dialog(
     mut commands: Commands,
     current_dialog: Option<ResMut<CurrentDialog>>,
@@ -231,7 +232,7 @@ fn create_dialog_rich_text(page: &Page, elapsed_time: f32) -> egui::RichText {
 
 fn create_choice_rich_text(index: usize, text: &str) -> egui::RichText {
     let text = format!("{}. {}", index + 1, text);
-    egui::RichText::new(&text)
+    egui::RichText::new(text)
         .color(egui::Color32::from_gray(220))
         .size(14.)
 }