Desktop First Loopstudios Landing Page

Community feedback

• 

  Gerard Davis•105

  @gerardguydavisPosted about 3 years ago

  Just a few thoughts:

  The animation on your hover selectors looks great, as does your sliding nav menu! They're all really smooth transitions.

  It looks like from the comments in your CSS that you’re working from a mobile-first approach, which is excellent! But you’re using a lot of different breakpoints with both min-width and max-width for various sections of your page. That can possibly make your code buggy and a little tricky to read. For a landing page like this, best practice may be to use no more than 2 or 3 breakpoints, and for a mobile-first approach, you may want to consider sticking with just min-width. (A simple example would be to use min-width: 768px for all of your tablet styles and min-width: 1024px for all of your desktop styles.)

  I’m not quite sure what’s causing your section two text to drop into the section below, but it looks like it may have something to do with using flex column and a relative position. Your HTML organization in this section looks good, so maybe using your min-width for desktop with flex-direction row and position initial (Are you trying to remove the absolute positioning with relative?) will help to keep it within its div.

  Also, just to be super nitpicky, some of your commented out lines are duplicates of lines that are being used (Ex: Lines 275 and 283). Make sure to go through your code and check for these sorts of duplicates, it’ll make reading and editing your code waaay easier in the long run!

  Otherwise, it’s looking really good!

  Marked as helpful

  1

  ---

  

  LuffyDrago•135

  @LuffyDragoPosted about 3 years ago

  @gerardguydavis Thank You for your humble reply and taking your time to review my work. I really appreciate you for spotting and enlightening my errors. It is true I used a lot of different breakpoints which made me write lots of CSS trying to make it Responsive. I appreciate your guidance and will humbly follow to the letter. I also tried fixing by removing the flex column sadly there are no changes on the display. Thank you once more for your honest and detailed feedback.

  0
• Account deleted

  Hi,

  At first glance I thought it looked good, until I saw the text, 'The leader...' mixing with the section below it.

  The mobile navigation stays on even when you switch to desktop view.

  Keep coding👍.

  0

  ---

  

  LuffyDrago•135

  @LuffyDragoPosted about 3 years ago

  @thulanigamtee Thank You for your Feedback

  0