Adding config options to change letter to be searched in words in capital.
- Group Reviewers
GCompris GCompris: Improvements
- Maniphest Tasks
- T5936: letter_in_word: Adding a choice between capital and non-capital letters
|69 ↗||(On Diff #23987)|
not a good variable name
|143 ↗||(On Diff #23987)|
We can have a combobox with the 3 possible values instead: lowercase, uppercase and mixed (we randomly choose if lower or upper) and we can be more precise telling it's only the letter to find that will be in lower/uppercase (the words won't be affected by this setting).
|152 ↗||(On Diff #23987)|
you should only reload the level on onSaveData where you need to store the value
|121 ↗||(On Diff #23987)|
if I'm not wrong (I didn't tested), there is a global configuration option to set all the texts in uppercase.
|69 ↗||(On Diff #24448)|
why don't you directly use the string value (not the text displayed, but a value associated to the combobox text)? It will be more explicit when used than the index
|70 ↗||(On Diff #24448)|
as it is only used in the combobox (and not in the js file), it is better to declare it in the combobox
|152 ↗||(On Diff #24652)|
If I put lowercase, leave the activity and return it is saved as mixed case. Same if I put mixed case, restart GCompris, it will be lowercase.
|293 ↗||(On Diff #24652)|
the dataset is fully on lowercase so you should keep the rand()%2 here in mixed case
Sorry for delay, there is still an issue with mixed case
|69 ↗||(On Diff #24711)|
The default value should be the same as ApplicationSettings.fontCapitalization
|293 ↗||(On Diff #24711)|
after testing, it does not work as expected for MixedCase.