keb/japanese-conjugation-drill
1
0
Fork 0
mirror of https://github.com/ZetaKebab/japanese-conjugation-drill.git synced 2025-01-14 22:08:44 +00:00
Code Issues Packages Projects Releases Wiki Activity

Added error message for plain/polite issue.

Browse Source 
The issue is that all tests involve either plain or polite and so unselecting
both will mean that all tests are unselected.

This patch adds error text that appears when both are unselected.
This commit is contained in:
doncr 2021-02-16 14:38:00 +00:00
parent 8daf3d81da
commit 241bb0e029
3 changed files with 14 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
conjugation/drill.css
View File

@ -308,6 +308,10 @@ button.btn.btn-primary {
color: red; color: red;
} }
#politePlainError {
color: red;
}
div.halfSpeed { div.halfSpeed {
font-size: 50%; font-size: 50%;
text-align: center; text-align: center;

4
conjugation/drill.html
View File

@ -147,6 +147,10 @@
</div> </div>
</div> </div>
<div id="politePlainError" style="display: none">
<div class="text-center mb-2">You must select at least one of 'Plain' and 'Polite'.</div>
</div>
<div class="text-center mb-4">Question pool size: <span id="questionCount">...</span></div> <div class="text-center mb-4">Question pool size: <span id="questionCount">...</span></div>
<div class="row justify-content-center mt-3 ml-2 mr-2"> <div class="row justify-content-center mt-3 ml-2 mr-2">

6
conjugation/drill.js
View File

@ -930,6 +930,12 @@ function updateOptionSummary() {
}); });
$("#questionCount").text(applicable); $("#questionCount").text(applicable);
if (!options.plain && !options.polite) {
document.querySelector('#politePlainError').style.display = 'block';
} else {
document.querySelector('#politePlainError').style.display = 'none';
}
} }
function updateVoiceSelect() { function updateVoiceSelect() {