Frontend Mentor | Mounir Moutawakil's profile

Tushar Biswas•4,080

@itushSubmitted almost 3 years ago

Hello Developers👋

Saas is awesome...😄

As always, I'd really appreciate if you could answer the following:👇

1.What did I do wrong? 2.What did I do right? 3.How can I improve? common mistakes?

🙏Thanks in advance Happy Coding😄

Fylo landing page with two column layout | Responsive | HTML Sass

1

Mounir Moutawakil•420

@mounir-m4Posted almost 3 years ago

hi you should add a container class in the illustrations sections and set the max-width to 1100px and overflow to hidden to remove the overflow .container{ max-width:1100px; overflow:hidden; } and you'll be good

1

Iuliia Konovalova•260

@IuliiaKonovalovaSubmitted almost 3 years ago

HTML, CSS, GRID, Vanilla JS

2

Mounir Moutawakil•420

@mounir-m4Posted almost 3 years ago

Just add a container class with these specific props and you'll be fine .container{ Max- width:1100px; Margin: 0 auto; Overflow: hidden; } Hope it'll work for you 😊

0

Iuliia Konovalova•260

@IuliiaKonovalovaSubmitted almost 3 years ago

HTML, CSS, GRID, Vanilla JS

2

Mounir Moutawakil•420

@mounir-m4Posted almost 3 years ago

great job! you should only fix the max-width because it's too wide on the footer and on mobile devices, you could remove extra padding and margin from the contact area

0

Yuko-code•235

@Yuko-codeSubmitted almost 3 years ago

I wonder how I can display the error message without adding extra height?

Any feedbacks will be appreciated. Thank you.

Ping single column coming soon page

1

Mounir Moutawakil•420

@mounir-m4Posted almost 3 years ago

hi, yuko! I think if you want to get rid of the added height is that you should do position absolute to the small tag and relative to the parent .

Marked as helpful

0

Mounir Moutawakil•420

@mounir-m4Submitted almost 3 years ago

any feedback would be appreciated

Insure landing Page SCSS with Flexbox & Grid

2

Mounir Moutawakil•420

@mounir-m4Posted almost 3 years ago

hi darryn! i really appreciate that you like written code and about the media query i only did it for mobile and lansacpe (because it's the same process and repetitive)

0

Abdihafid Adan•800

@Abdul400Submitted almost 3 years ago

This is my first time using a new CSS framework. Please leave me any feedback. Thanks :)

huddle landing page made with html, scss, bootstrap, and vanilla js.

1

Mounir Moutawakil•420

@mounir-m4Posted almost 3 years ago

you did a great job ! all you need to increase the font size of the headings and paragraphs

Marked as helpful

1

Kunal Munda•200

@ryu-kamiskiSubmitted almost 3 years ago

Feedback are more than welcome!😊

Also I have some questions!

I have a first priority question is that where should the images be places? In the HTML file itself or the CSS file.
Please do explain me how to make the 5 rated section indentation.
Also I learned about max() min() clamp() and having trouble with how they are behaving with the layout.

Fluid Width - Social Proof Section

1

Mounir Moutawakil•420

@mounir-m4Posted almost 3 years ago

you can easily reset your css style by : *{ margin:0; padding:0; box-sizing:border-box; } and a little bit of advice try to apply flexbox on this sample because is easier to align items and more responsive, compared with what have you done keep learning !

Marked as helpful

1