search_message improvement

Author: alanpoon

src/home/search_message.rs

@@ -31,6 +31,9 @@ use crate::{
     utils::unix_time_millis_to_datetime,
 };
 
+// Consider extracting this constant
+const SEARCH_HIGHLIGHT_COLOR: &str = "#fcdb03";
+
 live_design! {
     use link::theme::*;
     use link::shaders::*;
@@ -289,7 +292,8 @@ fn apply_highlights(text: String, highlights: &[String]) -> String {
             &result,
             |caps: &regex::Captures| {
                 format!(
-                    "<span data-mx-bg-color=\"#fcdb03\">{}</span>",
+                    "<span data-mx-bg-color=\"{}\">{}</span>",
+                    SEARCH_HIGHLIGHT_COLOR,
                     &caps[0].to_string() // Preserves original case
                 )
             },
@@ -328,29 +332,29 @@ pub fn highlight_search_terms_in_message(message: &mut RoomMessageEventContent,
     }
 
     match &mut message.msgtype {
-        MessageType::Text(text) => {
-            apply_highlights_to_content!(text);
+        MessageType::Text(content) => {
+            apply_highlights_to_content!(content);
         }
-        MessageType::Notice(text) => {
-            apply_highlights_to_content!(text);
+        MessageType::Notice(content) => {
+            apply_highlights_to_content!(content);
         }
-        MessageType::Emote(text) => {
-            apply_highlights_to_content!(text);
+        MessageType::Emote(content) => {
+            apply_highlights_to_content!(content);
         }
-        MessageType::File(text) => {
-            apply_highlights_to_content!(text);
+        MessageType::File(content) => {
+            apply_highlights_to_content!(content);
         }
-        MessageType::Image(text) => {
-            apply_highlights_to_content!(text);
+        MessageType::Image(content) => {
+            apply_highlights_to_content!(content);
         }
-        MessageType::Video(text) => {
-            apply_highlights_to_content!(text);
+        MessageType::Video(content) => {
+            apply_highlights_to_content!(content);
         }
-        MessageType::Audio(text) => {
-            apply_highlights_to_content!(text);
+        MessageType::Audio(content) => {
+            apply_highlights_to_content!(content);
         }
-        MessageType::VerificationRequest(text) => {
-            apply_highlights_to_content!(text);
+        MessageType::VerificationRequest(content) => {
+            apply_highlights_to_content!(content);
         }
         _ => {}
     }