drlippman / IMathAS

IMathAS Online Math Assessment
Other
110 stars 98 forks source link

Updated #334

Closed Cash4Programming closed 2 years ago

Cash4Programming commented 2 years ago

I have updated the conversion to use gettext as the default. Updated simple to include create simplexcreatelatexinequalities Updated simple.html to include the new functions and show DEPRECIATED ones.

drlippman commented 2 years ago

I'm embarrassed to say it, but I think that if your intent was to adjust the spelling of units for different users on the same server based on their browser's stated language preference, your original approach of using the language header was probably the right one, though we'd still want to use gettext on the fallback to handle servers configured for non-English languages.

I'm not sure if you can easily revert back to that version. If not, I can try to find your earlier version.

Cash4Programming commented 2 years ago

No problems - I copied an earlier version from the github commits and updated the files.

My logic is for right now test for en-GB, then en-US, if neither match then use the gettext() to return the word.

ingodahn commented 2 years ago

Can I help with de?

drlippman commented 2 years ago

@ingodahn once we get this merged, if you want to updates the de translations, that'd be great.

Cash4Programming commented 2 years ago

That would be awesome!

Mike

On Sat, May 7, 2022, 9:11 AM David Lippman @.***> wrote:

@ingodahn https://github.com/ingodahn once we get this merged, if you want to updates the de translations, that'd be great.

— Reply to this email directly, view it on GitHub https://github.com/drlippman/IMathAS/pull/334#issuecomment-1120234482, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA37HZREAFYXOTDHY342JRLVI2I2NANCNFSM5VKQRKPQ . You are receiving this because you authored the thread.Message ID: @.***>

Cash4Programming commented 2 years ago

Yes, please!

On Sat, May 7, 2022, 8:50 AM Ingo Dahn @.***> wrote:

Can I help with de?

— Reply to this email directly, view it on GitHub https://github.com/drlippman/IMathAS/pull/334#issuecomment-1120231312, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA37HZVX3EU3LEP7JZ64C5DVI2GM7ANCNFSM5VKQRKPQ . You are receiving this because you authored the thread.Message ID: @.***>

drlippman commented 2 years ago

BTW Mike, it'd be much appreciated in the future if when developing a new feature you could do it in a branch, then PR that branch. That helps isolate those changes, so multiple changes aren't mixed together.

drlippman commented 2 years ago

I ended up making some changes, using a mix of your techniques. It uses the browser-stated English variant to use the GB/CA spelling when asked for, then falls back to using gettext for translation otherwise (which default to en-US if no translation language is specified)