Skip to content

Conversation

@SnakyJake
Copy link
Contributor

convert formatcheck.js to amd module
handle different decimal separators like commas

@SnakyJake SnakyJake changed the title Stable 5.3 handle commas as decimal separators Mar 28, 2025
@SnakyJake SnakyJake changed the title handle commas as decimal separators handle commas as decimal separators, formatcheck as amd module Mar 28, 2025
@PhilippImhof
Copy link
Collaborator

Hello Jakob and thanks for the contribution. There is one problem though: the 5.3 branch is no longer under active development. And for the 6.x branch (main), I have already started rewriting all legacy Javascript code, e.g. in #163 and #164, because the whole structure of it has changed.

@PhilippImhof
Copy link
Collaborator

As a side note, it is still planned to integrate handling of comma as a decimal separator in future versions (maybe 6.1.x). But unfortunately, it's more complicated than just to replace every occurrence of a comma by a point, as there can be legitimate uses of a comma in the student input.

@SnakyJake
Copy link
Contributor Author

Ok, nice to know. Than please ignore this pull request.
We desperately need comma handling, so I implement it here so it works with our questions.
I will not do any pull request if you plan impleneting it anyways. I think you better know where to implement it than me trying to guess where the best place might be. Thanks alot...

@PhilippImhof
Copy link
Collaborator

Good to know that you mainly did it, because you needed it, so I do not make you lose your time. I want to make it very clear that I appreciate your contribution and I hope my answer did not come across hostile.

Also, one of the main objectives of my full rewrite was to make it easier for motivated people to contribute, thanks to more modern practices and better documentation in the code. Because when I started maintaining the plugin, I found it very hard to understand all the inner workings of the plugin and thought that it was not particularly "inviting"…

@PhilippImhof
Copy link
Collaborator

I will now lose this PR, as localised number formatting will be implemented in PR #228

I invite you to have a look over there and submit your feedback or ideas.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants