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

Redesign Homepage, by Misah - Fixed #6 #7

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

MisahSaid
Copy link
Collaborator

Ik heb een redesign gemaakt van de homepage in figma:

Screenshot 2024-05-17 at 11 23 55

Uit de feedback van de opdrachtgevers hebben we gekregen dat we meer animaties moeten toevoegen als dat lukt. Dat heb ik dus tot nu toe uitgewerkt tot version 1, hij is niet volledig af.

Maar waar opgelet moet worden tijdens de reviews is, of ik me aan de code regels heb gehouden en of ik de animatie op de beste manier heb gebouwd met PE in mind. (Ik heb namelijk voor de animatie alleen CSS gebruikt met als doel om zo min mogelijk JS te gebruiken.)

Screenshot 2024-05-17 at 11 27 52

CSS, Animatie:

/* right to left */
@keyframes moveSideToSide {
0% {
transform: translateX(0);
}
50% {
transform: translateX(100px);
}
100% {
transform: translateX(0);
}
}
/* left to right + little delay */
@keyframes moveSideToSide2 {
0% {
transform: translateX(0);
}
40% {
transform: translateX(-100px);
}
100% {
transform: translateX(0);
}
}
#moving-svg {
animation: moveSideToSide 4s ease-in-out ;
}
#moving-svg2 {
animation: moveSideToSide2 4s ease-in-out ;
}

CSS, Code regels:
https://github.com/lisavanmansom/pleasurable-ui/wiki/Week-1#code-conventies

Voorbeelden:

  • Wel vind ik dat het de html structuur beter kan, ivm tijd heb ik dat genegeerd, we kunnen hier bijvoorbeeld een issue van maken :)

Fixed #6

@lisavanmansom
Copy link
Owner

Super leuk design! Verder heb je ook code volgens de code conventies en is je design in de stijl van Tumi mundo.

Copy link
Collaborator

@KrijnHoetmer KrijnHoetmer left a comment

Choose a reason for hiding this comment

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

Er zijn best veel gewijzigde bestanden in dit pull request, dus dit lijkt meer dan wat je beschrijft. Probeer je pull requests kleiner te houden, zodat de code ook gereviewed kan worden.

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.

5 participants