Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Забележки по формата за протоколи #27

Open
vladimiroff opened this issue Jan 13, 2014 · 5 comments
Open

Забележки по формата за протоколи #27

vladimiroff opened this issue Jan 13, 2014 · 5 comments
Assignees

Comments

@vladimiroff
Copy link
Contributor

Когато ми се наложи да тествам тази форма се побърках. При най-малката грешка попълвам едни и същи данни постоянно със странни widget-и.

  1. Widget-а за часа е отвратително неудобен. Има AM/PM!? Django има widget за това.
  2. Widget-а за датата не му отстъпва с много. Без да искам попълних 0014-та година. След това не е лесно да се върне човек, а като избера дата той не се скрива, докато не натисна странен бутон с точица. Освен това изглежда зле. Django има widget и за това.
  3. Полетата за присъстващи, ако не се попълнят гърмят със странна грешка от типа "това поле не може да е -1"

Всяко едно от горепосочените полета + още няколко си губят данните при изпращане и наличие на грешка. Представи си следния use case. Има събрание, на което трябва да присъстват 60 души, но 18 от тях липсват и едва 4-ма са освободени. Представи си как ги попълваш веднъж, забравяш да попълнеш едно от много други полета и те накара да ги вкараш от ново.

@Ivaylo-Bachvarov
Copy link

Първите две са от Bootstrap и хората ги харесват. Ти си хейтър! 😆 :trollface:

Всички други форми разчитат на това, че след събмит се връща грешка от backend заедно с value-то на полетата. Но в тази форма всичко е твърде динамично и няма как да стане по този начин. Ще го преработя да работи с local storage и да запомня всичко, което си писал там. Така може да се вкара и функционалност за чернова, която искат от СС. (Да си запомниш нещо локално преди да го качиш)

@vladimiroff
Copy link
Contributor Author

Може да са много красиви, но не са функционални. Това с датата е очевиден бъг и няма нужда от никаква динамична информация за да се държи адекватно и след пращане. Освен това хортата, които харесват това AM/PM са американци. Това не е аргумент да ги имаме и ние.

Още веднъж: Django има widget-и и за двете. Ползвай тях и си ги стилизирай както искаш.

За присъстващи и отсъстващи пращаш id-та на потребители и техните id-та стоят там при грешка. Можеш да ги ползваш за да вземеш отново юзърите зад тях. Нямаш нужда от local storage. За организацията важи същото.

@Ivaylo-Bachvarov
Copy link

За PM и AM се съгласих.

Може ли да направим запомнянето на чернови в backenda. Защото ако ще ползваме localstorage по добре и динамичните форми да са с него?

@vladimiroff
Copy link
Contributor Author

Дюд, нямаш нужда от local storage за това. Във value-то на всеки един input field стойността трябва да бъде попълнена. Т.е. за организация ще има числото 1, за присъстващи също разни числа. Вземаш ги и си фетчваш.

@Ivaylo-Bachvarov
Copy link

Тази форма ще се рефакторира цялата. Доста е тегава така...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants