
Adam M
@AdamMzkrAll comments
- @AdamMzkr@AdamMzkr
Thanks but I used some of semantic tags like main footer section. But I meet to put more focus on this next Time
- @stefansnaerh@AdamMzkr
Nice solution but: -no images in the header of sections (background-image) -no alt text to images Anyway, really good work :)
- @rodriwaw@AdamMzkr
good solution, but i think its to low padding in all messages if you add some padding will look so better:)
Marked as helpful - @rontoyhacao@AdamMzkr
hi, pefect solution and this smooth animated and validated card number amazing!!
- @AdamMzkr@AdamMzkr
Thank' s a loot for your comment. I know about breakpoints but i don't focus on it. I want to finish all newbie level challenges.( I FINISHED 17 chellenges) and in the next level I will more focus on all aspect of design. But your comment it's really important for me, because you are experience user.👋. I am still amateur 😉.
- @felix-toledo@AdamMzkr
GOOD job, but you need to learn about form validation and validate form. Focus more abut placing elements in form (flexbox) Anyway nice solution:) You miss Id for labels and language rule in HTML(head).
Marked as helpful - @mohamedKhaled89@AdamMzkr
Good Job but, in mobile view your design as a little bit to big. To see all content you need to scroll horizontally. I think change width container to 90% will help.
Marked as helpful - @KTrick01@AdamMzkr
Hi GREAT job! nice looking smooth and nice animation. Only some HTML issues to fix :)
- @RosaliaLey@AdamMzkr
Good job but i think divs in mobile view is a little bit too small (80% will looks better) but anyway next really good solution.
- @RosaliaLey@AdamMzkr
Perfect, i have nothing to add contacts 🎉
- @RosaliaLey@AdamMzkr
Great job but, you can add a little bit bottom padding in each column in mobile view that's in my opinion will looks better 😉
- @RosaliaLey@AdamMzkr
It's something wrong with your mobile view. Please check this and fix it;)
- @RosaliaLey@AdamMzkr
It's really nice only one thing you can change is : The container is a little bit too wide;) i think 40em will looks closer to design but good job💪
- @RosaliaLey@AdamMzkr
Pixel perfect congratulations 🎉
- @RosaliaLey@AdamMzkr
Really nice solution even better than mine;D congratulations :) No issue pixel perfect Good Job!
- @hrjuny@AdamMzkr
Nice solution but, it's some bugs in report check this and fix. Change div <attribution> for <footer> for example:)
Marked as helpful - @danielosd68@AdamMzkr
Really nice solution! Only what you can change is add semantic tags instead div's ;)
Marked as helpful - @ElPoderosoLukita@AdamMzkr
Hi nice solution but i see one big issue: You need to add media queries and change of card width because now your solution it's not responsive. And use flexbox to card center.
I hope it will be helpful;)
Marked as helpful