LDBR-3.20: Правки после РК2 (#36)

* LDBR-3.20: Исправить МиБ, исправить ошибку несовпадающих паролей, исправить автофокус в попапах. * LDBR-3.20: разрешить все виды картинок.
frontend-park-mail-ru · Nov 25, 2021 · adce9ea · adce9ea
1 parent 90d0594
commit adce9ea
Show file tree

Hide file tree

Showing 10 changed files with 22 additions and 11 deletions.
diff --git a/src/constants/constants.js b/src/constants/constants.js
@@ -72,7 +72,7 @@ export const ConstantMessages = {
     NoSpecialSymbolsPassword: 'Введите пароль, не содержащий специальных символов',
     UseOnlyLatinLettersPassword: 'Введите пароль, содержащий только латинские буквы',
 
-    AvatarTooBig: 'Аватар не должен превышать 500 МиБ',
+    AvatarTooBig: 'Аватар не должен превышать 500 КБ',
 
     BoardTitleTooLong: 'Название доски превышает 60 символов',
     BoardTitleTooShort: 'Название доски слишком короткое',
@@ -87,6 +87,9 @@ export const ConstantMessages = {
     CardListTitleTooLong: 'Название списка карточек слишком длинное',
     CardListErrorOnServer: 'Не удалось создать список, попробуйте позднее',
 
+    CardTitleTooShort: 'Название карточки слишком короткое',
+    CardTitleTooLong: 'Название карточки слишком длинное',
+    CardErrorOnServer: 'Не удалось создать карточку, попробуйте позднее',
     UnsuccessfulRequest: 'Неудачный запрос, попробуйте позднее :]',
 };
 

diff --git a/src/modules/Validator/Validator.js b/src/modules/Validator/Validator.js
@@ -130,10 +130,10 @@ export default class Validator {
      */
     validateCardTitle(title) {
         if (title.length < 1) {
-            return ConstantMessages.CardListTitleTooShort;
+            return ConstantMessages.CardTitleTooShort;
         }
         if (title.length > 40) {
-            return ConstantMessages.CardListTitleTooLong;
+            return ConstantMessages.CardTitleTooLong;
         }
         return null;
     }

diff --git a/src/popups/BoardSetting/BoardSettingPopUp.js b/src/popups/BoardSetting/BoardSettingPopUp.js
@@ -53,6 +53,7 @@ export default class BoardSettingPopUp extends BaseComponent {
         this._elements.confirmBtn?.addEventListener('click', this._onDeleteConfirm);
         this._elements.rejectBtn?.addEventListener('click', this._onRejectConfirm);
         this._elements.saveBtn?.addEventListener('click', this._onSave);
+        this._elements.title?.focus();
     };
 
     /**

diff --git a/src/popups/Card/CardPopUp.js b/src/popups/Card/CardPopUp.js
@@ -74,6 +74,7 @@ export default class CardPopUp extends BaseComponent {
         this._elements.closeBtn?.addEventListener('click', this._onPopUpClose);
         this._elements.createBtn?.addEventListener('click', this._onCreate);
         this._elements.saveBtn?.addEventListener('click', this._onSave);
+        this._elements.card_name?.focus();
         this._elements.comments?.editBtns?.forEach((editCommentBtn)=>{
             editCommentBtn.addEventListener('click', this._onEditComment);
         });

diff --git a/src/popups/CardList/CardListPopUp.js b/src/popups/CardList/CardListPopUp.js
@@ -47,6 +47,7 @@ export default class CardListPopUp extends BaseComponent {
         this._elements.closeBtn?.addEventListener('click', this._onPopUpClose);
         this._elements.createBtn?.addEventListener('click', this._onCreate);
         this._elements.saveBtn?.addEventListener('click', this._onSave);
+        this._elements.title?.focus();
     };
 
     /**

diff --git a/src/popups/CreateBoard/CreateBoardPopUp.js b/src/popups/CreateBoard/CreateBoardPopUp.js
@@ -45,6 +45,7 @@ export default class CreateBoardPopUp extends BaseComponent {
         this._elements.closeBtn?.addEventListener('click', this._onPopUpClose);
         this._elements.submitBtn?.addEventListener('click', this._onSaveBtnClick);
         document.addEventListener('click', this._onPopUpClose);
+        this._elements.name?.focus();
     };
 
     /**

diff --git a/src/popups/DeleteCard/DeleteCardPopUp.hbs b/src/popups/DeleteCard/DeleteCardPopUp.hbs
@@ -9,8 +9,8 @@
                     <div class="horizontal-line"></div>
                     <div class="popup-form__elements-wrapper">
                         <div class="confirm-delete-wrapper">
-                            <button class="button button_red button_full-space" id="deleteCardPopUpConfirmBtnId">Удалить</button>
-                            <button class="button button_full-space" id="deleteCardPopUpRejectBtnId">Не удалять</button>
+                            <button class="button button_red button_full-space" id="deleteCardPopUpConfirmBtnId">Да, удалить</button>
+                            <button class="button button_full-space" id="deleteCardPopUpRejectBtnId">Нет, отменить</button>
                         </div>
                     </div>
                 </form>

diff --git a/src/popups/DeleteCardList/DeleteCardListPopUp.hbs b/src/popups/DeleteCardList/DeleteCardListPopUp.hbs
@@ -9,8 +9,8 @@
                     <div class="horizontal-line"></div>
                     <div class="popup-form__elements-wrapper">
                         <div class="confirm-delete-wrapper">
-                            <button class="button button_red button_full-space" id="deleteCLPopUpConfirmBtnId">Удалить</button>
-                            <button class="button button_full-space" id="deleteCLPopUpRejectBtnId">Не удалять</button>
+                            <button class="button button_red button_full-space" id="deleteCLPopUpConfirmBtnId">Да, удалить</button>
+                            <button class="button button_full-space" id="deleteCLPopUpRejectBtnId">Нет, отменить</button>
                         </div>
                     </div>
                 </form>

diff --git a/src/stores/UserStore/UserStore.js b/src/stores/UserStore/UserStore.js
@@ -263,9 +263,13 @@ class UserStore extends BaseStore {
             }
         }
 
-        if (data.hasOwnProperty('passwordRepeat') && (data.password !== data.passwordRepeat)) {
-            validation.passwordRepeat = ConstantMessages.NonMatchingPasswords;
-            this._storage.get(form).passwordRepeat = '';
+        if (data.hasOwnProperty('passwordRepeat')) {
+            if (data.password !== data.passwordRepeat) {
+                validation.passwordRepeat = ConstantMessages.NonMatchingPasswords;
+                this._storage.get(form).passwordRepeat = '';
+            } else {
+                validation.passwordRepeat = '';
+            }
         }
     }
 

diff --git a/src/views/ProfileView/ProfileView.hbs b/src/views/ProfileView/ProfileView.hbs
@@ -6,7 +6,7 @@
             <div class="profile-box__column-left">
                 <div class="profile-container">
                     <img class="profile-box__user-picture" src="{{avatar}}" alt="User Picture" id="avatar-img">
-                    <input type="file" name="avatar" id="avatar" class="profile-box__user-avatar" accept="image/png" value="{{avatar}}">
+                    <input type="file" name="avatar" id="avatar" class="profile-box__user-avatar" accept="image/*" value="{{avatar}}">
                     <label for="avatar" class="profile-box__upload-button">
                         <span class="profile-box__upload-icon-wrapper"><img class="profile-box__upload-icon" src="assets/icons/upload.webp" alt="Выбрать файл" width="25"></span>
                         <span class="profile-box__upload-button-text">Выберите файл</span>