@Abdul400
Submitted
please feel free to leave any feedback. thank you so much :0
@Abdul400
@Abdul400
Submitted
please feel free to leave any feedback. thank you so much :0
@Abdul400
Posted
hey @KaiPereira
thank you for your feedback. I will be considering them for my next challenge. :)
@Eman-AbdElZaher
Submitted
happy for your feedback
@Abdul400
Posted
hello @Eman-AbdElZaher, Generally, your work looks really good and is very responsive. However, here are a few things I noticed.
background-position
.::placeholder
property.Other than that. Good Job!!
@AnulQr
Submitted
Hi All Would appreciate feedback on this please. I'm using a screen size of 375px & 1440px
@Abdul400
Posted
hello @AnulQr. I think your work generally looks really good particularly for the desktop view. However, I noticed a few things.
Other than that, Great Job!
Marked as helpful
@Mhmd-Tarek-Mhmd
Submitted
@Abdul400
Posted
hello @Mhmd-Tarek-Mhmd,
I noticed that the JavaScript functionality for email validation works only when the email is invalid. as per the design files, I believe you should also have a (green text) notification when the email is valid. I also noticed that the background positioning is a bit off. Use the background-position
property to position your background.
Other than that, excellent work!
Marked as helpful
@sqle157
Submitted
My first challenge after a 4 months stop on coding. I hope this can give people inspiration on doing more challenges.
Feel free to give any feedbacks if you guys want!
@Abdul400
Posted
hi, @sqle157.
Generally, everything looks good and scalable. However, I think you have to work on the hover states for instance, use cursor: pointer
on the hover states for the avatar image, and also the name in the NFT component. Also, to allow for accessibility, kindly wrap the primary contents in a main html attribute. On the same note, provide alt descriptions for your images to also allow for accessibility.
Otherwise, nice Job for having a clean and readable code!
Marked as helpful
@Abdul400
Submitted
Hello guys, any feedback on my work would be highly appreciated. Thank you :)
@Abdul400
Posted
will definitely do so. Thank you for sharing your resources. I will definitely look into them.
@brodiewebdt
Submitted
This is my final version of the Easybank challenge. This was my toughest challenge yet. Because of the way I built the main nav I couldn't use border-bottom for the hover state without ruining it. I used a position:absolute "hack" to display the borders so they don't line up perfectly. Any feedback would be appreciated.
@Abdul400
Posted
I'd really like to know what your web dev setup looks like. I like how you develop so quickly and they still look so good.
@brodiewebdt
Submitted
This is my final version of the Easybank challenge. This was my toughest challenge yet. Because of the way I built the main nav I couldn't use border-bottom for the hover state without ruining it. I used a position:absolute "hack" to display the borders so they don't line up perfectly. Any feedback would be appreciated.
@Abdul400
Posted
I'm not sure how you develop your solutions so rapidly, I see you uploading in quick successions. Do you use bootstrap?
@Abdul400
Submitted
hi, any feedback on my work would be highly appreciated. Thank you
@Abdul400
Posted
Thank you so much, this solved my issue. I thought making the color above the picture and giving it a lower opacity was the only way to do so. I'm still not sure how your solution works, but it works. Could you please explain to me? unless the bg-intro-desktop.png itself has a transparent background, I'm not sure how the hsl(0, 100%, 74%) can be seen.
@brodiewebdt
Submitted
This is my final version of the Easybank challenge. This was my toughest challenge yet. Because of the way I built the main nav I couldn't use border-bottom for the hover state without ruining it. I used a position:absolute "hack" to display the borders so they don't line up perfectly. Any feedback would be appreciated.
@Abdul400
Posted
I have to admit this looks awesome!!
@Abdul400
Submitted
hi, any feedback on my work would be highly appreciated. Thank you
@Abdul400
Posted
That solved my issue. Thank you!!
@shashankbhat2
Submitted
@Abdul400
Posted
hello @shashankbhat2, your solutions looks great and the hover states are exactly as depicted. However, there are some things that I noted:
Marked as helpful