TODO updates

TODO.md

@@ -2,22 +2,26 @@
 
 ## Features
 
-- Saving and loading games
-- Character select
-- Stats
-- Better open / closed state graphics for rooms
-- Displaying sprites (characters, objects)
-- Inventory
-- Room labels
-- Background music
-- Sound effects
+- [ ] Saving and loading games
+- [ ] Character select
+- [ ] Stats
+- [ ] Better open / closed state graphics for rooms
+- [ ] Displaying sprites (characters, objects)
+- [ ] Inventory
+- [ ] Room labels
+- [ ] Background music
+- [ ] Sound effects
+
+## Bugs
+
+- [ ] Fix console rendering issues
 
 ### Dialogue
 
-- Text formatting
-- Character profiles / speaker ID
+- [ ] Text formatting
+- [ ] Character profiles / speaker ID
 
 ## Code architecture
 
-- Move dialogue- and option-drawing out of `Game.hx`
-- Use domkit for dialogue
+- [x] Move dialogue- and option-drawing out of `Game.hx`
+- [x] Use domkit for dialogue

compile-common.hxml

@@ -3,5 +3,5 @@
 -lib heaps:git:https://github.com/HeapsIO/heaps.git
 -lib tiledhx
 -lib hxyarn
--main Main
+-main Game
 -lib domkit

res/style.css

@@ -52,3 +52,35 @@ menubutton:hover text {
 menubutton:active {
 	background: #666;
 }
+
+flow.dialogue-container {
+	fill-width: true;
+	height: 20;
+	layout: vertical;
+}
+
+dialogue-box {
+	fill-width: true;
+	height: 20;
+	/* background: tile("ui.png", 5, 5); */
+	/* background-tile-pos: 1 1; */
+	background: blue;
+	padding: 10;
+	alpha: 0.9;
+}
+
+dialogue-options {
+	fill-width: true;
+	layout: vertical;
+}
+
+dialogue-option {
+	fill-width: true;
+	background: green;
+	padding: 10;
+	alpha: 0.9;
+}
+
+dialogue-option.disabled {
+	alpha: 0.5;
+}

src/Game.hx

@@ -7,13 +7,8 @@ import scene.GameScene;
 import event.EventBus;
 import event.ChangeSceneEvent;
 import Const;
-import dialogue.DialogueBox;
+import listeners.DialogueBoxController;
 import dialogue.DialogueManager;
-import dialogue.event.LineShown;
-import dialogue.event.OptionsShown;
-import dialogue.event.OptionSelected;
-import dialogue.event.DialogueComplete;
-import dialogue.event.NextLine;
 import dialogue.command.OpenRoomCommand;
 
 @:uiComp("game-container")
@@ -27,28 +22,26 @@ class GameContainer extends h2d.Flow implements h2d.domkit.Object {
 	public function new(?parent) {
 		super(parent);
 		initComponent();
+		this.maxWidth = Const.W;
+		this.minWidth = Const.W;
 	}
 }
 
 class Game extends hxd.App {
-	public var font:h2d.Font;
-	public var curDialogue:h2d.Object;
-	public var options:Array<h2d.ScaleGrid> = [];
-	public var dialogueManager:DialogueManager;
-
-	public var globalEventBus:EventBus;
-
+	public static var loader = new tiled.Tiled();
 	public static var current:Game; // Current Game (singleton)
 
-	var layer:Layers;
-
+	public var font:h2d.Font;
+	public var options:Array<h2d.ScaleGrid> = [];
+	public var dialogueManager:DialogueManager;
 	public var root:GameContainer;
 	public var console:Console;
+	public var globalEventBus:EventBus;
 
+	var dialogueBoxController:DialogueBoxController;
+	var layer:Layers;
 	var style = null;
 
-	public static var loader = new tiled.Tiled();
-
 	override function init() {
 		s2d.scaleMode = Stretch(Const.W, Const.H);
 		layer = new Layers(s2d);
@@ -62,7 +55,6 @@ class Game extends hxd.App {
 
 		root = new GameContainer();
 		root.horizontalAlign = root.verticalAlign = Middle;
-		// s2d.add(root);
 		layer.addChildAt(root, 1);
 		onResize();
 
@@ -72,14 +64,13 @@ class Game extends hxd.App {
 		style.addObject(root);
 
 		font = Res.font.minecraftia_regular_6.toFont();
+		font.resizeTo(18);
 
 		#if debug
 		console = new Console(font);
 		layer.addChildAt(console, 4);
 		#end
 
-		font.resizeTo(20);
-
 		globalEventBus = new EventBus(console);
 		globalEventBus.subscribe(ChangeSceneEvent, onChangeScene);
 
@@ -89,12 +80,15 @@ class Game extends hxd.App {
 		var yarnFileNames = [hxd.Res.text.intro.entry.name, hxd.Res.text.rooms.entry.name,];
 		dialogueManager.load(yarnText, yarnFileNames);
 
-		globalEventBus.subscribe(LineShown, dialogue);
-		globalEventBus.subscribe(OptionsShown, renderOptions);
-		globalEventBus.subscribe(OptionSelected, onOptionSelected);
-		globalEventBus.subscribe(DialogueComplete, onDialogueComplete);
 		dialogueManager.addCommandHandler(new OpenRoomCommand(globalEventBus));
 
+		var dialogueParent = new h2d.Flow();
+		dialogueParent.dom = domkit.Properties.create("flow", dialogueParent);
+		dialogueParent.dom.addClass('dialogue-container');
+		layer.addChildAt(dialogueParent, 2);
+		style.addObject(dialogueParent);
+		dialogueBoxController = new DialogueBoxController(globalEventBus, dialogueParent, style);
+
 		#if debug
 		setGameScene(new WorldMapScene(root));
 		#else
@@ -116,70 +110,6 @@ class Game extends hxd.App {
 		style.sync();
 	}
 
-	function dialogue(event:LineShown) {
-		if (curDialogue != null) {
-			curDialogue.remove();
-		}
-		var d = new DialogueBox(Const.W, 100, event.line());
-		d.y = 0;
-		d.onClick = function() {
-			globalEventBus.publishEvent(new NextLine());
-		};
-
-		curDialogue = d;
-	}
-
-	function renderOptions(event:OptionsShown) {
-		for (option in event.options) {
-			addOption(option.text, option.enabled, function() {
-				if (option.enabled) {
-					globalEventBus.publishEvent(new OptionSelected(option.index));
-				}
-			});
-		}
-	}
-
-	function newPanel(w, h) {
-		var g = new h2d.ScaleGrid(Res.ui.toTile(), 5, 5);
-		g.width = w;
-		g.height = h;
-		g.colorKey = 0xFF00FF;
-		root.scene.getH2dObject().getScene().add(g, 1);
-		return g;
-	}
-
-	function addOption(o:String, enabled:Bool, callback:() -> Void) {
-		var spr = newPanel(Const.W, 60);
-		curDialogue.addChild(spr);
-		var tf = new h2d.Text(font, spr);
-		if (!enabled) {
-			tf.color = new Vector4(0.5, 0.5, 0.5, 1);
-		}
-		tf.text = o;
-		tf.x = 10;
-		tf.y = 10;
-		spr.x = 0;
-		spr.y = 100 + options.length * 60;
-		var int = new h2d.Interactive(spr.width, spr.height, spr);
-		int.onClick = function(_) {
-			callback();
-		}
-		int.cursor = Button;
-		options.push(spr);
-		return spr;
-	}
-
-	function onOptionSelected(event:OptionSelected) {
-		for (o in options) {
-			o.remove();
-		}
-		options.splice(0, options.length);
-	}
-
-	function onDialogueComplete(event:DialogueComplete) {
-		curDialogue.remove();
-	}
-
 	public function onChangeScene(event:ChangeSceneEvent) {
 		setGameScene(event.newScene);
 	}
@@ -196,4 +126,8 @@ class Game extends hxd.App {
 		root.scene = gs;
 		style.addObject(gs.getH2dObject());
 	}
+
+	static function main() {
+		new Game();
+	}
 }

src/Main.hx

@@ -1,5 +0,0 @@
-class Main {
-	static function main() {
-		new Game();
-	}
-}

src/components/DialogueBoxComponent.hx

@@ -0,0 +1,148 @@
+package components;
+
+import dialogue.event.NextLine;
+import dialogue.event.OptionSelected;
+import dialogue.event.OptionsShown.OptionChoice;
+
+@:uiComp("dialogue-option")
+class DialogueOptionComponent extends h2d.Flow implements h2d.domkit.Object {
+	static var SRC = <dialogue-option>
+		<text text={label} public font={Game.current.font} id="labelTxt" />
+	</dialogue-option>;
+
+	public var label(get, set):String;
+
+	function get_label()
+		return labelTxt.text;
+
+	function set_label(s) {
+		labelTxt.text = s;
+		return s;
+	}
+
+	var choice:OptionChoice;
+
+	public function new(choice:OptionChoice, ?parent) {
+		super(parent);
+		initComponent();
+		this.choice = choice;
+		this.label = choice.text;
+		if (!choice.enabled) {
+			this.dom.addClass('disabled');
+		}
+		enableInteractive = true;
+		interactive.onClick = function(_) onClick();
+		interactive.onOver = function(_) {
+			dom.hover = true;
+		};
+		interactive.onPush = function(_) {
+			dom.active = true;
+		};
+		interactive.onRelease = function(_) {
+			dom.active = false;
+		};
+		interactive.onOut = function(_) {
+			dom.hover = false;
+		};
+	}
+
+	public dynamic function onClick() {
+		if (this.choice.enabled) 
+			Game.current.globalEventBus.publishEvent(new OptionSelected(this.choice.index));
+	}
+}
+
+@:uiComp("dialogue-options")
+class DialogueOptionsComponent extends h2d.Flow implements h2d.domkit.Object {
+	static var SRC = <dialogue-options>
+	${options}
+	</dialogue-options>;
+
+	public var options = new Array<DialogueOptionComponent>();
+
+	public function new(options:Array<OptionChoice>, ?parent) {
+		super(parent);
+		initComponent();
+		for (option in options) {
+			this.options.push(
+					new DialogueOptionComponent(option, this)
+			);
+		}
+	}
+}
+
+@:uiComp("dialogue-box")
+class DialogueBoxComponent extends h2d.Flow implements h2d.domkit.Object {
+	var timer:haxe.Timer;
+	var chan:hxd.snd.Channel;
+	var textPos:Int = 0;
+	var game:Game;
+
+	static var SRC = <dialogue-box>
+		<text text={displayedText} public font={Game.current.font} id="dialogueText" />
+	</dialogue-box>
+
+	public var displayedText(get, set):String;
+
+	function get_displayedText()
+		return dialogueText.text;
+
+	function set_displayedText(s) {
+		dialogueText.text = s;
+		return s;
+	}
+
+	public var internalText(default, set):String;
+
+	function set_internalText(t) {
+		internalText = t;
+		timer = new haxe.Timer(30);
+		timer.run = updateText;
+		displayedText = "";
+		textPos = 0;
+		return t;
+	}
+
+	public function new(text:String, ?parent) {
+		this.game = Game.current;
+		super(parent);
+		initComponent();
+		internalText = text;
+
+		enableInteractive = true;
+		interactive.onClick = function(_) onClick();
+	}
+
+	function updateText() {
+		if (textPos == internalText.length) {
+			timer.stop();
+			// onReady();
+			// if (chan != null)
+			// 	chan.stop();
+			return;
+		}
+		// if (chan != null) {
+		// 	switch (text.charCodeAt(textPos)) {
+		// 		case " ".code, "\n".code:
+		// 			chan.volume = 0;
+		// 		default:
+		// 			if (chan.volume == 0)
+		// 				chan.volume = 1
+		// 			else
+		// 				chan.volume *= 0.9;
+		// 	}
+		// }
+		textPos++;
+		displayedText = internalText.substr(0, textPos);
+	}
+
+	public dynamic function onClick() {
+		if (textPos == internalText.length)
+			Game.current.globalEventBus.publishEvent(new NextLine());
+		else if (textPos < internalText.length) {
+			textPos = internalText.length;
+			displayedText = internalText;
+			updateText();
+		};
+	}
+}

src/dialogue/DialogueBox.hx

@@ -1,107 +0,0 @@
-package dialogue;
-
-class DialogueBox extends h2d.Object {
-	var game:Game;
-	var bg:h2d.ScaleGrid;
-	var tf:h2d.Text;
-	var timer:haxe.Timer;
-	var int:h2d.Interactive;
-
-	public var width(default, set):Int;
-	public var height(default, set):Int;
-	public var text(default, set):String;
-
-	var textPos:Int = 0;
-	var chan:hxd.snd.Channel;
-
-	public function new(width:Int, height:Int, text:String) {
-		super();
-		if (text == null)
-			text = "NULL";
-		game = Game.current;
-		cast(game.root, h2d.Object).getScene().add(this, 1);
-		bg = new h2d.ScaleGrid(Res.ui.toTile(), 5, 5, this);
-		bg.colorKey = 0xFF00FF;
-		tf = new h2d.Text(game.font, this);
-		tf.y = 5;
-		tf.x = 7;
-		tf.dropShadow = {
-			dx: 0,
-			dy: 1,
-			color: 0,
-			alpha: 0.3
-		};
-		int = new h2d.Interactive(0, 0, this);
-		int.onClick = function(_) click();
-		this.width = width;
-		this.height = height;
-		this.text = text;
-	}
-
-	override function onRemove() {
-		super.onRemove();
-		if (chan != null)
-			chan.stop();
-		timer.stop();
-	}
-
-	function updateText() {
-		if (textPos == text.length) {
-			timer.stop();
-			onReady();
-			if (chan != null)
-				chan.stop();
-			return;
-		}
-		if (chan != null) {
-			switch (text.charCodeAt(textPos)) {
-				case " ".code, "\n".code:
-					chan.volume = 0;
-				default:
-					if (chan.volume == 0)
-						chan.volume = 1
-					else
-						chan.volume *= 0.9;
-			}
-		}
-		textPos++;
-		tf.text = text.substr(0, textPos);
-	}
-
-	public function click() {
-		if (textPos == text.length)
-			onClick()
-		else if (textPos < text.length) {
-			textPos = text.length;
-			tf.text = text;
-			updateText();
-		};
-	}
-
-	public dynamic function onClick() {}
-
-	public dynamic function onReady() {}
-
-	function set_text(t) {
-		text = t;
-		timer = new haxe.Timer(30);
-		timer.run = updateText;
-		tf.text = "";
-		textPos = 0;
-		return t;
-	}
-
-	function set_width(w:Int) {
-		bg.width = w;
-		int.width = w;
-		tf.maxWidth = w - 14;
-		tf.text = text;
-		return width = w;
-	}
-
-	function set_height(h:Int) {
-		bg.height = h;
-		int.height = h;
-		return height = h;
-	}
-}

src/event/EventBus.hx

@@ -40,6 +40,7 @@ class EventBus {
 			#end
 			return;
 		}
+		trace('Publishing event $type');
 
 		for (func in listeners.get(type))
 			func(event);

src/listeners/DialogueBoxController.hx

@@ -0,0 +1,74 @@
+package listeners;
+
+import Game;
+
+import event.EventBus;
+
+import dialogue.DialogueManager;
+
+import dialogue.event.LineShown;
+import dialogue.event.OptionsShown;
+import dialogue.event.OptionSelected;
+import dialogue.event.DialogueComplete;
+import dialogue.event.NextLine;
+
+import components.DialogueBoxComponent;
+
+class DialogueBoxController {
+	public var dialogueBox:DialogueBoxComponent;
+	public var dialogueOptions:DialogueOptionsComponent;
+
+	var game:Game;
+	var bg:h2d.ScaleGrid;
+	var tf:h2d.Text;
+	var int:h2d.Interactive;
+	var eventBus:EventBus;
+	var parent:h2d.Object;
+	var style:h2d.domkit.Style;
+
+	public function new(eventBus:EventBus, parent:h2d.Object, style:h2d.domkit.Style) {
+		this.parent = parent;
+		this.style = style;
+
+		eventBus.subscribe(LineShown, onLineShown);
+		eventBus.subscribe(OptionsShown, onOptionsShown);
+		eventBus.subscribe(OptionSelected, onOptionSelected);
+		eventBus.subscribe(DialogueComplete, onDialogueComplete);
+	}
+
+	// override function onRemove() {
+	// 	super.onRemove();
+	// 	// if (chan != null)
+	// 	// 	chan.stop();
+	// 	// timer.stop();
+	// }
+
+	public dynamic function onClick() {}
+
+	public dynamic function onReady() {}
+
+	function onLineShown(event:LineShown) {
+		if (dialogueBox != null) {
+			dialogueBox.remove();
+		}
+		var d = new DialogueBoxComponent(event.line(), this.parent);
+		style.addObject(d);
+
+		dialogueBox = d;
+	}
+
+	function onOptionsShown(event:OptionsShown) {
+		dialogueOptions = new DialogueOptionsComponent(event.options, this.parent);
+	}
+	
+	function onOptionSelected(event:OptionSelected) {
+		dialogueOptions.remove();
+	}
+
+	function onDialogueComplete(event:DialogueComplete) {
+		dialogueBox.remove();
+		if (dialogueOptions != null)
+			dialogueOptions.remove();
+	}
+
+}