Flaviane• 30
@Flaviane-Brum
Submitted
Please provide feedback, so I can do better in the future, thank you.
David Emad
@DavidEmad01
All comments
- David Emad• 925
@DavidEmad01
Posted
Very Nice Work
Just Keep Going
Marked as helpful
0 - Ren Ras• 420
@renras
Submitted
I actually made this as part of an assessment from an employer. Also my first time using sass and vanilla js on a project. Let me know what you guys think.
David Emad• 925@DavidEmad01
Posted
Nice Work Dear
=> You can use:
- transition-duration for the buttons
- hover for links
- hover for social links
Continue Coding ✌✌
Marked as helpful
1 - Tom Scanlon• 10
@tbscanlon
Submitted
My HTML and CSS fundamentals are a bit rusty, so any feedback focused there would be much appreciated!
David Emad• 925@DavidEmad01
Posted
Good Work
- You can use some margin-top for the main box
Keep Going ✌
0 - Mart• 60
@Olu-martins
Submitted
Please what do I need to improve on?
David Emad• 925@DavidEmad01
Posted
Nice Work
- You can remove type="button" from the button for html report
Keep Going ✌✌
Marked as helpful
0 - sdnitrogen• 30
@sdnitrogen
Submitted
My last challenge solution was using CSS Flexbox and this time I've used CSS Grid for this challenge as this layout seemed to be a nice one to implement inside grids. I didn't use grid-areas for this one which I'm still not very comfortable with using but will attempt to use it in my upcoming challenges. I've added an additional tablet layout between the mobile and desktop ones. Please let me know if you have any feedback or suggestions :)
David Emad• 925@DavidEmad01
Posted
Very Nice Work With simple animation 😍
Keep Going ✌✌✌
1 - Andre• 60
@andrecausingg
Submitted
Give some feed back on my mistake
David Emad• 925@DavidEmad01
Posted
Nice Work
You can use hover for the social links
Keep Going ✌✌
0 - Melwyn• 250
@melwynt
Submitted
Hi! Thanks for checking my submission. For this project I tried my best to make this pixel perfect. I used SASS and you can find the code in the git repo.
Let me know how you think this can be improved.
Thanks!
David Emad• 925@DavidEmad01
Posted
Perfect Job 😍😍
Keep Going ✌✌
Marked as helpful
1 - Jakub Łyko• 30
@jaklyk2103
Submitted
https://codesandbox.io/s/reverent-haslett-whjmo
David Emad• 925@DavidEmad01
Posted
Nice Work
You can use for the button:
- curser: pointer;
Keep Going ✌✌
Marked as helpful
0 - Emilio• 90
@E1000o
Submitted
SInce the size of the card is more or less the same for desktop and mobile version, I have used mainly px for heights and widths. The background is a bit off, but at the end of the day I think it's ok. Any comments appreciated. Love
David Emad• 925@DavidEmad01
Posted
Perfect Job 😍😍😍
Continue Coding ✌✌
Marked as helpful
0 - wowza18• 40
@elams18
Submitted
Hi all, after the feedback for the layout, I changed the positioning by using Flex box. Just one query, how do you decide on the sizes of the cards and the like?
Thank you.
David Emad• 925@DavidEmad01
Posted
Nice Work
You can use this for the button:
- border: none;
Keep Going ✌✌
Marked as helpful
1 - shahidAfrid1• 350
@shahidAfrid1
Submitted
feel free to give your feedback or suggestion, thanks in advance!
David Emad• 925@DavidEmad01
Posted
Good Job
But check the responsive for mobile,, as you can use some padding
Continue Coding ✌✌
Marked as helpful
0 - Rosi• 440
@strosi
Submitted
I have no exact question, just wondering if the code is correct and how the end result looks on different devices.
David Emad• 925@DavidEmad01
Posted
Nice Work
I think the code is correct,, You can try more challenges for more experience 😍
Keep Going ✌✌
Marked as helpful
0 - Camila Melo• 120
@IllaMelo
Submitted
I made this challenge. I think it worked, although responsiveness is still difficult for me. If anyone has any suggestions, please, I'm listening!
David Emad• 925@DavidEmad01
Posted
Good Job Dear 🤩
=> You can try the hover for links and the button
Continue Coding ✌✌
Marked as helpful
0 - Web Wizard• 5,690
@rsrclab
Submitted
Please feel free to provide feedback on my code. That will lead me to more perfect way. Thank you ~
David Emad• 925@DavidEmad01
Posted
Very Nice Work 😍😍😍
Continue the challenges and keep going ✌✌
Marked as helpful
1 - Beatriz Reis• 40
@BeaReis
Submitted
Hi! This is my solution for the 'signup form' challenge. Any feedback is highly appreciated!
David Emad• 925@DavidEmad01
Posted
Nice Work 😍😍
= You can try to remove The charset attribute from the script element
Keep Going ✌✌
Marked as helpful
1 - Sara Ferreira• 20
@saraferreira10
Submitted
This is my first challenge here. Feedbacks are welcome; thanks!
David Emad• 925@DavidEmad01
Posted
Very Nice Work as a start 😍😍
Continue the challenges, and Keep Going ✌✌
Marked as helpful
1 - Semih Murat• 40
@smhmurat
Submitted
Thanks for viewing my project. During the project process, I tried to use flexbox especially for multiple cards to come together in this way. However, the process was quite difficult and took longer than I expected. I would be happy if you share the mistakes I made about using responsive layout and flexbox.
David Emad• 925@DavidEmad01
Posted
Very Nice Work
For Html Validation:
use alt for img tag
Keep Going for more challenges ✌✌
0 - Florin.Porut• 380
@zastar23
Submitted
My 3rd project here,
Very smooth coding project, any type of feedback is welcome!
David Emad• 925@DavidEmad01
Posted
Good Job
You can edit the button more:
- outline
- Hover
- Cursor
Keep Going for more challenges ✌✌
Marked as helpful
0 - Okudero Michael• 350
@MicMond01
Submitted
Please don't hesitate to correct my mistakes.
Thank you.
David Emad• 925@DavidEmad01
Posted
Nice Work
You can add some padding at the top
Just Keep Going
Marked as helpful
1 - mohammedmuk• 30
@mohammedmuk
Submitted
show me if I did something wrong
David Emad• 925@DavidEmad01
Posted
Perfect Job 😍😍
You can edit the button of more-info add some animation
Keep Going ✌✌
Marked as helpful
0 - Adrian Hartanto• 60
@adrianh42
Submitted
I'd like some feedback on how to more efficiently write the CSS, also couldn't pick between semantic or utility classes, so it's very messy.
How is my Javascript? are there any unnecessary lines of code or anything that I should add, remove, or change?
I also haven't changed the divs to semantic elements, I'd do it later.
David Emad• 925@DavidEmad01
Posted
Perfect Design 🤩🤩
You can use a little transition-duration on hovering
0 - Jaycee Altamia• 330
@simplyJC
Submitted
Any feedback is highly appreciated. Thanks.
David Emad• 925@DavidEmad01
Posted
Good Job
You can use less box-shadow alpha, for more simple shadow
Continue Coding ✌✌
Marked as helpful
1 - Lucas Gabriell• 50
@LucasGabriell97
Submitted
I am willing to receive feedback for suggestions and improvements to my code.
David Emad• 925@DavidEmad01
Posted
Very Nice Work
I like it 🤩🤩
Just Keep Going for more great things ✌✌
1 - Osasere Edukhure• 20
@MirthOsas
Submitted
Hey guys! I just completed the challenge, feedbacks and criticisms would be appreciated.
David Emad• 925@DavidEmad01
Posted
Nice Work
You can try to edit the height of the boxes; Using some margins and paddings
Continue Coding ✌✌
Marked as helpful
0
