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

Mobile redesign #476

Merged
merged 31 commits into from
Mar 19, 2020
Merged

Mobile redesign #476

merged 31 commits into from
Mar 19, 2020

Conversation

Rotendahl
Copy link
Contributor

@Rotendahl Rotendahl commented Mar 10, 2020

En meget stor PR.
Den ændrer følgende:

  • Når du logger ind bliver du sendt til oversigts siden i stedet for familiesiden
  • Den skifter den gamle datepicker ud med browser standard picker. Specielt på mobiler er det lækkert. Der er en polypill så det virker i alle browsere (også IE11)
  • Fjerner en form der ikke blev brugt
  • Rydder op i html markup og CSS klasser
  • Lav en get_open_departsments function der returnerer en liste over alle afdelinger som ikke er lukket og har haft en aktivitet det seneste år. (Senere skal vi lave lavet sådan at den sender en mail til sek og afdelingsleder og lukker afdelingen)
  • CSS og javascript til nemt at lave faner
  • Opdaterer table CSS og javascript så det virker på en mobil
  • Generel styling
  • Ændrer i sider så ventelister og afdelinger er slået sammen til "tilmeldinger"

Screenshot kommer senere

@codecov
Copy link

codecov bot commented Mar 10, 2020

Codecov Report

Merging #476 into master will decrease coverage by 0.01%.
The diff coverage is 63.88%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master    #476      +/-   ##
=========================================
- Coverage   62.82%   62.8%   -0.02%     
=========================================
  Files         104     103       -1     
  Lines        2846    2815      -31     
=========================================
- Hits         1788    1768      -20     
+ Misses       1058    1047      -11
Impacted Files Coverage Δ
members/urls.py 100% <ø> (ø) ⬆️
members/forms/admin_signup_form.py 78.12% <ø> (ø) ⬆️
members/utils/address.py 11.11% <ø> (-55.56%) ⬇️
members/forms/__init__.py 100% <ø> (ø) ⬆️
members/forms/activity_signup_form.py 66.66% <ø> (ø) ⬆️
members/forms/activity_invite_decline_form.py 45.45% <ø> (ø) ⬆️
members/forms/volunteer_signup_form.py 78.12% <ø> (ø) ⬆️
members/forms/signup_form.py 100% <ø> (ø) ⬆️
members/views/PersonCreate.py 31.42% <0%> (ø) ⬆️
members/views/WaitingListSetSubscription.py 36.66% <0%> (ø) ⬆️
... and 13 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 73cfecd...8932172. Read the comment docs.

@Rotendahl Rotendahl temporarily deployed to medlemssyste-mobile-red-ns3gi9 March 10, 2020 12:06 Inactive
@Rotendahl Rotendahl temporarily deployed to medlemssyste-mobile-red-ns3gi9 March 10, 2020 12:27 Inactive
@Rotendahl Rotendahl temporarily deployed to medlemssyste-mobile-red-ns3gi9 March 10, 2020 13:47 Inactive
@Rotendahl Rotendahl temporarily deployed to medlemssyste-mobile-red-ns3gi9 March 10, 2020 14:03 Inactive
Quick update with testing command
@@ -10,7 +9,6 @@
ActivivtyInviteDeclineForm,
ActivitySignupForm,
PersonForm,
getLoginForm,
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Formen blev ikke brugt nogen steder

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Vi bør også fjerne get_login_form.py filen.

Comment on lines 22 to 26
section {
@extend %rounded-rect;
margin: 15px 0;
@extend %card;
}
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Visuel adskilningen mellem sections

@@ -1,5 +0,0 @@
{% if user.is_authenticated %}
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Blev ikke brugt

@Rotendahl Rotendahl marked this pull request as ready for review March 12, 2020 09:21
@JakobLibak
Copy link
Member

Kommentarer til /department_signup

image

Forslag til ny tekst:

De fleste afdelinger starter nye hold to gange om året. Som regel i februar og september. Du kan skrive dit barn op til en eller flere afdelinger her. Næste gang afdelingen åbner op for nye tilmeldinger vil I modtage en mail med en invitation.
Under menupunktet Arrangementer kan I se afdelinger der har åbent for tilmeldinger allerede nu og se invitationer sendt til jer. [Gå til Arrangementer]

image
Forslag til ny tekst:

Tilmeld Venteliste
Tilmeld jer kun de afdelinger, der har interesse for jer. Det er en god ide at kigge forbi hyppigt op til sæsonstart, da vi tilføjer nye afdelinger efterhånden som vi får bekræftet de åbner og børnene får kun tilbud om at komme med på de afdelinger de har skrevet sig op til. Se jeres nuværende ventelistestatus under "Se Pladser"

image
Forslag til ny tekst:

Pladser på venteliste af vejledende. Den enkelte afdeling kan vælge at prioritere børn længere nede på listen for at få den bedst mulige fordeling af børn på holdet. Dit barns plads på ventelisten bliver beregnet ud fra, hvornår barnet er oprettet i systemet. Det er altså barnets anciennitet, der bestemmer pladsen på ventelisten til samtlige afdelinger, barnet er skrevet op på. Jo højere anciennitet, des bedre plads på ventelisten(erne). Bemærk at jeres position på ventelisten afgøres ud fra barnets oprettelses tidspunkt i systemet - ikke tidspunktet I melder jeg på ventelisterne nedenfor! I kan derfor frit tilmelde og framelde jer ventelisterne uden at miste en position på listen.

@JakobLibak
Copy link
Member

Kommentarer til /activities

image
Knappen virker ikke. Den refererer til nuværende side og ikke /department_signup
image
Der er lidt trykfejl i nuværende tekst. Forslag til rettet tekst:

Her kan du tilmelde dit barn en aktivitet. Det kan enten være en åben aktivitet der ikke kræver opskrivning på venteliste eller hvis I har modtaget en invitation vil den kunne ses her. De aktiviteter som I allerede er tilmeldt vil også fremgå på denne side.

Jeg mangler at teste invitationer og nuværende tilmeldinger. Jeg vender tilbage, hvis jeg har nogle kommentarer på den del. @Rotendahl

@JakobLibak
Copy link
Member

family/activity/

image
Det er godt nok grønt heromkring. Faktisk så grønt at knapperne i bunden ikke rigtig ligner knapper.

@Rotendahl Rotendahl temporarily deployed to medlemssyste-mobile-red-ns3gi9 March 14, 2020 09:47 Inactive
@Rotendahl
Copy link
Contributor Author

Kommentarer til /family

image

Den-boksede "Opret"-tekst bør have samme farve som knapperne ender med at få.

Jeg foreslår at vi tilføjer en ekstra tekst

Under menupunktet Afdelinger kan du finde afdelinger skrive dit barn op til fremtidige klubaftener. Under Afdelinger kan du om der der allerede nu er afdelinger i nærheden af dig, der har åbent for tilmeldinger, og det vil også være her du kan tilmelde dit barn, hvis I har modtaget en invitation på mail.

Jeg har indsat din tekst og rykket "opret" knapperne op i toppen.
image, så ser man dem helt naturligt på mobil og desktop. Så tænker jeg vi ikke har brug for det andet.

Hvad tænker du om det @JakobLibak

@Rotendahl Rotendahl temporarily deployed to medlemssyste-mobile-red-ns3gi9 March 16, 2020 08:07 Inactive
@Rotendahl Rotendahl temporarily deployed to medlemssyste-mobile-red-ns3gi9 March 16, 2020 08:08 Inactive
@JakobLibak
Copy link
Member

Kommentarer til /family
image
Den-boksede "Opret"-tekst bør have samme farve som knapperne ender med at få.
Jeg foreslår at vi tilføjer en ekstra tekst

Under menupunktet Afdelinger kan du finde afdelinger skrive dit barn op til fremtidige klubaftener. Under Afdelinger kan du om der der allerede nu er afdelinger i nærheden af dig, der har åbent for tilmeldinger, og det vil også være her du kan tilmelde dit barn, hvis I har modtaget en invitation på mail.

Jeg har indsat din tekst og rykket "opret" knapperne op i toppen.
image, så ser man dem helt naturligt på mobil og desktop. Så tænker jeg vi ikke har brug for det andet.

Hvad tænker du om det @JakobLibak

Det ser rigtig godt ud. Teksten i toppen giver næsten ikke mening, så den må vi hellere få ændret også.

Din familie er registreret på e-mail adressen [email]. Information om tilmeldinger til hold og aktiviteter bliver sendt til denne adresse.

@Rotendahl Rotendahl temporarily deployed to medlemssyste-mobile-red-ns3gi9 March 16, 2020 21:13 Inactive
@Rotendahl
Copy link
Contributor Author

Din familie er registreret på e-mail adressen [email]. Information om tilmeldinger til hold og aktiviteter bliver sendt til denne adresse.

Det har du da helt ret i, den tekst er opdateret nu

@Rotendahl Rotendahl temporarily deployed to medlemssyste-mobile-red-ns3gi9 March 16, 2020 21:25 Inactive
@Rotendahl Rotendahl temporarily deployed to medlemssyste-mobile-red-ns3gi9 March 16, 2020 21:32 Inactive
@Rotendahl Rotendahl temporarily deployed to medlemssyste-mobile-red-ns3gi9 March 16, 2020 21:46 Inactive
@Rotendahl Rotendahl temporarily deployed to medlemssyste-mobile-red-ns3gi9 March 16, 2020 22:02 Inactive
Copy link
Contributor

@lakridserne lakridserne left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hvis du sørger for at fjerne den fil der er for meget så slipper vi den igennem :)

@@ -10,7 +9,6 @@
ActivivtyInviteDeclineForm,
ActivitySignupForm,
PersonForm,
getLoginForm,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Vi bør også fjerne get_login_form.py filen.

Comment on lines +87 to +106

@staticmethod
def get_open_departments():
result = []
a_year_ago = (timezone.now() - timedelta(days=366)).date()
for department in Department.objects.filter(closed_dtm=None).order_by(
"address__region", "name"
):
department_activties = Activity.objects.filter(
department=department
).order_by("-end_date")
if a_year_ago < department.created:
result.append(department)
elif (
len(department_activties) > 0
and a_year_ago < department_activties[0].end_date
):
result.append(department)

return result
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Skal vi lave en issue på det allerede så vi er opmærksomme på det?

@Rotendahl
Copy link
Contributor Author

@lakridserne get_login_form.py er allerde slettet.

Her er issue med det til afdelinger #480

@JakobLibak
Copy link
Member

Det ser godt ud herfra. Vi skubber den videre til staging, og så laver jeg en kort intro der kan kommunikeres ud til resten af foreningen.

@JakobLibak JakobLibak merged commit 664f568 into CodingPirates:master Mar 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants