@AmneisaOBSubmitted 5 months ago
Latest solutions
Meet Landing Page using Flexbox and Grid
Submitted 5 months agoI would like to ask you to check the code for readability and code duplication. Check responsiveness on different resolutions and give me tips on what I can improve.
Testimonials Grid
Submitted 5 months agoI'm asking for general advice on design, code readability, and responsiveness.
Four card feature section Using Grid
Submitted 6 months agoI would like to ask you to check the code, its readability, aesthetics. And if there are things that can improve responsiveness or cause problems in the future.
Product-preview-card-component using Flexbox
Submitted 6 months agoI would like to ask for general advice on the readability of the code, responsiveness and efficiency of my code.
Responsive Recipe Page
Submitted 6 months agoI still haven't managed to get the list markers to line up perfectly with the text. Is there a way to do this?
Social Link Profile
Submitted 6 months agoPlease review the entire code and check if there are places where it can be improved, written differently or to get rid of errors at different resolutions
Latest comments
- @Extendo99Posted 5 months ago
Hey, great job.
Below I'm posting the mistakes I've noticed and the things you can fix:
- Unfortunately, the page only looks good on the computer. It is not adapted to tablet and mobile resolutions because elements can run away from the page on lower resolutions, e.g. 768px.
I recommend the mobile first approach, i.e. first write the code for the phone and then use media queries to expand and adapt the elements for higher resolutions. Here is a link explaining this approach: https://www.interaction-design.org/literature/topics/mobile-first
-
In my opinion, in this project, you do not need to use flex for every element. You can simply position block objects using paddings and margins.
-
Footer on desktop is not 100% wide.
-
Text has too wide containers. It looks different in the design.
-
Slight differences in margins.
Marked as helpful0 - P@wecaxSubmitted 5 months agoWhat are you most proud of, and what would you do differently next time?
Starting to get the hang of CSS Grid
What specific areas of your project would you like help with?Any feedback is welcome
@Extendo99Posted 5 months agoHey,
Great job. The design looks almost identical. The code is readable and it's great that you use variables and rem units. It's great that you created a design for tablets so that the project will look good on various devices and not just phones and computers.
The only thing missing is a border around the profile pictures.
0 - @Saritha-SasiSubmitted 6 months ago@Extendo99Posted 6 months ago
You should use a margin at the top of the page.
The tabs should be white on the solution.
The mobile view looks incorrect because it is not centered. The text in the paragraph also has no padding, which is why it touches the edge of the screen.
It's great that there is a tablet view implemented, consisting of 4 tabs in a rectangle :)
The code looks okay, but I would group all the rules for a given media query, e.g. 1024, in one place and media 768 in another, because there is currently some confusion. Once there is media for 1024, then for 768, and then without media, which makes it difficult to read.
0 - @justine1607Submitted 6 months ago@Extendo99Posted 6 months ago
The style file should be called style.css
Add the font family that should be displayed if the user doesn't have it installed. In this case:
font-family: "Montserrat", sans-serif; font-family: "Fraunces", serif;
Overall it looks okay :)
1 - @nilmusSubmitted 6 months agoWhat specific areas of your project would you like help with?
I gave up at adhering to the mobile design because I had no idea how to make the padding of only the top image disappear on smaller screens. Any clue is highly appreciated.
@Extendo99Posted 6 months agoHey,
The font colors are different than in the design. Yours are darker. The gaps between the rows in the table are too small. The margins/paddings between the elements are also slightly different, but overall the design looks very good. There are only slight differences in spacing and colors.
As for the mobile view, maybe it would be easier to start with it? The mobile first approach is that we start by coding the view for the phone and then gradually enlarge and adapt the elements to larger screens. I recommend it, I did it myself and had no problems with spacing or the photo.
0 - @ardaacikgozSubmitted 6 months agoWhat are you most proud of, and what would you do differently next time?
I tried to use classes for manipulating CSS rather than using inline CSS.
What challenges did you encounter, and how did you overcome them?Locating the items to the center is hard thing to do. I used display block command and adding margin:0 auto command rather than display flex.
What specific areas of your project would you like help with?It would be great if you check my coding logic, responsiveness, and give feedback which parts I can improve.
@Extendo99Posted 6 months agoThe container has a different color than the one given in the project. There are also differences in the spacing between elements, especially the name and address. The font size also varies.
In CSS, you use enter for each attribute and there is a space. I recommend removing this to make the code more readable. It's better to use an external CSS file as well. Currently, the HTML file is difficult to read due to these CSS styles and it is difficult to make changes to it.
It is better to set the font size using REM
The code is not adapted to the different resolutions of mobile, tablet and desktop.
0
