From 2b083be998188187bf59ace622669a076a576f74 Mon Sep 17 00:00:00 2001
From: Aaron Fenyes <aaron.fenyes@fareycircles.ooo>
Date: Thu, 14 Nov 2024 00:16:32 -0800
Subject: [PATCH] Explain emoji icon kludge

---
 app-proto/main.css          | 4 ++++
 app-proto/src/add_remove.rs | 2 +-
 2 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/app-proto/main.css b/app-proto/main.css
index d7a71aa..b9fc0a1 100644
--- a/app-proto/main.css
+++ b/app-proto/main.css
@@ -47,6 +47,10 @@ body {
 }
 
 /* KLUDGE */
+/*
+  for convenience, we're using emoji as temporary icons for some buttons. these
+  buttons need to be displayed in an emoji font
+*/
 #add-remove > button.emoji {
   font-family: 'Noto Emoji', sans-serif;
 }
diff --git a/app-proto/src/add_remove.rs b/app-proto/src/add_remove.rs
index bea20f9..5590b6c 100644
--- a/app-proto/src/add_remove.rs
+++ b/app-proto/src/add_remove.rs
@@ -173,7 +173,7 @@ pub fn AddRemove() -> View {
                 }
             ) { "+" }
             button(
-                class="emoji", /* KLUDGE */
+                class="emoji", /* KLUDGE */ // for convenience, we're using an emoji as a temporary icon for this button
                 disabled={
                     let state = use_context::<AppState>();
                     state.selection.with(|sel| sel.len() != 2)