Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- 1. Potrzebujesz dwóch formularzy? Ja bym zrobiła jeden i dodawanie notatki byłoby na submicie, a to pole z wyszukaniem powinno działać po wpisaniu literki.
- 2. Jak to wyszkukiwanie miałoby działać? szuka odpowiedniego wyrazu w każdej notatce? Bo masz "wyszukaj swoją notatkę" - to tak jakby po nazwie notatki miało szukać. To w sumie rzeczy UXowe :P
- 3. Fajnie by było jakby po każdym dodaniu taska czyściło się automaczynie to, co wpisaliśmy wcześniej.
- 4. Mogę dodać pustą notatkę (tzn jak kilka spacji tam wrzucę), a to też trzeba chyba uniemożliwić.
- 5. Dlaczego inputy nie mają labeli? Czytnik nie będzie wiedział do czego dany input jest.
- 6. Tam gdzie masz:
- ```html
- <div class="task">
- <div class="task-item"></div>
- <div class="task-item"></div>
- <div class="task-item"></div>
- </div>
- ```
- 7. To przecież to aż prosi się o listę :P
- ```html
- <ul class="tasks">
- <li class="task-item"></li>
- <li class="task-item"></li>
- <li class="task-item"></li>
- </ul>
- ```
- 8. Dlaczego:
- ```html
- <div class="task-text">
- jakis tekst
- </div>
- ```
- a nie:
- ```html
- <p class="task-text">
- jakis tekst
- </p>
- ```
- ?
- 9. Do wstawiania godziny mamy odpowiedni znacznik + niepoprawnie ją wyświetla bo ja dostałam:
- `21-6-2018 godz.: 17:4`
- 10. Potrzebujesz używać id?
- 11. Po co:
- ```javascript
- let taskText = null;
- let noteForm = null;
- let searchInput = null;
- ```
- ?
- Można zapisać po prostu:
- ```javascript
- let taskText;
- let noteForm;
- let searchInput;
- ```
- I to będzie to samo co:
- ```javascript
- let taskText = undefined;
- let noteForm = undefined;
- let searchInput = undefined;
- ```
- 12. Komentarze piszemy po angielsku
- 13. Nie potrzbujesz tego zdarzenia:
- ```javascript
- 'DOMContentLoaded'
- ```
- Przecież mówiłam, kiedy to jest potrzebne i chyba sam nawet rozumiałeś, bo też o tym pisałeś :P
- I jeszcze na dodatek masz dwa razy takie zdarzenie podpięte.
Add Comment
Please, Sign In to add comment