@chri55
Abdulla Bashiroglu
@bashirogluAll comments
- @bashiroglu
Hi, It looks nice, only thing I noticed the list borders in the professional card is not aligned with others, I am not saying this is bad but in the design file, it is aligned, so you can challenge yourself how to accomplish that design. Good luck
- @Hedg0305@bashiroglu
Hi, It is always good to make main logo link and takes to index page, so you may improve by adding that. good luck
- P@aroquega@bashiroglu
Hi @hristoroque , it looks great, you may improve or make it like in design file by increasing padding x direction of main container. good luck
- @rahulxyz@bashiroglu
looks great, you can make it perfect by adding bg image in hero section. good job.
- @Hiaguedes@bashiroglu
hi @Hiaguedes, it looks great, as an improvement you can add correct margins. for example, in Ip address, location, etc they need more space in the bottom. good luck
- @Mostafa-T@bashiroglu
Hi @Mostafa-T, it looks great. I think you forget to add " sign to the background . Good luck
- @sjbcreative@bashiroglu
Hi @sjbcreative, so your solution looks great. And using jquery for this task is okay. If you want to learn vanilla js, you can use dom manipulation. It is easier than jquery in this particular case. Good luck
- @shivamvijaywargi@bashiroglu
HI @shivamvijaywargi , It will be perfect with practice. As a feedback to your work, it already looks well done. as a improvement you can change bg image in mobile by using media queries. Good luck.
- @zuolizhu@bashiroglu
Hi @zuolizhu, this looks amazing. Only thing I noticed, in the form section inputs somehow crapped from the top. especially with "f" it looks clear what I want to say.
- @KimHungChan@bashiroglu
Hi @KimHungChan, everything looks good, It can be improved with some better naming for example buttons container can be changed because they are not probably buttons. Good luck.
- @FelipeDecome@bashiroglu
Hi @FelipeDecome, it looks great, however, i noticed the store card get so small before vertical align, you can add some min width or change vertical design earlier. Good luck.
- @SankThomas@bashiroglu
@SankThomas hi, It really looks good, in order to make it more appealing, you can add some hover effects to button and social icons. Good luck
- @Dark-Lover@bashiroglu
hi @Dark-Lover, as David said you can make container element flex and alignitems and justify content center will solve alignment problem. Good luck.
- @bboumenir@bashiroglu
Hi @bboumenir , Generally it is great but there is a problem in the responsiveness of Keep track of your snippets, you can change to flex column early to solve the problem.
- @sagarkaurav@bashiroglu
@sagarkaurav, It looks good, two things I noticed, probably "try it free" is not a button just div or p. And it needs more space for form in a horizontal direction. Good luck.
- @deepak300811@bashiroglu
Hi @deepak300811, so there are many way what you want to accomplish. You can use keyframes with width property, 0 to 80% or you can access its with js and add 1 per sedond. Good luck.
- @PedroLealLopes@bashiroglu
Hi @PedroLealLopes, as a comment for your react code, you can use destructuring in your components to have cleaner code. for example
const Card = (props) => { const Card = ({isMain }) => {good luck
