strings: Common string context menu

Author: cohaereo

src/gui/common.rs

@@ -21,6 +21,8 @@ pub trait ResponseExt {
         texture_cache: Option<&TextureCache>,
         is_texture: bool,
     ) -> Self;
+
+    fn string_context(self, s: &str, hash: Option<u32>) -> Self;
 }
 
 impl ResponseExt for egui::Response {
@@ -76,6 +78,23 @@ impl ResponseExt for egui::Response {
             tag_hover_ui(ui, tag);
         })
     }
+
+    fn string_context(self, string: &str, hash: Option<u32>) -> Self {
+        self.context_menu(|ui| {
+            if ui.selectable_label(false, "Copy string").clicked() {
+                ui.ctx().copy_text(string.to_string());
+                ui.close();
+            }
+            if let Some(hash) = hash
+                && ui.selectable_label(false, "Copy hash").clicked()
+            {
+                ui.ctx().copy_text(format!("{:08X}", hash));
+                ui.close();
+            }
+        });
+
+        self
+    }
 }
 
 pub fn export_texture(texture_cache: &TextureCache, tag: TagHash) {

src/gui/raw_strings.rs

@@ -111,7 +111,7 @@ impl View for RawStringsView {
                     string_height,
                     self.strings_vec_filtered.len(),
                     |ui, range| {
-                        for (i, string, tags, _hash) in self.strings_vec_filtered[range].iter() {
+                        for (i, string, tags, hash) in self.strings_vec_filtered[range].iter() {
                             let response = ui.selectable_label(
                                 *i == self.selected_stringset,
                                 format!(
@@ -129,12 +129,9 @@ impl View for RawStringsView {
                                 self.selected_stringset = *i;
                             }
 
-                            response.on_hover_text(string).context_menu(|ui| {
-                                if ui.selectable_label(false, "Copy string").clicked() {
-                                    ui.ctx().copy_text(string.clone());
-                                    ui.close();
-                                }
-                            });
+                            response
+                                .on_hover_text(string)
+                                .string_context(string, Some(*hash));
                         }
                     },
                 );

src/gui/strings.rs

@@ -328,12 +328,7 @@ impl View for StringsView {
                                     }
                                 }
 
-                                response.context_menu(|ui| {
-                                    if ui.selectable_label(false, "Copy string").clicked() {
-                                        ui.ctx().copy_text(strings[0].clone());
-                                        ui.close();
-                                    }
-                                });
+                                response.string_context(&strings[0], Some(*hash));
                             }
                         },
                     );

src/gui/tag.rs

@@ -1147,6 +1147,7 @@ impl View for TagView {
                                                         ),
                                                     )
                                                     .on_hover_text(strings.join("\n"))
+                                                    .string_context(&strings[0], Some(*hash))
                                                     .clicked();
                                                 } else {
                                                     ui.selectable_label(
@@ -1156,6 +1157,7 @@ impl View for TagView {
                                                             strings[0], hash, offset
                                                         ),
                                                     )
+                                                    .string_context(&strings[0], Some(*hash))
                                                     .clicked();
                                                 }
                                             }
@@ -1271,7 +1273,8 @@ impl View for TagView {
                                                                     .iter()
                                                                     .map(|(s, _)| s)
                                                                     .join("\n"),
-                                                            );
+                                                            )
+                                                            .string_context(&strings[0].0, Some(*hash));
 
                                                         (response, strings[current_string].1)
                                                     } else {
@@ -1287,7 +1290,8 @@ impl View for TagView {
                                                                 strings[0].0, hash, offset
                                                             ))
                                                             .color(color),
-                                                        );
+                                                        )
+                                                        .string_context(&strings[0].0, Some(*hash));
 
                                                         (response, strings[0].1)
                                                     };