thexerteproject / xerteonlinetoolkits

Xerte Online Toolkits
www.xerte.org.uk
Apache License 2.0
62 stars 61 forks source link

Crossword page #1015

Closed FayCross closed 2 years ago

FayCross commented 3 years ago

A couple of bugs with the new page:

The following bugs are copied from Trello & are from Takeshi's testing:

Wishlist:

ingedonke commented 3 years ago

@FayCross Thx for testing. Can you share your example with me in https://xot310beta.dlearning.nl/ so I can try to reproduce this? The first item I also see in my examples but the second one not.

FayCross commented 3 years ago

@ingedonke I've just shared it with you. The first page is never created successfully - normally it just says it's because the word 'bean' can't be included but if you refresh it a few times you sometimes get a longer list of words that won't fit. The second page where I've removed the word bean seems to always work. On page 1 I think it would be better to just load the words it can in a crossword rather than just show a blank screen after the error message.

FayCross commented 3 years ago

@ingedonke I've pasted Takeshi's comments from his recent testing into this issue too. Thanks

FayCross commented 2 years ago

Closing this as most of bug list are now fixed. Moving around grid with keyboard could probably be slightly improved but it's not too bad