All comments

• 

  Sujeet Kumar Manjhi• 40

  @alias01

  Submitted almost 3 years ago

  I am learning please provide any feedback. Thanks.

  

  HTML CSS

  1

  10

  

  Nikolay Borzunov• 125

  @tydusgg

  Posted almost 3 years ago

  Hey!

  It is a good job for a newbie but I found some issues.

  First of all, try to avoid using element selectors in CSS: .card-details h1, .stats p , etc. The better is using classes for reusable blocks of style and ids for unique elements (the last is not a rule, just an option).

  If you want to write better CSS - google BEM (it is a kind of approach to writing CSS class names that helps you to reuse styles and extend the code base).

  Secondly, you could use CSS variables. It helps you to don't repeat your code several times, you can just pick a variable.

  But also, I saw you used media queries, it is great for the young developer!

  All in all, it is a very good job for you, you have to read more information about best practices in CSS and frontend at all and try to implement them in your next projects.

  Good luck!

  P.S. Here is some useful stuff.

  1. https://en.bem.info - Info about BEM. You can skip this if you don't understand something and come again later.
  2. https://css-tricks.com/a-complete-guide-to-custom-properties/
  3. https://blog.logrocket.com/5-things-to-consider-when-creating-your-css-style-guide-7b85fa70039d/
  4. https://medium.com/before-semicolon/50-css-best-practices-guidelines-to-write-better-css-c60807e9eee2
  5. https://www.webtips.dev/10-best-practices-for-quickly-improving-your-css

  Marked as helpful

  1

  This comment has received 1 replyShow
• 

  Florin Pavel• 240

  @florinpavel22

  Submitted over 3 years ago

  I haven't figured a way to make the tags filter work properly, at the minute you can filter the jobs only by one tag. If anyone can help I'd appreciate it.

  

  Job Listings [HTML, SASS, JS]

  1

  10

  

  Nikolay Borzunov• 125

  @tydusgg

  Posted over 3 years ago

  I have an idea how to realize multiple filter.

  let arr = [
    {
      // some stuff
      skills: ["JavaScript", "HTML", ......]
    },
    {.......}
  ];
  let filter = ["HTML" "CSS"];
  
  function filterArray(arr, filter){
    return arr.filter( item => filter.every(field => item.skills.includes(field) ) )
  };
  

  And i think you might render array items in JS, not in static HTML.

  Good luck :3

  0

  This comment has received 2 repliesShow
• 

  Abdullah Al Mahin• 60

  @mahin-abdullah

  Submitted over 3 years ago

  Any feedback would be really appreciated.

  

  Manage Landing Page using CSS, SASS.

  1

  10

  

  Nikolay Borzunov• 125

  @tydusgg

  Posted over 3 years ago

  Good job! I found some issues:

  • hamburger does not close when i click outside menu.
  • button hover issue in section after footer.
  • input and button are not on one baseline (in footer).

  1

  This comment has received 1 replyShow
• 

  Thomas Simonet• 50

  @thomas-simonet

  Submitted over 3 years ago

  

  FAQ Accordion Card

  1

  20

  

  Nikolay Borzunov• 125

  @tydusgg

  Posted over 3 years ago

  Good job! Check out my solution, more specifically, JS toggle script. I think it is good practice to avoid height resizing.

  0