Skip to content

Commit 1f15130

Browse files
committed
Just a little code optimitation
1 parent 5dc95cc commit 1f15130

File tree

4 files changed

+22
-20
lines changed

4 files changed

+22
-20
lines changed

assets/js/buttons-component.js

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -60,9 +60,13 @@ export default class ButtonsComponent extends ListComponent{
6060
let idNumber = button.parentElement.parentElement.id;
6161
button.addEventListener('click', async () => {
6262
store.dispatch('removeItem', {id});
63-
await request.deleteItemRequest(idNumber);
64-
this.onInitList();
65-
this.onInitButtons();
63+
request.deleteItemRequest(idNumber)
64+
.then(response => response)
65+
.catch(err => console.log(err))
66+
.finally(() => {
67+
this.onInitList();
68+
// this.onInitButtons();
69+
});
6670
})
6771
});
6872

assets/js/item-component.js

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -154,8 +154,6 @@ export default class ItemComponent extends ButtonsComponent {
154154
}
155155
});
156156

157-
158-
159157
/*
160158
Добавляем событие на элемент списка, которое открывает редактор элемента
161159
(делегирование)

assets/js/list-component.js

Lines changed: 13 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -30,8 +30,9 @@ export default class ListComponent extends FormComponent {
3030
this.anchorList.innerHTML = `
3131
<ul>
3232
<li class="content__main-results-list-item list--clear">
33-
<p class="content__main-results-list-item-text content--clear">
34-
No todos. You are free for today!</p>
33+
<p class="content__main-results-list-item-text content--clear">
34+
No todos. You are free for today!
35+
</p>
3536
</li>
3637
</ul>
3738
`;
@@ -42,17 +43,16 @@ export default class ListComponent extends FormComponent {
4243
`;
4344
} else {
4445
this.anchorList.innerHTML = `
45-
<ul>
46-
${store.props.map(todoItem => `
47-
<li class="content__main-results-list-item" id="${todoItem._id}"
48-
executionStatus="${todoItem.completed}" markedDone="${todoItem.completed}">
49-
<p class="content__main-results-list-item-text">${todoItem.text}</p>
50-
<div class="content__main-results-list-item-buttons"></div>
51-
</li>
52-
`).join('')}
53-
</ul>
54-
`;
55-
this.render();
46+
<ul>
47+
${store.props.map(todoItem => `
48+
<li class="content__main-results-list-item" id="${todoItem._id}"
49+
executionStatus="${todoItem.completed}" markedDone="${todoItem.completed}">
50+
<p class="content__main-results-list-item-text">${todoItem.text}</p>
51+
<div class="content__main-results-list-item-buttons"></div>
52+
</li>
53+
`).join('')}
54+
</ul>
55+
`;
5656
}
5757
console.log( 'ListComponent rendered' );
5858
}

assets/js/network-request.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -110,8 +110,8 @@ export default class NetworkRequest {
110110
return res;
111111
}
112112

113-
deleteItemRequest(id) {
114-
fetch(`${new URL(`todos/${id}`, this.serverURL)}`, {
113+
async deleteItemRequest(id) {
114+
await fetch(`${new URL(`todos/${id}`, this.serverURL)}`, {
115115
method: 'DELETE',
116116
headers: {
117117
'Content-Type': 'application/json',

0 commit comments

Comments
 (0)