@aanacif
Submitted
@aanacif
@aanacif
Submitted
@aanacif
Posted
Got it! Thank you for the link and all the help ;)
@aanacif
Submitted
@aanacif
Posted
Yeah, semantic HTML and aria landmarks is something I still need to read more about. Thank you so much for your input :)
@aanacif
Submitted
@aanacif
Posted
Noticed it was missing the Accessibility options as I finished it. Will bring that in on next project. Rusty :)
@luizzzabiassi
Submitted
Aceitando feedbacks, tive bastante dificuldade com a imagem, ainda sim não ficou perfeito, se tiverem sugestões de melhorias eu vou adorar! 🎈
@aanacif
Posted
Ficou top!! Qnd trabalhei nesse projeto tb usei 'opacity' na foto, nao gostei do resultado e fiz entao um overlay daquela cor, mexendo com o canal alpha pra ajustar a transparência... ficou melhor, mas ainda nao ficou perfeito... um pouco mais de paciencia acho que eu chegava lá.. :D
De repente tenta algo nessa linha.... Vlw aí !!
Marked as helpful
@aanacif
Submitted
Shout out to the design team, love the colors 😎
@aanacif
Posted
No I agree, I saw it at some point and knew I had to do something about it.... Will check it out later... thank you :)
@xsrpm
Submitted
@aanacif
Posted
Been a while since I've done this one so I don't quite remember the details, but I noticed you're changing the padding on hover?? Only thing I remember being necessary 'design-wise' was to make the background transparent, no?
Or what exactly other issues you see?
Marked as helpful
@emhossein
Submitted
I used insertAdjacentHTML to display error message but couldn't find any way to remove it after some seconds so after clicking on button again it doesn't display error over and over any solution?
@aanacif
Posted
I feel your pain, keep pushing tho.
Quite a few different ways you could approach this. Maybe have a look at JS Element.setAttribute(). So you could use the 'hidden' attribute for example.. Or even give the error message a class say 'show'... then use JS classList.toggle to add/remove it. Or even hook it to the form validations. Have a look at oninvalid event for the inputs...
Hope that helps :)
Marked as helpful
Hello, Second Challenge Accomplished. Rate my code from 1 to 10. Consider your ratings based on how clean my code is, How close to the mockup I am!! IF there are any errors or improvements please don't hesitate to give me those remarks!! One question, How many viewport dimensions do we have to consider when creating websites?? When I try inspect the responsiveness on chrome dev tools, changing width and height , there are some sizes where the website breaks a little . So my question, do we consider all screen sizes and types of devices?? Is it possible to create a site that works on all screen sizes and devices?? I don't know if this is a very beginner question or not Lol Lol. Thank you!!
@aanacif
Posted
Looks good, dude. I also just did that one myself, and wasn't really sure about some dimensions, specially the Ipad width. Couldn't decide what to do with the picture dimension to make it look good, but oh well, still enjoyed doing it... Anyways, good job man 👍
@carlwicker
Submitted
Any feedback welcome.
@aanacif
Posted
Looks so good! :) Not sure why the Annualy/Monthly switch isn't centered for me? And I'd replace React's tab icon by the FEM's one :) Other than that, good stuff, Sir 👍
@juanpb96
Submitted
Any comments on my solution will be appreciated! 😄
@aanacif
Posted
Solid! 👍
Feedback são bem-vindos!
@aanacif
Posted
Mandou muito! 👍
Marked as helpful