Get character stats (kind of) displaying in-scenario

This is pretty awful, but will let me wire up items more easily without
needing to do the big refactor into independent menu handlers

cmd/view-map/main.go

@@ -10,7 +10,9 @@ import (
 
 	"code.ur.gs/lupine/ordoor/internal/assetstore"
 	"code.ur.gs/lupine/ordoor/internal/config"
+	"code.ur.gs/lupine/ordoor/internal/flow"
 	"code.ur.gs/lupine/ordoor/internal/scenario"
+	"code.ur.gs/lupine/ordoor/internal/ship"
 	"code.ur.gs/lupine/ordoor/internal/ui"
 )
 
@@ -25,6 +27,7 @@ var (
 )
 
 type env struct {
+	flow     *flow.Flow
 	scenario *scenario.Scenario
 }
 
@@ -51,27 +54,39 @@ func main() {
 		log.Fatalf("Failed to load scenario %v: %v", *gameMap, err)
 	}
 
-	env := &env{
+	var realEnv *env
-		scenario: scenario,
+	if cfg.DefaultEngineName == "ordoor" {
+		ship := &ship.Ship{}
+
+		flow, err := flow.New(assets, cfg, ship)
+		if err != nil {
+			log.Fatalf("Failed to setup flow: %v", err)
+		}
+		flow.SetScenario(scenario)
+		realEnv = &env{flow: flow, scenario: scenario}
+	} else {
+		realEnv = &env{scenario: scenario}
 	}
 
-	win, err := ui.NewWindow(env, "View Map "+*gameMap, *winX, *winY)
+	win, err := ui.NewWindow(realEnv, "View Map "+*gameMap, *winX, *winY)
 	if err != nil {
 		log.Fatal("Couldn't create window: %v", err)
 	}
 
-	step := 32
-	win.WhileKeyDown(ebiten.KeyLeft, env.changeOrigin(-step, +0))
-	win.WhileKeyDown(ebiten.KeyRight, env.changeOrigin(+step, +0))
-	win.WhileKeyDown(ebiten.KeyUp, env.changeOrigin(+0, -step))
-	win.WhileKeyDown(ebiten.KeyDown, env.changeOrigin(+0, +step))
-
 	for i := 0; i <= 6; i++ {
-		win.OnKeyUp(ebiten.Key1+ebiten.Key(i), env.setZIdx(i))
+		win.OnKeyUp(ebiten.Key1+ebiten.Key(i), realEnv.setZIdx(i))
 	}
 
-	win.OnMouseClick(env.showCellData)
+	win.OnMouseClick(realEnv.showCellData)
-	win.OnMouseWheel(env.changeZoom)
+	win.OnMouseWheel(realEnv.changeZoom)
+
+	if realEnv.flow == nil {
+		step := 32
+		win.WhileKeyDown(ebiten.KeyLeft, realEnv.changeOrigin(-step, +0))
+		win.WhileKeyDown(ebiten.KeyRight, realEnv.changeOrigin(+step, +0))
+		win.WhileKeyDown(ebiten.KeyUp, realEnv.changeOrigin(+0, -step))
+		win.WhileKeyDown(ebiten.KeyDown, realEnv.changeOrigin(+0, +step))
+	}
 
 	if err := win.Run(); err != nil {
 		log.Fatal(err)
@@ -79,12 +94,20 @@ func main() {
 }
 
 func (e *env) Update(screenX, screenY int) error {
+	if e.flow != nil {
+		return e.flow.Update(screenX, screenY)
+	} else {
 		return e.scenario.Update(screenX, screenY)
 	}
+}
 
 func (e *env) Draw(screen *ebiten.Image) error {
+	if e.flow != nil {
+		return e.flow.Draw(screen)
+	} else {
 		return e.scenario.Draw(screen)
 	}
+}
 
 func (e *env) changeOrigin(byX, byY int) func() {
 	return func() {

internal/assetstore/map.go

@@ -102,9 +102,7 @@ func (m *Map) SpritesForCell(x, y, z int) ([]*Sprite, error) {
 
 		sprites = append(sprites, sprite)
 	}
-
+	if chr := m.CharacterAt(x, y, z); chr != nil {
-	for _, chr := range m.raw.Characters {
-		if chr.XPos == x && chr.YPos == y && z == 1 { // FIXME: sort out ZPos
 		// Look up the correct animation, get the frame, boom
 		anim, err := m.assets.CharacterAnimation(chr.Type, data.AnimActionNone)
 		if err != nil {
@@ -113,7 +111,18 @@ func (m *Map) SpritesForCell(x, y, z int) ([]*Sprite, error) {
 
 		sprites = append(sprites, anim.Frames[0])
 	}
-	}
 
 	return sprites, nil
 }
+
+func (m *Map) CharacterAt(x, y, z int) *maps.Character {
+	// FIXME: don't iterate
+	for i, _ := range m.raw.Characters {
+		chr := &m.raw.Characters[i]
+		if chr.XPos == x && chr.YPos == y && z == 0 { // FIXME: sort out ZPos
+			return chr
+		}
+	}
+
+	return nil
+}

internal/flow/flow.go

@@ -7,6 +7,7 @@ import (
 	"strings"
 
 	"github.com/hajimehoshi/ebiten"
+	"github.com/hajimehoshi/ebiten/inpututil"
 
 	"code.ur.gs/lupine/ordoor/internal/assetstore"
 	"code.ur.gs/lupine/ordoor/internal/config"
@@ -92,6 +93,11 @@ func New(assets *assetstore.AssetStore, config *config.Config, ship *ship.Ship)
 	return out, out.exit
 }
 
+func (f *Flow) SetScenario(scenario *scenario.Scenario) {
+	f.current = f.drivers[mainGame]
+	f.scenario = scenario
+}
+
 func (f *Flow) Update(screenX, screenY int) error {
 	if f.exit != nil {
 		return f.exit
@@ -113,6 +119,14 @@ func (f *Flow) Update(screenX, screenY int) error {
 		if ebiten.IsKeyPressed(ebiten.KeyDown) {
 			f.scenario.Viewpoint.Y += step
 		}
+
+		if inpututil.IsMouseButtonJustReleased(ebiten.MouseButtonLeft) {
+			f.scenario.SelectHighlightedCharacter()
+
+			// Now we need to update the info screens with data about the
+			// selected character. FIXME: oh, for data binding
+			f.selectedMainGameCharacter(f.scenario.SelectedCharacter())
+		}
 	}
 
 	if f.scenario != nil {

internal/flow/main_game.go

@@ -1,5 +1,9 @@
 package flow
 
+import (
+	"code.ur.gs/lupine/ordoor/internal/maps"
+)
+
 // TODO: There are Chaos and Ultramarine versions of MainGame. Do we really want
 // to duplicate everything for both?
 
@@ -16,7 +20,7 @@ func (f *Flow) linkMainGame() {
 	})
 
 	// 8:  Character stats
-	f.onClick(mainGame, "8.21", func() { // Stat more buttont
+	f.onClick(mainGame, "8.21", func() { // Stat more buttons
 		f.setActiveNow(mainGame, "7", true)
 		f.setActiveNow(mainGame, "8", false)
 	})
@@ -154,3 +158,48 @@ func (f *Flow) linkMainGameViewMenu() {
 	}))
 
 }
+
+func (f *Flow) maybeSetErr(next func() error) {
+	if f.exit != nil {
+		return
+	}
+
+	f.exit = next()
+}
+
+func (f *Flow) selectedMainGameCharacter(chr *maps.Character) {
+	if chr == nil {
+		chr = &maps.Character{}
+	}
+
+	d := f.drivers[mainGame]
+
+	// 7.1 Portrait
+	f.maybeSetErr(func() error { return d.SetValue("7.2", chr.Name) }) // Name
+	// 7.3 doesn't exit
+	// 7.4 more button (ignore)
+	// 7.5 AP icon
+	// f.maybeSetErr(func() error { return d.SetValueInt("7.6", chr.ActionPoints)}) // AP meter
+	f.maybeSetErr(func() error { return d.SetValueInt("7.7", chr.ActionPoints) }) // AP value
+	// 7.8 armor icon
+	// 7.9 armor meter
+	f.maybeSetErr(func() error { return d.SetValueInt("7.10", chr.Armor) }) // armor value
+	// 7.11 health icon
+	// 7.12 health meter
+	f.maybeSetErr(func() error { return d.SetValueInt("7.13", chr.Health) }) // health value
+	// 7.14 action points status bar
+	// 7.15 armor status bar
+	// 7.16 health status bar
+
+	// 8.1 to 8.10 are hot spots
+	f.maybeSetErr(func() error { return d.SetValueInt("8.11", chr.ActionPoints) })   // AP
+	f.maybeSetErr(func() error { return d.SetValueInt("8.12", chr.Health) })         // Health
+	f.maybeSetErr(func() error { return d.SetValueInt("8.13", chr.Armor) })          // Armor
+	f.maybeSetErr(func() error { return d.SetValueInt("8.14", chr.BallisticSkill) }) // Ballistic Skill
+	f.maybeSetErr(func() error { return d.SetValueInt("8.15", chr.WeaponSkill) })    // Weapon Skill
+	f.maybeSetErr(func() error { return d.SetValueInt("8.16", chr.Strength) })       // Strength
+	f.maybeSetErr(func() error { return d.SetValueInt("8.17", chr.Toughness) })      // Toughness
+	// 8.18 Initiative
+	// 8.19 Attacks
+	f.maybeSetErr(func() error { return d.SetValueInt("8.20", chr.Leadership) }) // Leadership
+}

internal/maps/maps.go

@@ -319,7 +319,8 @@ func loadMapFile(filename string) (*GameMap, error) {
 	nullTerminate(&out.Title)
 	nullTerminate(&out.Briefing)
 
-	for i, chr := range out.Characters {
+	for i, _ := range out.Characters {
+		chr := &out.Characters[i]
 		nullTerminate(&chr.Name)
 		fmt.Printf("Character %v: %s\n", i, chr.String())
 	}

internal/menus/menus.go

@@ -33,7 +33,7 @@ const (
 	SubTypeLineBriefing    SubMenuType = 41
 	SubTypeThumb           SubMenuType = 45 // A "thumb" appears to be a vertical slider
 	SubTypeInvokeButton    SubMenuType = 50
-	SubTypeDoorHotspot3    SubMenuType = 60 // Maybe? Appears in Arrange.mnu
+	SubTypeClickText       SubMenuType = 60
 	SubTypeOverlay         SubMenuType = 61
 	SubTypeHypertext       SubMenuType = 70
 	SubTypeCheckbox        SubMenuType = 91

internal/scenario/draw.go

@@ -36,7 +36,7 @@ func (s *Scenario) Update(screenX, screenY int) error {
 	}
 
 	// FIXME: adjust for Z level
-	s.selectedCell = screenPos.ToISO()
+	s.highlightedCell = screenPos.ToISO()
 
 	return nil
 }
@@ -112,7 +112,7 @@ func (s *Scenario) Draw(screen *ebiten.Image) error {
 	}
 
 	op := ebiten.DrawImageOptions{}
-	geo := s.geoForCoords(int(s.selectedCell.X), int(s.selectedCell.Y), 0)
+	geo := s.geoForCoords(int(s.highlightedCell.X), int(s.highlightedCell.Y), 0)
 	op.GeoM = geo
 	op.GeoM.Translate(-209, -332)
 	op.GeoM.Translate(float64(spr.Rect.Min.X), float64(spr.Rect.Min.Y))
@@ -125,7 +125,7 @@ func (s *Scenario) Draw(screen *ebiten.Image) error {
 	x1, y1 := geo.Apply(0, 0)
 	ebitenutil.DebugPrintAt(
 		screen,
-		fmt.Sprintf("(%d,%d)", int(s.selectedCell.X), int(s.selectedCell.Y)),
+		fmt.Sprintf("(%d,%d)", int(s.highlightedCell.X), int(s.highlightedCell.Y)),
 		int(x1),
 		int(y1),
 	)

internal/scenario/manage.go

@@ -1,6 +1,8 @@
 package scenario
 
 import (
+	"log"
+
 	"code.ur.gs/lupine/ordoor/internal/maps"
 )
 
@@ -10,8 +12,23 @@ type CellPoint struct {
 }
 
 func (s *Scenario) CellAtCursor() (*maps.Cell, CellPoint) {
-	cell := s.area.Cell(int(s.selectedCell.X), int(s.selectedCell.Y), 0)
+	cell := s.area.Cell(int(s.highlightedCell.X), int(s.highlightedCell.Y), 0)
-	return cell, CellPoint{IsoPt: s.selectedCell, Z: 0}
+	return cell, CellPoint{IsoPt: s.highlightedCell, Z: 0}
+}
+
+func (s *Scenario) HighlightedCharacter() *maps.Character {
+	// FIXME: characters are always at zIdx 0 right now
+	return s.area.CharacterAt(int(s.highlightedCell.X), int(s.highlightedCell.Y), 0)
+}
+
+func (s *Scenario) SelectedCharacter() *maps.Character {
+	return s.selectedCharacter
+}
+
+func (s *Scenario) SelectHighlightedCharacter() {
+	chr := s.HighlightedCharacter()
+	log.Printf("Selected character %s", chr)
+	s.selectedCharacter = chr
 }
 
 func (s *Scenario) ChangeZIdx(by int) {

internal/scenario/scenario.go

@@ -2,10 +2,10 @@
 package scenario
 
 import (
-	"fmt"
 	"image"
 
 	"code.ur.gs/lupine/ordoor/internal/assetstore"
+	"code.ur.gs/lupine/ordoor/internal/maps"
 )
 
 type Scenario struct {
@@ -14,7 +14,9 @@ type Scenario struct {
 
 	tick int
 	turn int
-	selectedCell IsoPt
+
+	highlightedCell   IsoPt
+	selectedCharacter *maps.Character
 
 	// All these must be modified by user actions somehow.
 	// TODO: extract into the idea of a viewport passed to Update / Draw somehow?
@@ -36,9 +38,9 @@ func NewScenario(assets *assetstore.AssetStore, name string) (*Scenario, error)
 	}
 
 	// Eager load sprites. TODO: do we really want to do this?
-	if err := area.LoadSprites(); err != nil {
+	//if err := area.LoadSprites(); err != nil {
-		return nil, fmt.Errorf("Eager-loading sprites failed: %v", err)
+	//	return nil, fmt.Errorf("Eager-loading sprites failed: %v", err)
-	}
+	//}
 
 	out := &Scenario{
 		area:      area,

internal/ui/group.go

@@ -78,8 +78,10 @@ func (d *Driver) buildRecord(r *menus.Record) (*Widget, error) {
 	switch r.Type {
 	case menus.SubTypeSimpleButton, menus.SubTypeInvokeButton:
 		_, widget, err = d.buildButton(r.Props())
-	case menus.SubTypeDoorHotspot1, menus.SubTypeDoorHotspot2, menus.SubTypeDoorHotspot3:
+	case menus.SubTypeDoorHotspot1, menus.SubTypeDoorHotspot2:
 		_, widget, err = d.buildDoorHotspot(r.Props())
+	case menus.SubTypeClickText:
+		_, widget, err = d.buildClickText(r.Props())
 	case menus.SubTypeOverlay:
 		_, widget, err = d.buildOverlay(r.Props())
 	case menus.SubTypeHypertext:

internal/ui/list_box.go

@@ -157,9 +157,9 @@ func (l *listBox) refresh() {
 		// FIXME: noninteractive isn't set up for dynamic text yet. Need to
 		// generate textImg on demand instead of once at start.
 		if ni.label != nil {
-			ni.label.text = ""
+			ni.label.str = ""
 			if len(l.strings) > l.offset+i {
-				ni.label.text = l.strings[l.offset+i]
+				ni.label.str = l.strings[l.offset+i]
 			}
 		}
 	}

internal/ui/noninteractive.go

@@ -32,12 +32,13 @@ type noninteractive struct {
 	hoverImpl
 }
 
-// Paint some text to screen
+// Paint some text to screen, possibly settable
 type label struct {
+	locator string
 	align   AlignMode
 	rect    image.Rectangle
-	text  string
 	font    *assetstore.Font
+	valueImpl
 }
 
 // This particular animation has entry and exit sequences, which are invoked
@@ -109,6 +110,33 @@ func (d *Driver) buildHypertext(p *menus.Properties) (*noninteractive, *Widget,
 	return ni, widget, nil
 }
 
+func (d *Driver) buildClickText(p *menus.Properties) (*noninteractive, *Widget, error) {
+	ni, err := d.buildNoninteractive(p)
+	if err != nil {
+		return nil, nil, err
+	}
+
+	fnt := d.menu.Font(p.FontType/10 - 1)
+
+	// FIXME: is this always right? Seems to make sense for Main.mnu
+	ni.label = &label{
+		locator: ni.locator,
+		font:    fnt,
+		rect:    ni.rect, // We will be centered by default
+		// Starts with no text. The text specified in the menu is hovertext
+	}
+
+	widget := &Widget{
+		Locator:       ni.locator,
+		Active:        p.Active,
+		ownClickables: []clickable{ni},
+		ownPaintables: []paintable{ni},
+		ownValueables: []valueable{ni.label},
+	}
+
+	return ni, widget, nil
+}
+
 // An overlay is a static image + some text that needs to be rendered
 func (d *Driver) buildOverlay(p *menus.Properties) (*noninteractive, *Widget, error) {
 	ni, err := d.buildNoninteractive(p)
@@ -129,7 +157,7 @@ func (d *Driver) buildOverlay(p *menus.Properties) (*noninteractive, *Widget, er
 		ni.label = &label{
 			font:      fnt,
 			rect:      ni.rect, // We will be centered by default
-			text: p.Text,
+			valueImpl: valueImpl{str: p.Text},
 		}
 	} else {
 		log.Printf("Overlay without text detected in %v", p.Locator)
@@ -253,6 +281,10 @@ func (a *animationHover) setHoverState(value bool) {
 	a.hoverImpl.setHoverState(value)
 }
 
+func (l *label) id() string {
+	return l.locator
+}
+
 // Top-left of where to start drawing the text. We want it to appear to be in
 // the centre of the rect.
 //
@@ -260,7 +292,7 @@ func (a *animationHover) setHoverState(value bool) {
 func (l *label) pos() image.Point {
 	pos := l.rect.Min
 
-	textRect := l.font.CalculateBounds(l.text)
+	textRect := l.font.CalculateBounds(l.str)
 
 	// Centre the text horizontally
 	if l.align == AlignModeCentre {
@@ -287,7 +319,7 @@ func (l *label) regions(tick int) []region {
 
 	pt := l.pos()
 
-	for _, r := range l.text {
+	for _, r := range l.str {
 		glyph, err := l.font.Glyph(r)
 		if err != nil {
 			log.Printf("FIXME: ignoring misssing glyph %v", r)