I would like comments on my styling. I couldn't make the shadow display properly on desktop view. I would also like comments on my use of HTML. I don't think I use the right syntax. Thank you.
Avinash DV
@avinashdvAll comments
- @AyodelearogSubmitted about 2 years ago@avinashdvPosted about 2 years ago
Hi, Nice work. I think you are a little confused with media queries, I would advise a few points that could make the website responsive, please consider these changes.
The below media query states that all the devices that starts from 375px. we can consider 426px instead of 375 as modern mobiles width is much higher than 375px. @media only screen and (min-width: 375px)
Change it to @media only screen and (min-width: 426px) All the code that we write in this media query will get applied to desktop as we stated the min-width 426 and above.
Now for mobile, use @media only screen and (max-width: 426px) All the code that we write inside this will get applied to mobile. It actually states that 0 to max:426 only.
Note: In the starter file, It is given that mobile and desktop as 375px and 1440px. Which therefore means the mobile responsiveness should start from 375px and below, above 375 it should be desktop.
Marked as helpful0 - @Iamweird2Submitted about 2 years ago
Feedbacks highly appreciated :)
@avinashdvPosted about 2 years agoHi, The webpage looks really cool, nice work. A minor UI bug I came across: On selecting the tags, if there are only 1 or 2 jobs, the background color is covered for only the 2 jobs instead of entire body. Changing the h-auto to height:100vh on the div which is inside <div id="__next"> will solve this issue. I hope this helps, happy coding.
Marked as helpful0 - @DannyLenkSubmitted about 2 years ago
Hi. Please, don't pass by. I need your advice! Any advice on where I can improve. And how much do I need to improve to get hired? Thank you for your opinion
@avinashdvPosted about 2 years agoHi, The project is very nice with respect to UI. I haven't gone through code, but one UIchange I could suggest is that: The last row contains only two countries, because justify-content:space-between is used that worked well if the row contains 4 countries, but with 2 or 3 countries there is a lot of space in between the countries and mis-alignment of columns.
I would suggest to utilize 1 or 2:
1
- justify-content: flex-start;
- /* justify-content: space-between; */
- column-gap: 50px;
2 :last-child pseudo element to take care of the last row with css
Note: with (1) the column-gap might not be supported in all browsers and you can check it from https://caniuse.com/ and take the decision to provide support for cross-browser compatibility.
I hope this helps, Happy Coding.
1 - @f0ur3y3sSubmitted about 2 years ago
I'm having trouble putting the woman's desk behind the parent container, if anyone could take a look and put out some ideas that would be great!
I used ReactJS again for this challenge, and every FAQ question can open up to reveal more information. The chevrons even have animations on them to spin 180, and I added some animations for the box on her desk which I thought was a cool little detail. Haven't implemented the mobile view yet, but I plan to do so.
@avinashdvPosted about 2 years agoHi, The table can't be kept behind parent container, then the image go below the white div.
One way to do it is to cut an svg with viewBox by moving them around x & y axis. The first two values are x & y axis where as the next two values are width and height of svg.
Here is a link that works for an svg within a div element: https://jsfiddle.net/avinash_dv/7cfxhuL9/1/
Note: viewBox should be added to <svg > tag.
Reference: https://www.geeksforgeeks.org/svg-viewbox-attribute/
I hope this helps in solving the issue.
0 - @natalie-0073Submitted about 2 years ago@avinashdvPosted about 2 years ago
Hi @natalie-0073 I see that you haven't registered a click event in any of the ratings. They should be clickable and when the user click's on any rating, it should be stored in a variable. Upon clicking the submit button the value stored in the variable should be used to show the rating on "Thank you" Card.
I've written on plain JS using event-delegation. Have a look to get some idea. Repo: https://github.com/avinashdv/interactive-rating
Marked as helpful0 - @JustinFowlerArtSubmitted about 2 years ago@avinashdvPosted about 2 years ago
Hi @JustinFowlerArt UI is neat and clean and is very responsive on all devices. I found a bug functionality wise. Once the user adds the product to cart and upon checkout from cart, the message displayed correct as 'Order received. Thanks for shopping!' and trash icon is showing the appropriate message, but If user adds product to cart again, and now if user clicks on trash icon from cart, I'm getting the msg as order received instead of 'Your cart is empty'. Apart from this, I really liked your work.
Marked as helpful1