Frontend Mentor | Adarsh Pratap's profile

@AryaDixitSubmitted 10 months ago

This is my step to the world of Web Development. Please let me know if I can improve anything. I would love to get your suggestions.🥰

Responsive New HomePage

1

Adarsh Pratap•5,515

@adarshcodesPosted 10 months ago

Hi @AryaDixit, first of all, welcome to this amazing community! You did nice work on this solution👏.

Few things I've liked and noticed about your solution:

Design aspect: Your design is very similar to the given design file. Only a couple of things I've noticed that you can change the font colors for the paragraphs.

Responsive aspect: It's responsive and will work on most of the screens. Good work.

Functionality aspect: It's working fine.

Code: While everyone has their own style and approach to code few things make the code clear and clean and you really did amazing work on that.

Accessibility and HTML Issue: You can check the report and you'll notice very small issues causing those issues, you can always fix that and regenerate the report here.

You did amazing work👍. Keep practicing. Happy coding :)

1

JFREUN•20

@JFREUNSubmitted about 3 years ago

Hiya!

This is my solution. Any tips or recommendations are welcome!

Order Summary Page

2

Adarsh Pratap•5,515

@adarshcodesPosted about 3 years ago

Hi @JFREUN, you did nice work on this challenge. Your design is very close to the original one. Keep up the good work👏

0

Sean Young•90

@seanyoung247Submitted about 3 years ago

This is my first react project, so I'd be interested to hear all the ways I got react wrong :).

ReactJS Rock Paper Scissors

1

Adarsh Pratap•5,515

@adarshcodesPosted about 3 years ago

Hi @seanyoung247, first of all, let me tell you I've done this project and I know it's really hard. Second thing is you not only nailed it but also took it to another level🤯. I myself am the person who tries to add many extra things and animations in my solution but this solution is really very amazing and I really loved it.😍

The subtle animation and the addition of extras without messing with the actual design. Amazing👏👏👏.

I can't give you the review on your react code as I'm a newbie in react. Just fascinated with the solution.

Happy Coding :)

1

carlin-mitchell•20

@carlin-mitchellSubmitted about 3 years ago

I know that there is a lot for me still to learn. My next goal will be to re-factor this project as I get more experience with best practices.

My main question is: Can I keep the background image centered when I resize the window in only one direction?

Flexbox, Google

2

Adarsh Pratap•5,515

@adarshcodesPosted about 3 years ago

Hi @carlin-mitchell, first of all, welcome to this amazing community, and you did really nice work on your first challenge👏.

And yes there's a huge room to learn the best practices which you'll learn eventually by practicing. Just a suggestion from my side is to keep working on the project consistently and take a look at others' code but don't carry it away by the approach of others. Every one of us has our own style of coding the things you should learn is the best practice and what you like in others' code. Create your own style of code while keeping the best practices in mind.

For your question: leave the background image to the top only, it'll mess with the design if you centered it.

Congratulations on submitting your first solution🎉🎉🎉. It's a long and beautiful way to go. Keep learning and practicing. Happy Coding :)

P.S: take a look at your accessibility and HTML issues and try to fix them. And yeah, Every mistake is an opportunity to learn something new, so never miss it.👍

Marked as helpful

1

Erick Rinald•60

@JrlinkinSubmitted about 3 years ago

Open to feedback, tips, and suggestions! Thank you. :D.

Tip Calculator App

3

Adarsh Pratap•5,515

@adarshcodesPosted about 3 years ago

Hi @Jrlinkin, you did nice work on this solution👏.

Few things I've liked and noticed about your solution:

Design aspect: Your design is very similar to the given design file. Only a couple of things I've noticed missing are the icons on the bills and the number of person input fields.
Responsive aspect: It's responsive and will work on most of the screens. Good work.
Functionality aspect: It's working fine.
Code: While everyone has their own style and approach to code few things make the code clear and clean and you really did amazing work on that. I like you added the comment wherever needed and your code looks really clean.
Accessibility and HTML Issue: You can check the report and you'll notice very small issues causing those issues.

You did great work👍. Keep practicing. Happy coding :)

Marked as helpful

2

Nafsuki•245

@NafsukiSubmitted almost 4 years ago

I coded desktop first, using Sass and grid layout.

It was very challenging for me to keep the image fully covered inside of the div, and change the content's order for the responsive layouts. JS email validation part was very exciting to learn.

Any feedback & suggestion for improvement would be very much appreciated :)!!

Base Apparel Coming Soon Page using Sass, CSS grid and flexbox

3

Adarsh Pratap•5,515

@adarshcodesPosted almost 4 years ago

Hi @Nafsuki, your solution looks really nice.

Design looks similar to the original one, make the width of the container 100%, it will look more similar.
Responsiveness is working fine. Keep it up. Happy Coding😀

1

uBsakuL•90

@uBsakuLSubmitted almost 4 years ago

Hello. This time I did on bigger dimensions (1440x800) and tried to be as accurate as possible. One question, how to change button arrow color? Tried color , and background-color, and fill, still nothing. So I did second button instead. If you have any advices, please feel free to share!

jQuery

3

Adarsh Pratap•5,515

@adarshcodesPosted almost 4 years ago

Hi @FitItGuy, you did good work on this solution. I think most of the things @SzymonRojek has already covered above so please go with the suggestion. Keep it up. Happy coding😀

3