I will like to receive feedbacks on this projects
Enes Eken
@enesekenAll comments
- @Error-at-nightSubmitted over 1 year ago@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 fromGoogle Fonts
(best way) you should read this document.If you are interested in buying website building service, you can check my fiverr profile
1 - @leecockcroftSubmitted over 1 year ago
Please could someone advise best practice for submitting the form and obtaining the value. I was able to get the email value and add this to the text, however I couldnt add the error message. putting required on input and onsubmit, my js didn;t work.
Any other advise/tips please let me know.
@enesekenPosted over 1 year agoU should add "padding: 15px;" to your "image" it will be greater than this 👍
0 - @LuriianSubmitted over 1 year ago@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
Marked as helpful0 - @yetkinkrsngrSubmitted over 1 year ago@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 - @AnnieleoliveiraaSubmitted over 1 year ago
Projeto componente de cartão. Analisem, e me deem feedback !
@enesekenPosted over 1 year agoDo 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 - @panchal-amanSubmitted over 1 year ago@enesekenPosted over 1 year ago
.container:first-child { display: flex; background-color: #4534F2; color: white; border-radius: 20px; flex-direction: column; justify-content: center; height: 100%; }
0 - @Tech-BenSubmitted over 1 year ago
Had fun, make sure you do too. Your preview matters!
- @Y-aburobSubmitted over 1 year ago
I just used Bootstrap to make the card responsive, hope you like it :D
@enesekenPosted over 1 year agoHello @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 helpful0 - @Drayken37Submitted over 1 year ago
- @icewonnSubmitted over 1 year ago
Hi! I am very new to this coding business. Probably I have terrible mistakes. I am open to any feedback. Thanks :)
@enesekenPosted over 1 year agoMerhaba 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.Marked as helpful0 - @Tissyashri77Submitted over 1 year ago@enesekenPosted over 1 year ago0
- @jonhenrikaavitslandSubmitted over 1 year ago
I am unsure if the responsiveness is good enough.
@enesekenPosted over 1 year agoHello 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 ! 🎉
Marked as helpful0 - You should change