@KetzaliFSubmitted over 3 years ago
hello! Any feedback is received to continue growing.
Youssef Mohamed Hassan
@YoussefMohamed2k19All comments
- @YoussefMohamed2k19Posted over 3 years ago
Hey @Ketzalif, Good Solution but take a look on desktop view not styled good.
1 - @handleryouthSubmitted over 3 years ago
Open to any advice, thank you!
@YoussefMohamed2k19Posted over 3 years agoHey @Handleryouth! -Use bootstrap It'll be easy for using flex box and and position things it'll be easier. -Make fonts
text-align: leftnot center at desktop screens. -Set better colors to borders of buttons and use btn:hover make box-shadow to look good look.0 - @IcEWaRRiOr01Submitted over 3 years ago
Please say me how to adapt the project for mobile devices?
@YoussefMohamed2k19Posted over 3 years agoHey @IcWaRRiOr01, Good solution! You should try upload your website on Netlify or github pages. For responsive screen use media query and decrease width and margin of
class="attribution"and center card.0 - @DuhliSubmitted over 3 years ago
Feedback is highly appreciated, like what could I have done better?
@YoussefMohamed2k19Posted over 3 years agoHey @Duhli check your preview link, It's not working.
0 - @Oliveiras96Submitted over 3 years ago
This project is not done, i want to solve some issues regarding responsive features but for now, all i want is feedback. Also, i know that there's a lot of code refactoring to be done, and i'd love some hints on where to begin (i am a newbie!).
@YoussefMohamed2k19Posted over 3 years agoHey! It was looks awesome. keep coding @Oliveiras96!❤
0 - @EdonmorinaSubmitted over 3 years ago
So i used html and css and a little javascript for the mobile navigation hamburger icon.I had some problems with the responsiveness portion on my desktop version mainly on the scaling of the typography which doesnt help on the responsiveness but the mobile version is full responsive. Some help with the typography would be very much appreciated :) Thanks.
@YoussefMohamed2k19Posted over 3 years agoHey! It looks pretty good!
- At mobile view you can make
margin-left: 10%not0%to center container at center. - At Desktop view remove
margin-top: 7%;atclass="head-box"to make text box more visible.
keep coding ❤
1 - At mobile view you can make
- @alejandroAylSubmitted over 3 years ago
Any feedback is welcome. Thank you!
@YoussefMohamed2k19Posted over 3 years agoHey! It looks pretty good! keep coding ❤
0 - @KerdaelSubmitted over 3 years ago
Hi guys ! As always thanks for your feedback, cheers !
- @KetzaliFSubmitted over 3 years ago
Hello everyone! Any feedback would dbe helpful, I had a few issues whit the curved sections.
@YoussefMohamed2k19Posted over 3 years agoPerfect Solution @ketzalif, keep going❤️!
1 - @just-a-devguySubmitted over 3 years ago
I feel like this was not my best work considering I just wanted to do it as a quick distraction. Please give feedback on what I could've improved. I feel like there might be plenty
@YoussefMohamed2k19Posted over 3 years agoGood Solution @adluders but you should decrease width of laptop image alt="laptop graphic" at responsive screens
1 - @zjerefSubmitted over 3 years ago
Feedback Please
@YoussefMohamed2k19Posted over 3 years agoHey @zjeref! Good work and nice solution, keep going ❤️
0 - @JeremyWaruiSubmitted over 3 years ago
It was a great challenge. Corrections, comments are welcomed.
@YoussefMohamed2k19Posted over 3 years agoHey @JeremyWarui! Good work and nice solution, keep going ❤️
0
