search

h5p / h5p-blanks

9 stars 67 forks source link

User-specified alternate answer separator #85

Open davidjabon opened 3 years ago

davidjabon commented 3 years ago

An issue with the blanks content type is that the answer separator is required to be forward slash (/). This requirement means that answers that need forward slashes (like fractions, for mathematics instruction) are not possible. Through a few very minor changes in the codebase, I propose a version that leaves the forward slash in place as a default but allows users to specify an alternate answer separator. This is a non-breaking change. An alternative would be some kind of escape character, but that is not implemented here. I propose this as a conservative but very helpful improvement.

otacke commented 2 years ago

@davidjabon Just noticed this pull request: You'll have to update translation files as well when you change labels in semantics.json or if you add fields to it. H5P-CLI can help you with that. I am not a member of the H5P core team, but they'd most likely request this.