Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Us AJAX for tests #112

Closed
HugoFara opened this issue Apr 27, 2023 · 2 comments
Closed

Us AJAX for tests #112

HugoFara opened this issue Apr 27, 2023 · 2 comments
Labels
enhancement Develop an existing feature ux User Experience could be better

Comments

@HugoFara
Copy link
Owner

Tests have some inconsistent behaviors: timer is not displayed, refresh rate is note perfect and worst of all: sounds are cut. The reason is that originally, everything was in different frames, and only the test frame was reloaded, this is no longer an option.

Some nice AJAX requests could bring back good behaviors!

@HugoFara HugoFara added enhancement Develop an existing feature ux User Experience could be better labels Apr 27, 2023
HugoFara added a commit that referenced this issue Apr 30, 2023
HugoFara added a commit that referenced this issue Apr 30, 2023
A defreeze button still here as it is an experimental feature.
@HugoFara
Copy link
Owner Author

HugoFara commented May 8, 2023

Phew, that is a lot of work!

I'm starting to see the end of it though. Anyone checking out 7997c95 will find:

  • New words coming from AJAX request.
  • Footer repaired (partially): timer is okay, words known/to do/done wrong/... are being updated, but the central slider has some minor appearance problems.
  • A debug button is still here (may be going away soon enough).
  • Musics play entirely.
  • A bug: after may words tested, no more words from the same text are retrieved.

It stills needs testing and fixing before going to master, but it is promising.

HugoFara added a commit that referenced this issue May 24, 2023
Word tests now use AJAX.
Prototype for a REST API.
Test colored slider repaired.
Fixed a bug which caused all files path to be wrong.
Implements #111: change textbox language according to text language.
Fixes #118: couldn't archive more than one text.
Fixes #121: saving settings was causing a warning.
@HugoFara
Copy link
Owner Author

Things seem to work, merged to master, enjoy!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Develop an existing feature ux User Experience could be better
Projects
None yet
Development

No branches or pull requests

1 participant