Jordane Gengo

Latest comments

• 

  Graeme Byrne•520

  @grmbyrnSubmitted about 3 years ago

  

  Ping single column coming soon page with some Javascript

  2

  10
  

  Jordane Gengo•590

  @jgengo-altPosted about 3 years ago

  Hello Graeme Byrne, I'm glad to see you did this assignment!

  🪴 Your solution looks great however I think:

  🌱 • (UX) • It would have been great if the text input also permit to validate the form with the ENTER key. You could have achieve that by using a real form html element instead of a div or by adding event listener with JS.

  🌱 • (UX) • The dashboard image attract all of the focus, it could look a bit better if have lower this effect by increasing the size of your headers in desktop mode to balance it out.

  🌱 • (UI) • The error message in desktop mode isn't place properly.

  Your mobile version is perfect 👌 

  I hope it helps Cheers happy coding 💪

  0
• 

  duynguyen0613•280

  @duynguyen0613Submitted about 3 years ago

  

  using simple flex to layout the solution

  1

  20
  

  Jordane Gengo•590

  @jgengo-altPosted about 3 years ago

  Hello there 👋

  First of all, good job for completing the assignment 🚀

  It seems the validation of the form isn't working, and can't find any trace of javascript, I would suggest you try to do it you could learn a lot! 🌱

  Also, looks like you didn't do the mobile version of the page.

  Keep going 💪

  0
• 

  Luciana Santos•60

  @Luciana-SantosSubmitted about 3 years ago

  

  Ping coming soon page solution

  1

  10
  

  Jordane Gengo•590

  @jgengo-altPosted about 3 years ago

  Hello Luciana 👋

  You really nailed it! 🚀

  I don't have any negative feedback, just wanted to pass-by to let you know I like your solution, works pretty nite and your JS code is super clean!

  Keep doing a great job 💪

  Marked as helpful

  0
• 

  👾 Ekaterine Mitagvaria 👾•7,780

  @catherineisonlineSubmitted about 3 years ago

  

  Pricing component with toggle

  6

  520
  

  Jordane Gengo•590

  @jgengo-altPosted about 3 years ago

  Hello Catherine 👋 good job completing this challenge. Keep up the good work 💪

  🪴 Your solution looks great however I think:

  🌱 • In terms of UX it would be better if your toggle button is clickable from the whole div and not only the circle

  🌱 • Also, your side buttons seems to change the border-radius on hover which makes it a bit weird to the user eye.

  🌱 • On wide screen it looks extremely huge, you would benefits having a container with a max-width

  Hope this help and happy coding!

  Marked as helpful

  1
• 

  Gustavo del prat gudden•20

  @GustavoGuddenSubmitted about 3 years ago

  

  seção de 4 cartoes

  1

  30
  

  Jordane Gengo•590

  @jgengo-altPosted about 3 years ago

  Hello Gustavo 👋

  Unfortunately your solution is not responsive, are you planing on re-taking it and trying to implement it? That would be great!

  Keep going my friend! 💪

  0
• 

  Antonio Rodrigues Júnior•500

  @juniorrdgsSubmitted about 3 years ago

  

  Article Preview Component

  1

  20
  

  Jordane Gengo•590

  @jgengo-altPosted about 3 years ago

  Hello 👋

  First of all, great job Antonio 🚀

  My only feedback regarding your assignment is that your share button has a position absolute relative to the body, not the card so on a wide screen your share button isn't inside the card.

  Keep going 💪

  Don't hesitate to follow me and give me feedback to my assignments :) I will follow back and make sure to give you advices in the future if you are interested!

  Marked as helpful

  0