dice: Break out occ_label

We are going to want to call this for situations where we have an
OccultSpec but not an occultable image.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice: Reorganise occultation handling

I was trying to support ilks properly.  This code is in fact going to
be considerably simplified.

But it will still make sense to split the interpretation of the spec
from the assembly of the Arc<dyn InertPieceImage> for the occulted
view.  That will make it possible for the spec interpretation to
provide an occulted view in more situations.

In fact it appears that these plans are going to be postponed and not
done right now, but I'm keeping this refactoring.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice: Break out OccultSpec (breaking change)

Earlier I wrote: We want to add an ilk here.

But this is no longer going to be true, because we are getting rid of
ilk mixing for dice.  Nevertheless, this change is going in a roughly
good direction in spec API terms since it will allow us to add other
things to occult.

Notably, if we allow composition of a die image from a base and faces,
or if we allow specifying an occulted face image separately from the
face images.

And `occult_label` is just a bad pattern.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

demo table: Move die

We're going to add another die

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Reduce proposed stability promise re management protocol

This will allow us not to worry about errors, which are otherwise
awkward.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dbgc: Remove a call in clock.rs

This looks like leftovers.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dbgc: Replace many calls in hidden.rs with trace_dbg

This looks like leftovers.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

trace_dbg: Remove some use's.

This seems to generate spurious unused import warnings in some
situations.  Perhaps because it constantly reimports ?

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice js: Sort out the logging

Don't log every frame!

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice js: Actually do the animation

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice: Rename remprop parameter to wasm die_cooldown_path

Apparently I missed renaming this when I introduced the "rempprop"
name.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice js: Obtain radius from the input dataset

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice: Always provide the label text node

This will give us a way to tell which elements are part of the
cooldown clock and therefore need to be removed when it runs down.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice js: Obtain information from the input dataset etc.

We want this all nicely in our JS object.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice js: Break out die_request_animation

We're going to want to re-request animiation in the frame renderer,
obviously.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice js: Make s.anim_id be possibly null

We might not have requested an animation frame.  If so we shouldn't
cancel it.  Nor, if we have already cancelled it, or if it has
occurred.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice js: Initial plumbing of animiation

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

js: Introduce SpecialRendering handling

This is the generic plumbing.  We must implement at least a no-op for
Die right away, because otherwise we throw an exception during game
load.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

js: Notice errors during start() and display them

Otherwise if the game is buggy, you can get a fundamentally broken and
unresponsive UI, and no indication that this isn't your fault.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice: Change SpecialClientRendering representation

Making this an internally-tageed union is going to be nicer for the
JS, which then doesn't have to prat about with Object.keys(thing)[0]
etc.

And the special rendering is Die, not DieCooldown, since if dice get a
second kind of special rendering thta will have to be here too.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice: overlay extractor: fix autogeneration comment

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice: Support rolling

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

fake-rng: Provide range() method

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice: Break out check_permit_flip_roll

The "roll" ui operation is going to want this too.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice: Break out cooldown_start_value

The "roll" ui operation is going to want this too.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice: Prevent flipping during cooldown

This involves a new hook for the piece trait.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

flip: Use description after flip, not before

This makes the log messages about dice true.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice: Actually display the labels

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice: Add definite article to log entries

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice: Rename iface from face

This is a mangled version of the requested face which we should only
use for the image, since it might be 0 even if there are many labels.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

die: Adjust text size

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

die: Prepare for extracting text template too (2)

Bracket our cooldown output all together.

I did it the other way because I had forgotten that the output order
is controlled by the script, not the input.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

die: Prepare for extracting text template too

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

die: Hack about at SVG to make a nicer template for label

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice: Add labels to demo game

But they don't display yet...

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice: Add cross-referencing notes and comments

This may help us find things in future.  Used inkscape's "document
properties", "metadata" pane, and then manually moved the new metadata
tag nearer to the top of the svg file.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice: Fix cooldown radius

This looks good now.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Provide cooldown timer stroke width to Rust

We now have this in two places which is a bit poor, but we already
have some duplication between die.svg and the Rust code - eg, the path
generation which was just c&p.

I have left a comment.  And arranged to filter it out for distribution.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Provide select stroke width to Rust

We now have this in two places which is a bit poor, but I don't fancy
plumbing it into the ts machinery.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice: Use bespoke outline path for surround (selection)

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

OutlineTrait: Document outline_path

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice: Fix a radius/diameter confusion

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

die svg: Shrinking by 10: Change window

Just asked inkscape to recenter the view.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

die svg: Shrinking by 10: Reduce die box size

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

die svg: Shrinking by 10: Reduce stroke widths

We don't have anything to rescale the elements we extract for use as
the cooldown timer, so we need to get the linewidths as we finally
want them.

In this commit we mostly do the stroke widths and the circle size.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice: Adjust sizes, wip

Adjust the strokes of the circles using inkscape.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice: Add a test die to the demo game

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice: Cooldown template: Add header about autogeneration

This may help avoid editing it manuallly...

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice: Make moveable

Don't bake into the piece implementation that these aren't moveable.
If this is desired, it can be part of the game spec.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice: Sort out defaults

 * Abolish the separate DEFAULT_COOLDOWN constant.  This seems otiose.
 * Clarify what the radius scale is supposed to mean
 * Provide a default radius scale

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

test.table.toml: Add a currently-working rune

Many of the others are obsolete but I don't want to take the time now
to clean them up.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice: Start a cooldown the first time a die is added to the game

This is useful for testing.  It also means you can't just add a die
showing what you want using the game management interface...

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Provide InternalError::FutureInstantOutOfRange

The dice itself wants to sometimes do this conversion (notably, on
initial piece load), and if it's out of range that's implausible.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice: Always use a circular Outline and adjust radiii

The circular outline will go nicely with the cooldown timer.

The radius etc. is still not quite right.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice: Set cooldown_expiry to None when we can

This will avoid pointless calls to Instant::now, which would otherwise
be needed to discover that the cooldown time has expired (perhaps long
ago).

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice: Tidy remprop handling

This macro was overkill.  It existed mostly to avoid uselessly
comparing the remaining duration with 0 and storing None when it was
there already.  That's a daft microoptimisation.

Doing it this way instead lets us only do the remprop calculation when
it's wanted, and it will let us provide a way to set cooldown_expires
to None if and when we have &mut State and can see it's timed out.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice: Initial implementation of piece

Does not work properly yet.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Provide FutureTimeInstant, for use by dice

This loads and saves as a Duration.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice: cooldown_path etc.: rename remaining proportion to "remprop"

This distinguishes "remaining" (a duration) from "remprop" (a proportion).

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice: Cooldown template: Use total_ms for duration variable

Make the units clearer.  We will provide a value for this from a
Serialize context struct in the Rust code, which is still to come.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice: Generate cooldown template using extractor

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Move die helper materials to dice/ subdirectory

This declutters the toplevel a bit.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Special client rendering: Do explain why not to provide xdata here

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Special client rendering: Fix quoting of JSON payload

The needs to be in " " (and the JS will evaluate it).

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Provide plumbing for special client rendering instructions

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Make PieceBaseTrait dyn_upcast-able

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

pcrender: prep for more attributes in pelem

We are going to add a .special.

NFC.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

die: wip svg, add more tests

Again, verified by hand-hacking and using Refresh in inkscape, that
they look right.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

die: wip svg, cooldown timer better assert

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

die: wip svg, fix cooldown timer path and add test cases

I've verified that the 1. test case shows as expected.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

die: wip svg, make example less than half

This let me c&p the string into the new test case.  Although, the test
will need to be different because of rounding errors.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice: Provide die_cooldown_path

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

die: wip svg, fix colour of white

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

die: wip svg extractor

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

die: wip svg, change timewhite path (back) to a circle

We want to be able to auto-adjust it by fiddling with r=.  That's
going to be awkward with the path element but easy with circle.

I have checked that one can (eg) change the width of the line in
inkscaqpe, and it remains a circle.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

die: wip svg, remove unneeded parts

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

die: wip svg, remove zigzag

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

die: wip svg

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

die: wip svg

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

die: wip svg

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

InertPieceTrait: Add comment clarifying FaceId etc.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Gamestate: Fix a more style nit

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

SpecError: Add a comma (for consistent coding style)

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Provide load_inert for SimplePieceSpec

This will let us use these as backgrounds for dice.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Move fn load out of SimplePieceSpec into macro

The only call site was in impl_PieceSpec_for_SimplePieceSpec.
Hoisting it there makes little codegen difference but seems more
natural.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Combine impls into impl_PieceSpec_for_SimplePieceSpec

We want to add load_inert to this.  It's getting too repetitive.

We can't use generic impl on <P: SimplePieceSpec> because we need to
specxify the serde typetag.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Pass face to InertPieceTrait::describe_html

This ought to have been part of
  Extend InertPieceTrait to support multiple faces

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Make OccultIlkName a type alias, not a newtype

There's not really an abstraction here defending anything, and it
lacks constructors etc.  Just abolish it.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Plumb xdata through to InertPieceTrait svg

Occulted dice will need this to display the cooldown timer.

This makes dice's InertPieceTrait impl not very inert, but it is OK
because nothing can set off an inert die's cooldown timer.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Move itemname into PieceBaseTrait

We want this becasue dice want to use inert pieces as images, and use
their images' item names in their own item names.

This involves providing an itemname for ItemInertForOcculted.
Happily the SvgBaseName will do nicely.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Introduce PieceBaseTrait and put nfaces() in it

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

humantime: Switch to humantime_serde

We are going to want this for dice cooldown spec.
Using the humantime re-export avoids version problems.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

humantime: Promote dependency to otter main crate

We are going to want this for dice cooldown spec.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

spec errors: Expand WrongNumberOfFaces error

We're going to want to reuse this in a new context.  And, this already
produces better error messages for the case where it's used now.

It seems better to provide this rather generic error rather than
expect each funky piece kind to invent its own kind in SpecError.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

docs: Document scope of semver guarantees

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Make GenericSimpleShape::count_faces return RawFaceId

RawFaceId is smaller than usize.  It is better not to ask callers to
do the unwrap.  Motivated by wanting add a new call site which wants a
RawFaceId.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Extend load_inert to offer an occultable piece

Now inert pieces can be occultable too.  Currently this is meaningless
since we only call load_inert when resolving the occultation
for *another* piece, where we discard the occultation information.

But that is going to change.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Genericise SpecLoaded

We are going to want to make inert pieces occultable too.  Currently
this is meaningless since we only call load_inert when resolving the
occultation for *another* piece.  But that is going to change.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Fix message for IE::JSONEncode

The name is JSONEncode, so this must be serialisation, and indeed that
is true of both the construction sites.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Implement InertPieceTrait for SimpleShape

Right now, this allows simple circles and rectangles to be used as
backs (for occultation).  It will also allow them to be used for dice,
when we have those.

It is not convenient to provide a test for this because occultation
methods must be provided in libraries, and we don't have a test
library.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>