Fixed issue #14858: Upload status is not visible enough (#1272)

LimeSurvey · May 8, 2019 · ffee4e1 · ffee4e1 · Shnoulle · May 8, 2019
1 parent 62fa3a0
commit ffee4e1
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/application/controllers/UploaderController.php b/application/controllers/UploaderController.php
@@ -325,7 +325,7 @@ function run($actionID)
                 </div>
 
                 <p class="alert alert-info uploadmsg">'.sprintf(gT("You can upload %s under %s KB each."), $qidattributes['allowed_filetypes'], $qidattributes['max_filesize']).'</p>
-                <div class="uploadstatus" id="uploadstatus"></div>
+                <div id="uploadstatus" class="uploadstatus alert alert-warning hidden"></div>
 
                 <!-- The list of uploaded files -->
             </div>

diff --git a/assets/scripts/uploader.js b/assets/scripts/uploader.js
@@ -219,11 +219,11 @@ function doFileUpload()
                 $('#' + fieldname + '_filecount').val(filecount);
 
                 if (filecount < minfiles) {
-                    $('#uploadstatus').html(uploadLang.errorNeedMore.replace('%s', (minfiles - filecount)));
+                    $('#uploadstatus').html(uploadLang.errorNeedMore.replace('%s', (minfiles - filecount))).removeClass('hidden');
                 } else if (filecount < maxfiles) {
-                    $('#uploadstatus').html(uploadLang.errorMoreAllowed.replace('%s', (maxfiles - filecount)));
+                    $('#uploadstatus').html(uploadLang.errorMoreAllowed.replace('%s', (maxfiles - filecount))).removeClass('hidden');
                 } else {
-                    $('#uploadstatus').html(uploadLang.errorMaxReached);
+                    $('#uploadstatus').html(uploadLang.errorMaxReached).removeClass('hidden');
                 }
                 fixParentHeigth(fieldname);
                 if (filecount >= maxfiles) {