Joust icon indicating copy to clipboard operation
Joust copied to clipboard

Fully implement pluralization (language rule 4)

Open jleclanche opened this issue 6 years ago • 6 comments

Followup on #220. The text no longer looks weird, but pluralization is incorrect.

Until this is fixed we will be incorrectly displaying the text of some cards in some languages, such as Swipe in French.

Note: Unlikely to happen in Joust. I suspect we'll do it in Sunwell and by the time we do it, we'll switch to that.

jleclanche avatar Mar 24 '18 10:03 jleclanche

@jleclanche Can you clarify what the work to be done here is?

joolean avatar Jul 05 '18 18:07 joolean

This ticket is to implement card pluralization rules (|4 ...) which right now, althought they're parsed in Joust, they don't actually pluralize correctly (we don't bother parsing the number, we always assume it's plural IIRC).

The most likely fix is to switch to Sunwell for card rendering, which does have it implemented.

jleclanche avatar Jul 05 '18 18:07 jleclanche

My understanding was the Sunwell was too slow to use in Joust. Has that changed?

joolean avatar Jul 05 '18 21:07 joolean

It has not. There's a lot of performance work to do before we can use one inside the other.

jleclanche avatar Jul 05 '18 21:07 jleclanche

In that case, I think we should fix this in Joust, running the risk that the work is rendered unnecessary if we for some reason prioritize Sunwell performance improvements.

joolean avatar Jul 05 '18 21:07 joolean

I'm fine with that

jleclanche avatar Jul 05 '18 21:07 jleclanche