akai-org / AKAI-generator-wnioskow

1 stars 6 forks source link

Zapamiętanie info, usuwanie achievków i kosmetyczne zmiany #28

Closed kalkson closed 4 years ago

kalkson commented 4 years ago

Dodałem możliwość usuwania osiągnięć, które nam nie pasują. Jak mniemam użytkownik musi wprowadzić choć jedno osiągnięcie, więc przy pierwszym osiągnięciu przycisku brak. Poza tym poprawiłem footer nieco powiększając jego wysokość i dodałem kilka kosmetycznych zmian.

Oprócz tego napisałem parę linijek w JSie, dzięki którym nasze informacje wprowadzone do forma zostają, tak na wypadek jakby ktoś chciał coś zmienić. Generalnie kod jest trochę dirty, ale wszystko chyba działa poprawnie (nie zaszkodzi potestować nieco więcej, żeby nie wrzucić fuckupa). W przyszłości zrobię jakiś refactor tego jsa żeby ładniej to wyglądało.

kalkson commented 4 years ago

Generalnie zrequestowałem do starszej wersji, bo nie zrobiłem pulla na nową xd, ale można zmergować akceptując obie zmiany

Jeeejeets commented 4 years ago

Bardzo ładnie, ale mam uwag kilka:

kalkson commented 4 years ago

Wprowadziłem uwagi, tj. zmieniłem event handler na keydown keyup, zamiast blura, żeby stan zapisywał się w czasie rzeczywistym. Usunąłem też hover na buttonie.

marcinlawnik commented 4 years ago

@kalkson dasz radę zrobić rebase mastera tutaj? Bo mamy konflikt, a nei chcę w tym grzebać żeby nie zepsuć ;)

kalkson commented 4 years ago

Proszę

marcinlawnik commented 4 years ago

deployed