Skip to content
OverviewSolutions
23
Comments
21
Enes Eken
@eneseken

All comments

  • Enes Eken•540
    @enesekenPosted over 1 year ago

    Hi coder,

    You should add font which is provided in styles-guide.md , if you have any questions about importing fonts from Google Fonts (best way) you should read this document.

    If you are interested in buying website building service, you can check my fiverr profile

    1
  • Enes Eken•540
    @enesekenPosted over 1 year ago

    U should add "padding: 15px;" to your "image" it will be greater than this 👍

    Visit My Website ⭐

    Freelance 🚀

    0
  • Enes Eken•540
    @enesekenPosted over 1 year ago

    U should wrap your elements in a section or main tag

    .main{
display: flex;
justify-content: center;
align-items: center;
min-height: 100svh;
width: 100%;
}

    Why you should use SVH ? Click Here

    Visit My Website ⭐

    Freelance 🚀

    Marked as helpful
    0
  • Enes Eken•540
    @enesekenPosted over 1 year ago

    Abi bu challenge efsaneymiş bende yapacağım (Webscrap ile), bgjar adlı siteden arka plan alabilirsin. Birde responsive olması için her zaman flex'i kullan, ben çoğunlukla flex'i tercih ederim.

    Örnek:

    .smallSide {
display: flex;
flex-direction: row;
flex-wrap: wrap;
gap: 30px;
}
    0
  • Enes Eken•540
    @enesekenPosted almost 2 years ago

    Do this for all of your cards

    align-items: start;
display: flex;
flex-direction: column;

    Also update these:

    .container-boxes {
display: flex;
flex-wrap: wrap;
align-items: center;
justify-content: center;
}

    .container {
display: flex;
justify-content: center;
margin-top: 40px;
flex-wrap: wrap;
}
    0
  • Enes Eken•540
    @enesekenPosted almost 2 years ago
    .container:first-child {
display: flex;
background-color: #4534F2;
color: white;
border-radius: 20px;
flex-direction: column;
justify-content: center;
height: 100%;
}
    0
  • Enes Eken•540
    @enesekenPosted almost 2 years ago

    Your codes are awesome my brother 🔥

    1
  • Enes Eken•540
    @enesekenPosted almost 2 years ago

    Hello @Y-aburob 👋

    I edited the main tag for you, I hope it helps you

    main {
/* margin-top: 50px; */ --> Delete this
text-align: center;
background-image: url(images/pattern-background-desktop.svg);
background-repeat: no-repeat;
background-size: contain;
align-items: center;
justify-content: center;
/* background-position: 0 -20px; */ --> Delete this
min-height: 100svh;
}
    Marked as helpful
    0
  • Enes Eken•540
    @enesekenPosted almost 2 years ago

    You should use text-align:center; and font-size: x;

    0
  • Enes Eken•540
    @enesekenPosted almost 2 years ago

    Merhaba Mert 👋

    Kodun süper , <div class="main"></div> yerine HTML5 ile gelen "Semantic Elements" özelliğini kullanabilirsin. Bunun içinde <main></main> etiketini kullanmalısın.

    Ayrıca <div class="qr"></div> yerine <img/> kullanırsan daha doğru olur.

    Websitemi Ziyaret Et 🌐

    Marked as helpful
    0
  • Enes Eken•540
    @enesekenPosted almost 2 years ago

    Hello coder 👋

    You should add hover

    Good Job ! 🎉

    Visit My Website 🌐

    0
  • Enes Eken•540
    @enesekenPosted almost 2 years ago

    Hello coder 👋

    • You should change <a></a> to <button></button>
    • Also you can add "transition" to <button></button>

    Here is some tips for you 💡

    button{
border: none;
background-color: var(--dark-cyan);
padding: 16px 0 16px 0;
border-radius: 6px;
display: flex;
justify-content: center;
transition: .3s all;
}

    For more about transitions you should visit here

    Good Job ! 🎉

    Visit My Website 🌐

    Marked as helpful
    0
  • Enes Eken•540
    @enesekenPosted almost 2 years ago

    Hello coder 👋

    You should add "transition" to <button></button>

    .card__button {
border: none;
background-color: var(--dark-cyan);
padding: 16px 0 16px 0;
border-radius: 6px;
display: flex;
justify-content: center;
transition: .3s all;
}

    For more about transitions you should visit here

    Good Job ! 🎉

    Visit My Website 🌐

    0
  • Enes Eken•540
    @enesekenPosted almost 2 years ago

    Hello coder 👋

    You Should Change "add-to-button" div to <button></button>

    And also you should add hover What is hover?

    Good Job ! 🎉

    GitHub Source Codes

    Visit My Website 🌐

    0
  • Enes Eken•540
    @enesekenPosted almost 2 years ago

    Hello Coder 👋

    Here is some tips for you 💡

    body {
background-image: url(images/pattern-background-desktop.svg);
background-position: top;
background-repeat: no-repeat;
background-size: contain;
}

    .button_payment:hover{
box-shadow: 0px 5px 2rem hsla(245, 75%, 52%, 0.5); // Delet this in .button_payment
}

    Also you should add "X" before the "Cancel Order"

    GitHub Website

    0
  • Enes Eken•540
    @enesekenPosted almost 2 years ago

    bacground-image and text-color quality is low for UI design

    2
  • Enes Eken•540
    @enesekenPosted almost 2 years ago

    Here is some tips for you I wish these are work;

    First of all you should nested your elements in <main></main> if you want to center a div or something you can use flex properties

    main{
display: flex;
justify-content: center;
align-items: center; 
}

    You can use object-fit for showing propertly your image: object-fit: cover;

    You can use hover effect when mouse on button: button:hover{background-color: x;}

    Also I saw you icon is not same within provided:

    • Warning <img src="images/shopping-cart.png" alt="cart-image">
    • Solved <img src="images/x.SVG" alt="cart-image">

    GitHub Source Codes

    Marked as helpful
    0
  • Enes Eken•540
    @enesekenPosted almost 2 years ago

    It seems cool! I have tip for you button:hover{background-color: red;} dont forget to use classes :)

    Follow me please :(

    0