All comments

• 

  Solomon Ekrebe• 610

  @krebeDev

  Submitted over 4 years ago

  My first advanced project. Learnt a lot working on this. Feedback is highly appreciated.

  

  REST Countries API with color theme switcher (JavaScript, API)

  1

  7

  0

  

  Mohamed ELIDRISSI• 150

  @MohamedElidrissi

  Posted over 4 years ago

  Nice work, if you want more challenge, you can seperate the functionality into seperate files and use ES Modules with a module bundler such as Webpack or Parcel, or even a framework like React. Good luck!

  1
• 

  HawKyre• 40

  @HawKyre

  Submitted over 4 years ago

  I could not figure out how to put the dot next to "pricing" so I put a dash line instead. Any idea?

  

  Project tracking intro component using HTMLL and CSS

  2

  0

  0

  

  Mohamed ELIDRISSI• 150

  @MohamedElidrissi

  Posted over 4 years ago

  I'd guess you already know it doesn't display well in mobile currently?

  0
• 

  Hamza El Aissaoui• 110

  @kuroyza

  Submitted over 4 years ago

  I would love to know some suggestion to enhance my design, Thank You so much!

  

  CSS Grid, Flexbox, BEM, Desktop first site

  2

  2

  0

  

  Mohamed ELIDRISSI• 150

  @MohamedElidrissi

  Posted over 4 years ago

  Great job Hamza, happy to see more Morrocans here ;) Did you know you can do most of the form validation using CSS only? You can use the :invalid pseudo class in conjunction with the required attribute to trigger stylings only when an input is invalid

  2

  This comment has received 1 replyShow
• 

  Alfonso Suarez G.• 55

  @alfonsosuarezg

  Submitted over 4 years ago

  Is my code readable? That is, I want to know if my HTML code is intuitive and my CSS style sheet well-documented.

  

  Four card feature section (using only floats properties for layout)

  2

  3

  0

  

  Mohamed ELIDRISSI• 150

  @MohamedElidrissi

  Posted over 4 years ago

  I'm in no way an expert, but if I'm using floats I'd make sure it displays properly on IE, here's how it looks on IE11: https://imgur.com/a/Dnc3Xgq

  2

  This comment has received 3 repliesShow
• 

  Joey Bergeron• 30

  @joeybergeron

  Submitted over 4 years ago

  

  RWD, Flexbox, HTML, SCSS, Accessibility

  1

  2

  0

  

  Mohamed ELIDRISSI• 150

  @MohamedElidrissi

  Posted over 4 years ago

  Did you notice at a certain width like 1185px it breaks?

  0

  This comment has received 2 repliesShow
• 

  AlbertoIJS• 135

  @AlbertoIJS

  Submitted over 4 years ago

  • I couldn't get the "we're" with the right font-weight.
  • I did the mobile version with chrome dev tools and it seems to work right but when i re-size the window manually it doesn't work for me.

  Any suggestions are appreciated.

  

  HTML, CSS

  3

  1

  0

  

  Mohamed ELIDRISSI• 150

  @MohamedElidrissi

  Posted over 4 years ago

  I notice that you made the heading text uppercase in your HTML, that will make screen readers spell it word by word(as if its an abbreviation), you should apply a text-transform: uppercase in CSS instead.

  1

  This comment has received 1 replyShow