Merge pull request #19 from ksenya3112/bug-fix3

Добавляет исправленные замечания по защите

Author: Spearance

js/form.js

@@ -104,6 +104,7 @@ const resetVisuals = () => {
   currentScale = DEFAULT_SCALE;
   imgPreview.style.transform = `scale(${DEFAULT_SCALE / 100})`;
   scaleControlValue.value = `${DEFAULT_SCALE}%`;
+  scaleControlValue.setAttribute('value', `${DEFAULT_SCALE}%`);
   updateEffect('none');
   uploadForm.querySelector('#effect-none').checked = true;
 };
@@ -118,9 +119,10 @@ const closeUploadModal = () => {
   document.removeEventListener('keydown', onDocumentKeydown);
 };
 
-// Обработчик Esc (не закрывать, если фокус в полях ввода)
+// Обработчик Esc
 function onDocumentKeydown(evt) {
-  if (isEsc(evt) && document.activeElement !== hashtagInput && document.activeElement !== commentInput) {
+  const isErrorMessageOpen = !!document.querySelector('.error');
+  if (isEsc(evt) && !isErrorMessageOpen && document.activeElement !== hashtagInput && document.activeElement !== commentInput) {
     evt.preventDefault();
     closeUploadModal();
   }
@@ -154,6 +156,7 @@ scaleControlSmaller.addEventListener('click', () => {
     currentScale -= SCALE_STEP;
     imgPreview.style.transform = `scale(${currentScale / 100})`;
     scaleControlValue.value = `${currentScale}%`;
+    scaleControlValue.setAttribute('value', `${currentScale}%`);
   }
 });
 
@@ -162,6 +165,7 @@ scaleControlBigger.addEventListener('click', () => {
     currentScale += SCALE_STEP;
     imgPreview.style.transform = `scale(${currentScale / 100})`;
     scaleControlValue.value = `${currentScale}%`;
+    scaleControlValue.setAttribute('value', `${currentScale}%`);
   }
 });
 

js/main.js

@@ -1,6 +1,5 @@
 
 import './util.js';
-import './data.js';
 import './form.js';
 
 import { renderThumbnails } from './thumbnails.js';

js/message.js

@@ -9,7 +9,7 @@ function hideMessage() {
   if (messageElement) {
     messageElement.remove();
   }
-  document.removeEventListener('keydown', onDocumentKeydown);
+  document.removeEventListener('keydown', onDocumentKeydown, true);
   document.removeEventListener('click', onBodyClick);
 }