MastersAcademy / frontend-course-2021

Репозиторій з курсу «Advanced Front-end» проекту Masters Academy
MIT License
5 stars 29 forks source link

3-async #112

Closed Vladimior closed 3 years ago

Vladimior commented 3 years ago

https://vladimior.github.io/frontend-course-2021/homeworks/volodymyr.khrushch_Vladimior/3-async/index.html

TArch64 commented 3 years ago

Пофікси еррори лінтерів

Vladimior commented 3 years ago

Пофікси еррори лінтерів на данний момент я не знаю як його пофіксити, я пробував багато варіантів. переписував по іншому фукцію, писав ось так. і все рівно вивалює таку помилку. я весь час сиджу читаю доки, дивлюсь як виправити. `const filter = function () { const input = document.querySelector('[data-input]');

input.addEventListener('keyup', () => {
    const filters = input.value.toLowerCase();
    const filterElements = container.querySelectorAll('[data-title]');

    filterElements.forEach((item) => {
        if (item.innerHTML.toLowerCase().indexOf(filters) > -1) {
            item.closest('[data-card]').style.display = '';
        } else {
            item.closest('[data-card]').style.display = 'none';
        }
    });`
TArch64 commented 3 years ago

Пофікси еррори лінтерів на данний момент я не знаю як його пофіксити, я пробував багато варіантів. переписував по іншому фукцію, писав ось так. і все рівно вивалює таку помилку. я весь час сиджу читаю доки, дивлюсь як виправити. `const filter = function () { const input = document.querySelector('[data-input]');

input.addEventListener('keyup', () => {
    const filters = input.value.toLowerCase();
    const filterElements = container.querySelectorAll('[data-title]');

    filterElements.forEach((item) => {
        if (item.innerHTML.toLowerCase().indexOf(filters) > -1) {
            item.closest('[data-card]').style.display = '';
        } else {
            item.closest('[data-card]').style.display = 'none';
        }
    });`

Пробема в тому що ти всередині функції змінюєш поле аргумента, що наш лінтер забороняє. Підправив правило щоб так можна було робити

Vladimior commented 3 years ago

дякую. "Пробема в тому що ти всередині функції змінюєш поле аргумента, що наш лінтер забороняє.", постараюсь десь почитати чому так не бажано робити.

TArch64 commented 3 years ago

Виконання не повністю відповідає дизайну image

TArch64 commented 3 years ago

В полі сьорчу немає відступів зправа та зліва image

TArch64 commented 3 years ago

Там вуглики мають інший колір ніж інша частина фону image