Skip to main content
deleted 1 character in body
Source Link
kleinfreund
  • 3.7k
  • 3
  • 32
  • 63

Some quick notes:

  1. You can use list-style: none; instead of list-style-type: none; – Just a bit shorter

  2. You mention in a comment /* only for tablet */, but there are no media queries in sight.

     @media screen and (max-width: 480px) {
         .international {
             max-width: 240px;
         }
     }
    
  3. It's common practise to use dash-delimited class names in CSS, not CamelCase. This completely personal and you can do whatever you want.

  4. OmmitOmit the units on zero values: background-position: 0 0; instead of background-position: 0px 0px;

  5. I recommend writing single-line CSS: Every property declaration on a new line. There is no reason to write rule declarations with only one property declaration on one line. Also I don't find it helpful writing top, right, bottom and left on one line.

If you want to save file size and improve performance, you should minimize your CSS. That's so much more efficient plus you keep the legibility and maintainability in your CSS file.

That's it for the moment. Feel free to add a comment to discuss something or visit our great chat.

Some quick notes:

  1. You can use list-style: none; instead of list-style-type: none; – Just a bit shorter

  2. You mention in a comment /* only for tablet */, but there are no media queries in sight.

     @media screen and (max-width: 480px) {
         .international {
             max-width: 240px;
         }
     }
    
  3. It's common practise to use dash-delimited class names in CSS, not CamelCase. This completely personal and you can do whatever you want.

  4. Ommit the units on zero values: background-position: 0 0; instead of background-position: 0px 0px;

  5. I recommend writing single-line CSS: Every property declaration on a new line. There is no reason to write rule declarations with only one property declaration on one line. Also I don't find it helpful writing top, right, bottom and left on one line.

If you want to save file size and improve performance, you should minimize your CSS. That's so much more efficient plus you keep the legibility and maintainability in your CSS file.

That's it for the moment. Feel free to add a comment to discuss something or visit our great chat.

Some quick notes:

  1. You can use list-style: none; instead of list-style-type: none; – Just a bit shorter

  2. You mention in a comment /* only for tablet */, but there are no media queries in sight.

     @media screen and (max-width: 480px) {
         .international {
             max-width: 240px;
         }
     }
    
  3. It's common practise to use dash-delimited class names in CSS, not CamelCase. This completely personal and you can do whatever you want.

  4. Omit the units on zero values: background-position: 0 0; instead of background-position: 0px 0px;

  5. I recommend writing single-line CSS: Every property declaration on a new line. There is no reason to write rule declarations with only one property declaration on one line. Also I don't find it helpful writing top, right, bottom and left on one line.

If you want to save file size and improve performance, you should minimize your CSS. That's so much more efficient plus you keep the legibility and maintainability in your CSS file.

That's it for the moment. Feel free to add a comment to discuss something or visit our great chat.

Source Link
kleinfreund
  • 3.7k
  • 3
  • 32
  • 63

Some quick notes:

  1. You can use list-style: none; instead of list-style-type: none; – Just a bit shorter

  2. You mention in a comment /* only for tablet */, but there are no media queries in sight.

     @media screen and (max-width: 480px) {
         .international {
             max-width: 240px;
         }
     }
    
  3. It's common practise to use dash-delimited class names in CSS, not CamelCase. This completely personal and you can do whatever you want.

  4. Ommit the units on zero values: background-position: 0 0; instead of background-position: 0px 0px;

  5. I recommend writing single-line CSS: Every property declaration on a new line. There is no reason to write rule declarations with only one property declaration on one line. Also I don't find it helpful writing top, right, bottom and left on one line.

If you want to save file size and improve performance, you should minimize your CSS. That's so much more efficient plus you keep the legibility and maintainability in your CSS file.

That's it for the moment. Feel free to add a comment to discuss something or visit our great chat.