From bdade0a8c233e456030785e9bc38b7db0cf226b8 Mon Sep 17 00:00:00 2001
From: Kevin Cope <inivekin@gmail.com>
Date: Sun, 16 May 2021 09:36:08 +0800
Subject: [PATCH] Moves button list into MIXIN instance

---
 basis/ui/gadgets/buttons/buttons.factor       | 10 --------
 .../button-list/button-list-tests.factor      |  0
 basis/ui/tools/button-list/button-list.factor | 23 +++++++++++++++++--
 3 files changed, 21 insertions(+), 12 deletions(-)
 delete mode 100644 basis/ui/tools/button-list/button-list-tests.factor

diff --git a/basis/ui/gadgets/buttons/buttons.factor b/basis/ui/gadgets/buttons/buttons.factor
index 4ced17e86c..c7609dbe06 100644
--- a/basis/ui/gadgets/buttons/buttons.factor
+++ b/basis/ui/gadgets/buttons/buttons.factor
@@ -11,16 +11,6 @@ IN: ui.gadgets.buttons
 
 TUPLE: button < border pressed? selected? quot tooltip ;
 
-SYMBOL: active-buttons
-active-buttons [ H{ } ] initialize
-
-: label-from-button ( button -- str )
-    children>> [ label? ] find swap [ text>> ] [ drop "unknown" ] if ;
-
-M: button graft* dup label-from-button active-buttons get set-at ;
-
-M: button ungraft* label-from-button active-buttons get delete-at ;
-
 <PRIVATE
 
 : find-button ( gadget -- button )
diff --git a/basis/ui/tools/button-list/button-list-tests.factor b/basis/ui/tools/button-list/button-list-tests.factor
deleted file mode 100644
index e69de29bb2..0000000000
diff --git a/basis/ui/tools/button-list/button-list.factor b/basis/ui/tools/button-list/button-list.factor
index 07ad1a2ccc..83a4733e25 100644
--- a/basis/ui/tools/button-list/button-list.factor
+++ b/basis/ui/tools/button-list/button-list.factor
@@ -4,13 +4,32 @@ USING: accessors arrays assocs classes kernel math.rectangles
 models models.arrow namespaces sequences ui ui.gadgets
 ui.gadgets.buttons ui.gadgets.glass ui.gadgets.labeled 
 ui.gadgets.presentations ui.gadgets.search-tables
-ui.gadgets.tables ui.gadgets.wrappers ui.gestures ui.theme 
-ui.tools.browser.popups ui.tools.common ;
+ui.gadgets.labels ui.gadgets.tables ui.gadgets.wrappers ui.gestures
+ui.theme ui.tools.browser.popups ui.tools.common ;
 FROM: ui.gadgets.wrappers => wrapper ;
 IN: ui.tools.button-list
 
 TUPLE: button-list-popup < wrapper ;
 
+MIXIN: clickable
+
+SYMBOL: active-buttons
+active-buttons [ H{ } ] initialize
+
+: label-from-button ( button -- str/f )
+    children>> [ label? ] find swap [ text>> ] [ drop f ] if ;
+
+: store-labelled-button ( button -- str/f )
+    dup label-from-button [ [ active-buttons get set-at ] keep ] [ drop f ] if* ;
+
+: remove-labelled-button ( button -- str/f )
+    label-from-button [ dup active-buttons get delete-at ] [ f ] if* ;
+
+M: clickable graft* [ store-labelled-button drop ] [ call-next-method ] bi ;
+M: button ungraft* [ remove-labelled-button drop ] [ call-next-method ] bi ;
+
+INSTANCE: button clickable
+
 : <active-buttons-table> ( model -- table )
     [ keys [ ">" swap 2array ] map ] <arrow> trivial-renderer [ second ] <search-table> 
     dup table>>
-- 
2.34.1