chore: Factor out the hyperlink generation logic

Author: scrabsha

src/renderer/render.rs

@@ -359,17 +359,14 @@ fn render_title(
         label_width += title.level().as_str().len();
         if let Some(Id { id: Some(id), url }) = &title.id() {
             buffer.append(buffer_msg_line_offset, "[", label_style);
-            if let Some(url) = url.as_ref() {
-                buffer.append(
-                    buffer_msg_line_offset,
-                    &format!("\x1B]8;;{url}\x1B\\"),
-                    label_style,
-                );
-            }
-            buffer.append(buffer_msg_line_offset, id, label_style);
-            if url.is_some() {
-                buffer.append(buffer_msg_line_offset, "\x1B]8;;\x1B\\", label_style);
-            }
+            let link = Hyperlink {
+                url: url.as_deref(),
+            };
+            buffer.append(
+                buffer_msg_line_offset,
+                &format!("{link}{id}{link:#}"),
+                label_style,
+            );
             buffer.append(buffer_msg_line_offset, "]", label_style);
             label_width += 2 + id.len();
         }
@@ -2759,6 +2756,29 @@ fn newline_count(body: &str) -> usize {
     }
 }
 
+struct Hyperlink<D: fmt::Display> {
+    url: Option<D>,
+}
+
+impl<D: fmt::Display> Default for Hyperlink<D> {
+    fn default() -> Self {
+        Self { url: None }
+    }
+}
+
+impl<D: fmt::Display> fmt::Display for Hyperlink<D> {
+    fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
+        let Some(url) = self.url.as_ref() else {
+            return Ok(());
+        };
+        if f.alternate() {
+            write!(f, "\x1B]8;;\x1B\\")
+        } else {
+            write!(f, "\x1B]8;;{url}\x1B\\")
+        }
+    }
+}
+
 #[cfg(test)]
 mod test {
     use super::{OUTPUT_REPLACEMENTS, newline_count};