Redesign Homepage, by Misah - Fixed #6 #7
Open
MisahSaid wants to merge 5 commits into
Open
Conversation
Owner
|
Super leuk design! Verder heb je ook code volgens de code conventies en is je design in de stijl van Tumi mundo. |
KrijnHoetmer
left a comment
Collaborator
There was a problem hiding this comment.
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.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Ik heb een redesign gemaakt van de homepage in figma:
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.)
CSS, Animatie:
pleasurable-ui/public/styleM.css
Lines 130 to 162 in 1f7642e
CSS, Code regels:
https://github.com/lisavanmansom/pleasurable-ui/wiki/Week-1#code-conventies
Voorbeelden:
comments:
pleasurable-ui/public/styleM.css
Lines 130 to 132 in 1f7642e
inspringen:
pleasurable-ui/views/index.ejs
Lines 7 to 10 in 1f7642e
Fixed #6