Eslam Mohamed• 110
@eslammohamedtolba
Submitted
I am happy to discuss my design with you, All feedback and questions are welcome
Žydrūnas
@ZLikas
All comments
- Žydrūnas• 150
@ZLikas
Posted
Hi, you should use more breakpoints in your layout, for screen size around 831px everything becomes not responsive, as well use margin instead of padding example bellow
main .head div { margin: 0 auto; /* padding: 0 380px; */ font-size: 13px; }
text will not hide under containers. instead of backslash ( \ ) use ( / ) then you will not get HTML issue
0 - Iseoluwa Osho• 180
@theCephas
Submitted
Feedbacks are welcome.
Žydrūnas• 150@ZLikas
Posted
Hi, should use the <p> tag where is text, as well as nest image and content in one container. In mobile preview, there is a gap between image and content. When starting to write code, you should start writing for mobile layout first after that do media query for desktop. Read for more info below https://xd.adobe.com/ideas/process/ui-design/what-is-mobile-first-design/
Marked as helpful
0 @Fran505
Submitted
why the mobile version is not looking good? why my page is not centered in mobile version?
Žydrūnas• 150@ZLikas
Posted
Hi, you should start creating for mobile, then much easier to achieve the desktop layout. Now mobile layout needs to be fixed. As well there is an HTML issue you should use the <main> tag to wrap all main content.
<h1> tag should be used instead of <p>. Don't use \ instead use /.Marked as helpful
0
