@saki1994
Submitted
Hi everyone :)
Please provide a constructive comment about my project. I haven't added Drag and Drop.
Thank you so much!
Looking to hire developers?
@nikkuv
@saki1994
Submitted
Hi everyone :)
Please provide a constructive comment about my project. I haven't added Drag and Drop.
Thank you so much!
@nikkuv
Posted
Hye! I'm working on the same project, for now, your one is really good, plus you can use react Dnd for drag and drop in react.
Marked as helpful
@Akshita-Sharma-25
Submitted
How can I improve
@nikkuv
Posted
Hye Akshita ! you can improve the style of your testimonial card and in the footer, the logo is in black, if it is available in SVG you can change the "fill" property to set it white, also you can improve your social media icons, and set "aria-label" to resolve the accessibility issues on both the input and the link.
@sonickonic
Submitted
My first try to animate a landing page with CSS animations. Even though it is a newbie challenged I found it super interesting for this purpose! 🥳
Feedback is welcome and appreciated 🚀
@nikkuv
Posted
All animations are really really cool, there are accessibility issues but if you'll use "aria-label" on your links then it will be fine.
@SankThomas
Submitted
Any feedback on how to do the form validation
@nikkuv
Posted
First, apply all error styles with a class to the element, then select the element in js DOM and add that class to the element using the email regex function with conditionals. For email validation, you can use this email regex function👇
function validateEmail(email) {
const re = /^(([^<>()\[\]\\.,;:\s@"]+(\.[^<>()\[\]\\.,;:\s@"]+)*)|(".+"))@((\[[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}\])|(([a-zA-Z\-0-9]+\.)+[a-zA-Z]{2,}))$/;
return re.test(String(email).toLowerCase());
}
@Kerri-AnnBates
Submitted
Had a little trouble figuring out how to change the color of the share button image, so opted to creating a new image with a different color. Any advice on how to accomplish this?
@nikkuv
Posted
You did a great job on your first project. It almost looks the same as in the design. All you have to do is on your share button set the style of "outline: none" on focus, like this. &:focus{ outline: none; }
@RusssianMusketeer
Submitted
Any feedback is appreciated, i am learning flexbox now. Slowly improving
@nikkuv
Posted
You can solve accessibility issue by using "aria-label" on input and a button of "type=submit" in form:) that was also my first mistake !! And remove the space from your file name for HTML Validation.