From 3a7f8fc1a7dbaf2afc22807b177cc08a3bde17ea Mon Sep 17 00:00:00 2001
From: iaada <iaada@users.noreply.github.com>
Date: Fri, 5 Sep 2025 17:24:50 -0500
Subject: [PATCH] update for glue/lube tube inhand

---
 .../Objects/Consumable/Drinks/drinks_base.yml       | 11 ++++++-----
 .../Objects/Consumable/Drinks/drinks_fun.yml        | 13 ++++++++-----
 2 files changed, 14 insertions(+), 10 deletions(-)

diff --git a/Resources/Prototypes/Entities/Objects/Consumable/Drinks/drinks_base.yml b/Resources/Prototypes/Entities/Objects/Consumable/Drinks/drinks_base.yml
index 80e7fa10cb..c4e8b020e7 100644
--- a/Resources/Prototypes/Entities/Objects/Consumable/Drinks/drinks_base.yml
+++ b/Resources/Prototypes/Entities/Objects/Consumable/Drinks/drinks_base.yml
@@ -18,7 +18,7 @@
   - type: Edible
     edible: Drink # Namesake. This entity is a sort of cup you can drink directly from.
     destroyOnEmpty: false
-    utensil: None
+    utensil: None # Spoons don't fit in cups, also conflicts with bar spoon stirring
     solution: &sol drink # solution name alias
   - type: Spillable
     solution: *sol
@@ -81,16 +81,17 @@
   parent: DrinkBaseOpenable
   id: DrinkBaseOpenableOpen
   components:
-  - type: Sprite
-    layers:
-    - state: icon_open
-      map: ["enum.OpenableVisuals.Layer"]
   - type: Openable
     opened: true
   - type: Sealable
     sealed: false
+#  - type: Sprite # Needs to be done manually, since most drinks have more than one layer. Or just let the visualizer handle it
+#    layers:
+#    - state: icon_open
+#      map: ["enum.OpenableVisuals.Layer"]
 
 ## Bases for visuals
+# TODO standardize state names for fill levels and openable visuals
 
 # Basic visualizer for an openable entity. Requires DrinkBaseOpenable
 - type: entity
diff --git a/Resources/Prototypes/Entities/Objects/Consumable/Drinks/drinks_fun.yml b/Resources/Prototypes/Entities/Objects/Consumable/Drinks/drinks_fun.yml
index 504c336a21..1197356553 100644
--- a/Resources/Prototypes/Entities/Objects/Consumable/Drinks/drinks_fun.yml
+++ b/Resources/Prototypes/Entities/Objects/Consumable/Drinks/drinks_fun.yml
@@ -1,6 +1,6 @@
 - type: entity
   abstract: true
-  parent: [ DrinkBaseMaterialPlastic, DrinkBase, DrinkBaseOpenable, DrinkVisualsOpenable ] # todo Should use DrinkVisualAll, but sprite has no inhand
+  parent: [ DrinkBaseMaterialPlastic, DrinkBase, DrinkBaseOpenable, DrinkVisualsAllFilled ]
   id: BaseSqueezeBottle
   components:
   - type: Openable
@@ -20,12 +20,15 @@
         maxVol: 30
   - type: SolutionContainerVisuals
     maxFillLevels: 6
-    fillBaseName: fill
+    fillBaseName: fill # TODO rename to "fill-"
+  - type: GenericVisualizer
+    visuals:
+      enum.OpenableVisuals.Opened:
+        enum.OpenableVisuals.Layer:
+          True: {state: "icon_open"}
+          False: {state: "icon"}
   - type: TrashOnSolutionEmpty
     solution: drink
-  - type: ExaminableSolution # Remove when DrinkVisualAll is added
-    examinableWhileClosed: true
-    heldOnly: false
 
 - type: entity
   parent: BaseSqueezeBottle
-- 
2.51.2